RE: [Patch] File descriptor leak in ManagerBase - please advise.

2004-08-31 Thread Renato
Hi, I just saw your fix. I'll give it a try and let you know. Thanks. --- "Shapira, Yoav" <[EMAIL PROTECTED]> wrote: > > Hi, > randomIS is set to null in the getBytes method. But > that's not as good > as an explicit close. So I'll add an explicit close > before setting > randomIS to null.

RE: [Patch] File descriptor leak in ManagerBase - please advise.

2004-08-31 Thread Shapira, Yoav
Hi, randomIS is set to null in the getBytes method. But that's not as good as an explicit close. So I'll add an explicit close before setting randomIS to null. Your patch I don't like as much because it closes the stream before the readLong method is invoked on it. Yoav Shapira Millennium Rese