On May 5 2014, at 11:54 , Ivan Gerasimov <ivan.gerasi...@oracle.com> wrote:
> Thank you Mike! > > On 05.05.2014 21:37, Mike Duigou wrote: >> The fix looks good though I wonder if the conditional checks can be >> optimized a bit. > > Hm. I don't see lots of room for optimization here without revising > nexSetBit() and nextClearBit() implementation. > > The first check at 1190 is already used to catch two cases: > - at the first entrance into the loop, when i was MAX_VALUE, > - at the other cycles, when endOfRun was either MAX_VALUE or -1. OK. (I was just musing, I hadn't thought too hard whether there practical improvements possible) > >> I am concerned that the suggested iteration method provided in the >> nextSetBit() javadoc induces the same failure we just corrected. Perhaps we >> should revise that advice? > Yes. I added a check into the loop. It seems to be the simplest way to make > the sample correct. Agreed. > > Would you please take a look at the updated webrev? > > http://cr.openjdk.java.net/~igerasim/8040806/1/webrev/ Looks good. > > Sincerely yours, > Ivan > >> Mike >> >> On May 5 2014, at 09:56 , Ivan Gerasimov <ivan.gerasi...@oracle.com> wrote: >> >>> Hello! >>> >>> This is a friendly reminder. >>> Could a Reviewer please review/approve this fix? >>> >>> The problem is that a BitSet can have bit # MAX_VALUE set, but trying to >>> convert it to String throws an exception. >>> >>> Sincerely yours, >>> Ivan >>> >>> On 22.04.2014 16:13, Ivan Gerasimov wrote: >>>> Hello! >>>> >>>> If a BitSet has the bit Integer.MAX_VALUE set, then calling toString() >>>> will throw an exception. >>>> The same thing happens if the bit (Integer.MAX_VALUE-1) is set. >>>> >>>> Would you please help review the fix for this issue? >>>> >>>> BUGURL: https://bugs.openjdk.java.net/browse/JDK-8040806 >>>> WEBREV: http://cr.openjdk.java.net/~igerasim/8040806/0/webrev/ >>>> >>>> Sincerely yours, >>>> Ivan >>>> >> >> >