stevedlawrence commented on code in PR #812:
URL: https://github.com/apache/daffodil/pull/812#discussion_r926882548


##########
daffodil-cli/src/main/scala/org/apache/daffodil/Main.scala:
##########
@@ -873,8 +883,6 @@ object Main {
 
               val parseResult = eitherOutputterOrHandler match {
                 case Right(saxContentHandler) =>
-                  // reset in case we are streaming
-                  saxContentHandler.reset()

Review Comment:
   Skimming through the EXI spec, it says:
   
   > An EXI body can represent an EXI document with a single root element or an 
EXI fragment with zero or more root elements
   
   Maybe when when streaming we use the "fragment" mode and each "infoset" is 
treated as a fragment. Without reading the spec more throughly though, it's not 
clear to me exactly what a fragment represents is and if it is streamable. But 
might be another option and something to look into if concatenated EXI's 
doesn't work.
   
   Note that I don't think we need to add this support in this PR. An error as 
Mike suggests seams reasonable for now.



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