Big patch changing semantics of ReadStream::eos():

eos() now only returns true _after_ trying to read past the end of the stream.

This has a large potential for regressions. Please test!

svn-id: r34549

1 files changed, 6 insertions, 2 deletions

diff --git a/test/common/bufferedreadstream.h b/test/common/bufferedreadstream.h
index 2644745af2..c580fd18de 100644
--- a/test/common/bufferedreadstream.h
+++ b/test/common/bufferedreadstream.h
@@ -9,7 +9,7 @@ class BufferedReadStreamTestSuite : public CxxTest::TestSuite {
 		Common::MemoryReadStream ms(contents, 10);
 
 		// Use a buffer size of 4 -- note that 10 % 4 != 0,
-		// so we test what happens if the cache can't be completly
+		// so we test what happens if the cache can't be completely
 		// refilled.
 		Common::BufferedReadStream srs(&ms, 4);
 
@@ -21,6 +21,10 @@ class BufferedReadStreamTestSuite : public CxxTest::TestSuite {
 			TS_ASSERT_EQUALS( i, b );
 		}
 
-		TS_ASSERT( srs.eos() );
+		TS_ASSERT( !srs.eos() );
+
+		b = srs.readByte();
+
+		TS_ASSERT ( srs.eos() );
 	}
 };