The problem was that the base Stream.ReadByte () method allocates a temporary
byte array and then calls Read (..., 1) on it. To solve this, we override
ReadByte () in LogStream and use a private buffer to hold the result.
It's a bit of a mystery to me why Stream.ReadByte () does it this way. The
documentation for Stream explicitly says that instance methods aren't thread
safe, so it would be perfectly fine for Stream.ReadByte () to do what we do
here...
set => throw new NotSupportedException ();
}
+ readonly byte[] _byteBuffer = new byte [1];
+
public LogStream (Stream baseStream)
{
if (baseStream == null)
throw new NotSupportedException ();
}
+ public override int ReadByte ()
+ {
+ // The base method on Stream is extremely inefficient in that it
+ // allocates a 1-byte array for every call. Simply use a private
+ // buffer instead.
+ return Read (_byteBuffer, 0, sizeof (byte)) == 0 ? -1 : _byteBuffer [0];
+ }
+
public override int Read (byte[] buffer, int offset, int count)
{
return BaseStream.Read (buffer, offset, count);