[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
RE: [cp-patches] FYI: Fixlet for DataInputStream.readLine
From: |
Jeroen Frijters |
Subject: |
RE: [cp-patches] FYI: Fixlet for DataInputStream.readLine |
Date: |
Thu, 16 Jun 2005 11:52:03 +0200 |
Roman Kennke wrote:
> Am Mittwoch, den 15.06.2005, 18:00 +0200 schrieb Jeroen Frijters:
> > Hi Roman,
> >
> > I don't think this is a good idea. It adds a dependency on
> > the internals of two other classes and assumes invariants
> > that may not even be true (for example, think of subclasses
> > of BufferedInputStream). It also seems like it is too
> > complicated for very little gain. Do you have a real
> > world scenario where performance is hurt by the previous approach?
>
> No. I just came over this code while merging GNU Classpath with the
> Aicas code, where we did even worse stuff at this point. If you think
> this is bad, I will reverse my patch. Especially in the light
> that this method is deprecated with the statement 'it doesn't work
> correctly' in the JDK, this will not matter much anyway.
I don't know how others feel about this, but I would be more comfortable
if you took it out.
Thanks.
Regards,
Jeroen