On 2 Sep 2013, at 22:58, Seán Coffey <sean.cof...@oracle.com> wrote:
> Some minor modification (and further simplifying of conditions) : > > http://cr.openjdk.java.net/~coffeys/webrev.8023563.3/webrev/ Looks good to me. -Chris. > > regards, > Sean. > > On 02/09/13 21:07, Alan Bateman wrote: >> On 02/09/2013 19:06, Seán Coffey wrote: >>> This might be a slightly easier one to read. (fast path logic code first) >>> >>> http://cr.openjdk.java.net/~coffeys/webrev.8023563.2/webrev/ >> This is nicer (and I see Chris has already pointed out the redundant check). >> >> -Alan >