On 5/28/2014 3:46, Sean Mullan wrote: > Ok. The fix looks fine to me, though it looks like the code in the 2 > methods is very similar - could it be refactored into a common method? Yes, it's now http://cr.openjdk.java.net/~weijun/8036709/webrev.02/ Thanks Max