> 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!

Reply via email to