Some minor modification (and further simplifying of conditions) :
http://cr.openjdk.java.net/~coffeys/webrev.8023563.3/webrev/ 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)This is nicer (and I see Chris has already pointed out the redundant check).http://cr.openjdk.java.net/~coffeys/webrev.8023563.2/webrev/-Alan