BUG in java/io/BufferedReader.java ; patch

Henner Zeller kaffe@rufus.w3.org
Tue, 22 Jun 1999 02:49:51 +0200 (CEST)


  This message is in MIME format.  The first part should be readable text,
  while the remaining parts are likely unreadable without MIME-aware tools.
  Send mail to mime@docserver.cac.washington.edu for more info.

--126684171-222267283-930012591=:29008
Content-Type: TEXT/PLAIN; charset=US-ASCII


Hi Archie,
On Mon, 21 Jun 1999, Archie Cobbs wrote:
AC| Do you have a simple test case to demonstrate this bug? I have a slightly
AC| different patch I'd like to try.

In the Attachement. Should return something like 012 / 123 / 234 / ...

I see what you mean, my patch was buggy as well: we've to refill the
Buffer at position 'size', not 'pos':
---
 if (size < readAheadLimit)
   refillBuffer (size)
---

ciao,
  -hen
---
Henner Zeller                                 zeller@fh-heilbronn.de
 PGP pub key [77F75B39]: finger zeller@lemming.stud.fh-heilbronn.de 

 If Microsoft is the answer, it must have been a VERY silly question.

--126684171-222267283-930012591=:29008
Content-Type: TEXT/PLAIN; charset=US-ASCII; name="BufferedReaderBug.java"
Content-ID: <Pine.LNX.4.05.9906220249510.29008@lara.stud.fh-heilbronn.de>
Content-Description: 
Content-Disposition: attachment; filename="BufferedReaderBug.java"
Content-Transfer-Encoding: BASE64

LyoNCiAqIFRlc3QtY2FzZSB0byBjaGVjayB3ZXRoZXIgamF2YS5pby5CdWZm
ZXJlZFJlYWRlciB3b3Jrcy4NCiAqIEguWi4NCiAqLw0KaW1wb3J0IGphdmEu
aW8uQnVmZmVyZWRSZWFkZXI7DQppbXBvcnQgamF2YS5pby5TdHJpbmdSZWFk
ZXI7DQoNCnB1YmxpYyBjbGFzcyBCdWZmZXJlZFJlYWRlckJ1ZyB7DQoNCiAg
ICBwdWJsaWMgc3RhdGljIGludCBSRUFETEVOID0gMzsNCg0KICAgIHB1Ymxp
YyBzdGF0aWMgdm9pZCBtYWluIChTdHJpbmcgYXJndltdKSB7DQoJLyoNCgkg
KiB0aGUgc291cmNlIHdlIHJlYWQgZnJvbS4NCgkgKi8NCglTdHJpbmdSZWFk
ZXIgc291cmNlID0gbmV3IFN0cmluZ1JlYWRlciAoIjAxMjM0NTY3ODlBQkMi
KTsNCg0KCS8qDQoJICogbGltaXQgQnVmZmVyIHNpemUgdG8gNQ0KCSAqLw0K
CUJ1ZmZlcmVkUmVhZGVyIGIgPSBuZXcgQnVmZmVyZWRSZWFkZXIgKHNvdXJj
ZSwgNSk7DQoJY2hhciBidWZbXSA9IG5ldyBjaGFyIFtSRUFETEVOXTsNCgkN
Cgl0cnkgew0KCSAgICBmb3IgKDs7KSB7DQoJCWIubWFyayggUkVBRExFTiAp
Ow0KCQlpbnQgY2hhcnNSZWFkID0gMDsNCgkJaW50IGxlbiA9IDA7DQoJCS8v
IHJlYWQgdW50aWwgcmVxdWlyZWQgc2l6ZSByZWFkIC4uDQoJCXdoaWxlIChj
aGFyc1JlYWQgPCBSRUFETEVOICYmIA0KCQkgICAgICAgKGxlbiA9IGIucmVh
ZCAoYnVmLCBjaGFyc1JlYWQsIFJFQURMRU4tY2hhcnNSZWFkKSkgPj0gMCkN
CgkJICAgIGNoYXJzUmVhZCArPSBsZW47DQoJCWlmIChsZW4gPCAwKSBicmVh
azsgICAvLyBkb25lLg0KCQlTeXN0ZW0uZXJyLnByaW50bG4gKCJyZWFkICIg
KyBidWYpOw0KCQliLnJlc2V0KCk7DQoJCWIucmVhZCgpOyAgICAvLyBuZXh0
IGNoYXINCgkgICAgfQ0KCX0NCgljYXRjaCAoamF2YS5pby5JT0V4Y2VwdGlv
biBlKSB7DQoJICAgIGUucHJpbnRTdGFja1RyYWNlKCk7DQoJfQ0KICAgIH0N
Cn0NCg==
--126684171-222267283-930012591=:29008--