> if (method_newXPath == null) { > synchronized(XmlNotPortable.class) { > if (method_newXPath == null) { > > method_newXPath creation is protected by the sync > and inside this with another null check (double-checked locking idiom). > This could theoretically fail as we have not set > method_newXPath to be "volatile". >
Blast these aging eyes ... thanks for fixing the other issue so quickly!