jadams-tresys opened a new pull request #402: URL: https://github.com/apache/incubator-daffodil/pull/402
When the variables infrastructure was changed to be mutable, a changedVariablesStack was introduced to track what variables had been changed in the current scope. These variables would be reset when the PState was reset. The problem was that new lists of variables would only be pushed when entering a point of uncertainty and popped when leaving, however the call to PState.reset() can happen outside of points of uncertainty, such as with nillable elements. This change makes it so that whenever a PState.Mark is created, a new list of changed variables is pushed on the stack. When a mark is discarded or the PState is reset, we pop the stack. DAFFODIL-2374 ---------------------------------------------------------------- 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]
