Rename XmlGeneratorInitializer.setDTD() to XmlGeneratorInitializer.addDTD()

Author: cowtowncoder

src/main/java/tools/jackson/dataformat/xml/ser/XmlGeneratorInitializer.java

@@ -31,13 +31,13 @@ public void initialize(SerializationConfig config, JsonGenerator g) throws Jacks
         }
     }
 
-    public XmlGeneratorInitializer setDTD(String rootName,
+    public XmlGeneratorInitializer addDTD(String rootName,
             String systemId, String publicId,
             String internalSubset) {
-        return setDTD(new DTD(rootName, systemId, publicId, internalSubset));
+        return addDTD(new DTD(rootName, systemId, publicId, internalSubset));
     }
 
-    public XmlGeneratorInitializer setDTD(DTD dtd) {
+    public XmlGeneratorInitializer addDTD(DTD dtd) {
         _dtd = dtd;
         return this;
     }

src/test/java/tools/jackson/dataformat/xml/ser/XmlGeneratorInitializerTest.java

@@ -20,7 +20,7 @@ public void testDTDWithOnlyRootElement() throws Exception
     {
         ObjectWriter w = MAPPER.writer().with(
                 new XmlGeneratorInitializer()
-                        .setDTD("StringBean", null, null, null));
+                        .addDTD("StringBean", null, null, null));
         assertEquals(a2q("<!DOCTYPE StringBean>"
                 +"<StringBean><text>test</text></StringBean>"),
                 w.writeValueAsString(new StringBean("test")));
@@ -31,7 +31,7 @@ public void testDTDWithPublicId() throws Exception
     {
         ObjectWriter w = MAPPER.writer().with(
                 new XmlGeneratorInitializer()
-                        .setDTD("StringBean", "system", "http://foo.bar", ""));
+                        .addDTD("StringBean", "system", "http://foo.bar", ""));
         assertEquals(a2q("<!DOCTYPE StringBean PUBLIC 'http://foo.bar' 'system'>"
                 +"<StringBean><text>test</text></StringBean>"),
                 w.writeValueAsString(new StringBean("test")));
@@ -42,7 +42,7 @@ public void testDTDWithSystemIdOnly() throws Exception
     {
         ObjectWriter w = MAPPER.writer().with(
                 new XmlGeneratorInitializer()
-                        .setDTD("StringBean", "system", "", null));
+                        .addDTD("StringBean", "system", "", null));
         assertEquals(a2q("<!DOCTYPE StringBean SYSTEM 'system'>"
                 +"<StringBean><text>test</text></StringBean>"),
                 w.writeValueAsString(new StringBean("test")));
@@ -53,7 +53,7 @@ public void testDTDWithInternalSubset() throws Exception
     {
         ObjectWriter w = MAPPER.writer().with(
                 new XmlGeneratorInitializer()
-                        .setDTD("StringBean", "system", "http://foo.bar", "<!ELEMENT root (#PCDATA)>"));
+                        .addDTD("StringBean", "system", "http://foo.bar", "<!ELEMENT root (#PCDATA)>"));
         assertEquals(a2q("<!DOCTYPE StringBean PUBLIC 'http://foo.bar' 'system' "
                 +"[<!ELEMENT root (#PCDATA)>]>"
                 +"<StringBean><text>test</text></StringBean>"),
@@ -69,7 +69,7 @@ public void testDTDWithXmlDeclaration() throws Exception
                 .build();
         ObjectWriter w = mapper.writer().with(
                 new XmlGeneratorInitializer()
-                        .setDTD("StringBean", "system", "http://foo.bar", null));
+                        .addDTD("StringBean", "system", "http://foo.bar", null));
         // XML declaration is emitted with single quotes, DOCTYPE with double quotes,
         // so cannot use a2q() on the whole string here.
         assertEquals("<?xml version='1.0' encoding='UTF-8'?>"
@@ -85,7 +85,7 @@ public void testDTDInvalidNoRoot() throws Exception
         try {
             /*ObjectWriter w =*/ MAPPER.writer().with(
                 new XmlGeneratorInitializer()
-                    .setDTD("", null, null, null));
+                    .addDTD("", null, null, null));
             fail("Should not pass");
         } catch (IllegalArgumentException e) {
             verifyException(e, "Illegal argument for 'rootName': must be");