[GitHub] [daffodil] stevedlawrence commented on a change in pull request #736: Disable capture value length parsers for all delimited simple types

[GitBox <gi...@apache.org>]

stevedlawrence commented on a change in pull request #736:
URL: https://github.com/apache/daffodil/pull/736#discussion_r791879566



##########
File path: daffodil-core/src/main/scala/org/apache/daffodil/runtime1/ElementBaseRuntime1Mixin.scala
##########
@@ -81,6 +81,9 @@ trait ElementBaseRuntime1Mixin { self: ElementBase =>
 
     // For simple elements with text representation, valueLength is captured in
     // individual parsers since they handle removing delimiters and padding.
+    // Related, simple types that aren't text (e.g. hex binary, packed decimal)
+    // but are delimited also use the same test delimited parsers that capture

Review comment:
       should be "text delimited parsers", will fix




-- 
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: commits-unsubscribe@daffodil.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org