stevedlawrence commented on a change in pull request #262: Unordered sequences
URL: https://github.com/apache/incubator-daffodil/pull/262#discussion_r318129410
##########
File path:
daffodil-core/src/main/scala/org/apache/daffodil/grammar/AlignedMixin.scala
##########
@@ -260,6 +285,7 @@ trait AlignedMixin extends GrammarMixin { self: Term =>
case LengthKind.Prefixed => LengthMultipleOf(1) // NYI
}
}
+ case mg: ModelGroup => LengthMultipleOf(1)
Review comment:
This is a bit surprising. This is called elementSpecifiedLengthApprox, so I
wouldn't expect it to get called on a ModelGroup. Makes me think something else
is wrong somewhere this is called. Any idea how this code path is getting hit?
----------------------------------------------------------------
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
