Fix #282

Author: cowtowncoder

src/main/java/com/fasterxml/jackson/dataformat/xml/XmlMapper.java

@@ -235,11 +235,10 @@ public void writeValue(XMLStreamWriter w0, Object value) throws IOException {
         @SuppressWarnings("resource")
         ToXmlGenerator g = getFactory().createGenerator(w0);
         super.writeValue(g, value);
-        /* NOTE: above call should do flush(); and we should NOT close here.
-         * Finally, 'g' has no buffers to release.
-         */
+        // NOTE: above call should do flush(); and we should NOT close here.
+        // Finally, 'g' has no buffers to release.
     }
-    
+
     /*
     /**********************************************************
     /* Overridden methods

src/main/java/com/fasterxml/jackson/dataformat/xml/ser/XmlSerializerProvider.java

@@ -54,7 +54,9 @@ public XmlSerializerProvider(XmlSerializerProvider src,
      */
     protected XmlSerializerProvider(XmlSerializerProvider src) {
         super(src);
-        _rootNameLookup = src._rootNameLookup;
+        // 21-May-2018, tatu: As per [dataformat-xml#282], should NOT really copy
+        //    root name lookup as that may link back to diff version, configuration
+        _rootNameLookup = new XmlRootNameLookup();
     }
 
     /*

src/test/java/com/fasterxml/jackson/dataformat/xml/MapperCopyTest.java

@@ -70,7 +70,6 @@ public void testMapperSerialization() throws Exception
         assertEquals("foo", mapper2.getFactory().getXMLTextElementName());
     }
 
-    /*
     // [dataformat-xml#282]
     public void testCopyWith() throws Exception
     {
@@ -90,5 +89,4 @@ public void testCopyWith() throws Exception
             fail("Should NOT use name 'AnnotatedName' but 'Pojo282', xml = "+xml1);
         }
     }
-    */
 }