jadams-tresys commented on code in PR #812:
URL: https://github.com/apache/daffodil/pull/812#discussion_r918070127


##########
daffodil-runtime1/src/main/scala/org/apache/daffodil/processors/DaffodilParseOutputStreamContentHandler.scala:
##########
@@ -309,3 +318,67 @@ class DaffodilParseOutputStreamContentHandler(out: 
OutputStream, pretty: Boolean
     // do nothing
   }
 }
+
+/**
+ * ContentHandler implementation that receives SAX events from 
DaffodilParseXMLReader to output
+ * EXI to the specified outputStream.
+ *
+ * @param out outputStream object to write generated XML to
+ */
+class DaffodilParseOutputStreamEXIContentHandler(out: OutputStream)
+  extends DaffodilParseOutputStreamContentHandler(out) {

Review Comment:
   This class may not be entirely necessary, but was convenient for wrapping 
the EXIResult contentHandler to add a reset() function as well as wrapping in 
the OutputStream that we are outputting to.  This could potentially be replaced 
in the Main daffodil-cli tool with the use of an implicit class that extends 
the EXIResult's content handler to support the reset function and the 
OutputStream, but I think that would be difficult to do if someone was using 
the Java API.



-- 
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]

Reply via email to