#172 static TFACTORY

yegor256 · yegor256 · commit 56e2a2d3b275 · 2022-08-25T20:27:51.000+03:00
diff --git a/src/main/java/com/jcabi/xml/XMLDocument.java b/src/main/java/com/jcabi/xml/XMLDocument.java
@@ -59,7 +59,6 @@
 import javax.xml.xpath.XPathConstants;
 import javax.xml.xpath.XPathExpressionException;
 import javax.xml.xpath.XPathFactory;
-import lombok.EqualsAndHashCode;
 import org.w3c.dom.Document;
 import org.w3c.dom.Node;
 import org.w3c.dom.NodeList;
diff --git a/src/main/java/com/jcabi/xml/XSLDocument.java b/src/main/java/com/jcabi/xml/XSLDocument.java
@@ -112,6 +112,12 @@ public final class XSLDocument implements XSL {
     private static final DocumentBuilderFactory DFACTORY =
         DocumentBuilderFactory.newInstance();
 
+    /**
+     * Transformer factory.
+     */
+    private static final TransformerFactory TFACTORY =
+        TransformerFactory.newInstance();
+
     /**
      * XSL document.
      */
@@ -411,24 +417,22 @@ private void transformInto(final XML xml, final Result result) {
      * @return The transformer
      */
     private Transformer transformer() {
-        final TransformerFactory factory;
-        synchronized (XSLDocument.class) {
-            factory = TransformerFactory.newInstance();
-        }
-        factory.setURIResolver(this.sources);
         final Transformer trans;
-        try {
-            trans = factory.newTransformer(
-                new StreamSource(new StringReader(this.xsl), this.sid)
-            );
-        } catch (final TransformerConfigurationException ex) {
-            throw new IllegalArgumentException(
-                String.format(
-                    "Failed to create transformer by %s",
-                    factory.getClass().getName()
-                ),
-                ex
-            );
+        synchronized (XSLDocument.TFACTORY) {
+            XSLDocument.TFACTORY.setURIResolver(this.sources);
+            try {
+                trans = XSLDocument.TFACTORY.newTransformer(
+                    new StreamSource(new StringReader(this.xsl), this.sid)
+                );
+            } catch (final TransformerConfigurationException ex) {
+                throw new IllegalArgumentException(
+                    String.format(
+                        "Failed to create transformer by %s",
+                        XSLDocument.TFACTORY.getClass().getName()
+                    ),
+                    ex
+                );
+            }
         }
         XSLDocument.prepare(trans);
         for (final Map.Entry<String, Object> ent : this.params.entrySet()) {
