mbeckerle commented on a change in pull request #748:
URL: https://github.com/apache/daffodil/pull/748#discussion_r802043696



##########
File path: 
daffodil-core/src/main/scala/org/apache/daffodil/grammar/AlignedMixin.scala
##########
@@ -202,9 +202,15 @@ trait AlignedMixin extends GrammarMixin { self: Term =>
       val parentAlignmentApprox =

Review comment:
       Not in this PR, but for users I'm quite sure we're still going to need 
the switches to "just shut the d*mn alignment stuff off" for them. 
   
   It is beyond state-of-the-art in data format description to help people 
debug and give sensible easily followed diagnostics, in these sorts of circular 
or near-circular alignment relationships. "Just shut off alignment" is a 
perfectly acceptable response from users to our currently minimal attempts to 
provide diagnostics. 
   




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