elharo commented on code in PR #39:
URL: https://github.com/apache/xerces-j/pull/39#discussion_r2462753328


##########
src/org/apache/xml/serialize/SerializerFactory.java:
##########
@@ -34,7 +34,19 @@
  */
 @Deprecated
 public abstract class SerializerFactory {
-    
+
+    /**
+     * This value should be used as a system property for registering 
additional implementations of SerializerFactory.
+     * <p>
+     *     Multiple factories can be registered by using a delimiter between 
the values. Any of the following are supported:
+     * </p>
+     * <ul>
+     *     <li><i>(a single space)</i></li>

Review Comment:
   probably don't need parentheses here



##########
src/org/apache/xml/serialize/SerializerFactory.java:
##########
@@ -132,8 +151,10 @@ public abstract Serializer makeSerializer( Writer writer,
      * using the output byte stream and the encoding specified in the
      * output format.
      *
-     * @throws UnsupportedEncodingException The specified encoding is
-     *   not supported
+     * @param output an output byte stream that will be written to
+     * @param format the output format to use
+     * @return a new serializer for the given output format
+     * @throws UnsupportedEncodingException The specified encoding is not 
supported

Review Comment:
   if the



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to