This is the mail archive of the java-patches@gcc.gnu.org mailing list for the Java project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]

Re: Merging BufferedInputStream with Classpath


>>>>> "Mark" == Mark Wielaard <mark@klomp.org> writes:

Mark> Note that the patch I posted made refill() a private method.
Mark> This is incorrect since InputStreamReader accesses refill() directly.
Mark> The InputStreamReader.refill() comments actually say:
Mark>     // We have knowledge of the internals of BufferedInputStream
Mark>     // here.  Eww.

Thanks -- with this change it is fine to go in.

Note that the use of refill in InputStreamReader is still a bit wrong.
I think there are still cases where InputStreamReader can fail to make
progress.  This code needs some attention and reworking.

Tom


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]