mbeckerle commented on a change in pull request #276: Updated Unordered
Sequence code to better support vCard
URL: https://github.com/apache/incubator-daffodil/pull/276#discussion_r334552109
##########
File path:
daffodil-runtime1/src/main/scala/org/apache/daffodil/processors/parsers/SequenceChildBases.scala
##########
@@ -154,6 +154,14 @@ object ParseAttemptStatus {
*/
case object MissingItem extends Missing
+ /**
+ * Means that a failure has occurred after succesfully parsing a
discriminator.
+ * For example, in a choice with initiated content if an initiator is
+ * successfully parsed, but the content of the choice branch fails, we should
+ * not continue to attempt to parse the other branches.
+ */
+ case object DiscriminatedFailure extends FailedParseAttemptStatus
Review comment:
Is this used for anything other than unordered sequence? If not then I
suggest UnorderedSeqDiscriminatedFailure as the name.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services