On Mon, 2013-07-29 at 19:05 +0400, Andrew Brygin wrote:
> Hi Mario,
>
> thanks for the comments!
>
> I agree that 'stream' and 'streamRef' are not appropriate names now,
> because these entities point to instances of writer/reader.
>
> However, these entities use the writer/reader only as a holder of
> certain set of I/O routines, so probably names for these entities
> should reflect their I/O nature.
>
> Would 'ioRef' and 'input'/'output' sound better than 'stream'?
Hi Andrew,
Those seems better defaults, yes.
At first I got confused with "stream" because the struct used to hold
the Input/Output stream for real. I guess what really bothers me though
is the comment:
jweak streamRef; // ImageInputStream or ImageOutputStream
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
Which is misleading now :) so thanks for addressing this!
Cheers,
Mario
