stevedlawrence commented on a change in pull request #88: Daffodil 1919 
separators
URL: https://github.com/apache/incubator-daffodil/pull/88#discussion_r208280805
 
 

 ##########
 File path: 
daffodil-core/src/main/scala/org/apache/daffodil/dsom/SequenceGroup.scala
 ##########
 @@ -339,8 +337,89 @@ class Sequence(xmlArg: Node, parent: SchemaComponent, 
position: Int)
   }.value
 }
 
-final class UnorderedSequence(xmlArg: Node, xmlContents: Seq[Node], parent: 
SchemaComponent, position: Int)
-  extends Sequence(xmlArg, parent, position) {
-  // A shell, the actual XML representation is passed in
-  // from Sequence
+/**
+ * A choice branch can be an element with multiple occurrances, or varying 
occurrences.
 
 Review comment:
   A choice branch can also be a sequence, choice, or group ref too, right? But 
this isn't for those, correct? This class only represents <element> branches 
that are not scalars?

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on 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

Reply via email to