mbeckerle commented on a change in pull request #560:
URL: https://github.com/apache/daffodil/pull/560#discussion_r638071046
##########
File path:
daffodil-cli/src/it/scala/org/apache/daffodil/debugger/TestCLIDebugger.scala
##########
@@ -1265,20 +1265,29 @@ class TestCLIdebugger {
shell.sendLine("step")
shell.sendLine("step")
shell.sendLine("step")
- shell.sendLine("step")
- shell.sendLine("step")
+ //
+ // NOTE FOR REVIEWER
+ //
+ // This test had to be modified to work with this change set.
+ // I have no idea why this change set would affect behavior
+ // of this kind.
+ //
Review comment:
I commented out the new serialization stuff, and there is no change to
the behavior of these tests.
I remain mystified.
It appears that some alignment optimizations are working now, so that
alignment unparsers aren't being created any longer in one of these tests
anyway. That or they're being resolved and aren't in the data structure any
more when it is being displayed.
--
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]
