> My personal instinct, unless anyone knows a good reason not, would be to > remove this specific synchronization point.
The docs are quite specific that a DocumentBuilderFactory is *not* guaranteed to be thread-safe: http://java.sun.com/j2se/1.4.2/docs/api/javax/xml/parsers/DocumentBuilderFac tory.html As noted in the docs, it's fine to have one DocumentBuilderFactory per thread though. Best regards, Robert Lowe http://RMLowe.com/