[kaffe] Problem with BufferedInputStream [patch]
Mark Wielaard
mark at klomp.org
Thu Jun 5 09:43:01 PDT 2003
Hi,
On Thu, 2003-06-05 at 17:54, Mark Wielaard wrote:
> On Thu, 2003-06-05 at 15:48, Helmer Krämer wrote:
> > So I would not completely remove the loop, but rather replace it with
> > something like "do {} while (len>0 && available()>0)"; dunno whether
> > that's really more correct, though...
> >
> > What do you think?
>
> Seems that is more correct. But since available() isn't that reliable I
> wouldn't implement it that way since it might make the method block much
> more then needed. See for example the java.util.ZipInputStream
> available() method definition (which says to almost always returns 1!).
>
> Also note that the javadoc say:
>
> Subclasses of this class are encouraged, but not required, to
> attempt to read as many bytes as possible in the same fashion.
>
> So users cannot rely on this behaviour anyway.
But I did try it out and the attached patch does what you suggest. Makes
the simple ReadLineTest and Snark work correctly and doesn't introduce
Mauve or make check regressions.
Cheers,
Mark
-------------- next part --------------
A non-text attachment was scrubbed...
Name: BufferedInputStream.patch2
Type: text/x-patch
Size: 760 bytes
Desc: not available
Url : http://kaffe.org/pipermail/kaffe/attachments/20030605/d6db917c/attachment-0002.bin
More information about the kaffe
mailing list