bsloane1650 commented on issue #225: Implement daf:lookAhead URL: https://github.com/apache/incubator-daffodil/pull/225#issuecomment-498388966 @mbeckerle I don't think we can merge DState with P/UState. We need to have a (degenerate) DState without a P/UState for constant folding. I also think it makes sense to keep them seperate just as an organizational matter, as our DPath interperatetor is relatively isolated from our main DFDL interperator. A back pointer is probably the way to go. Having said that, it might make sense to merge a bunch of the setters. Now that we are passing in a pointer to the entire state, we might want to stop passing in pointers to indiviual fields of the state, and replace them with functions in DState that refer to the main state object.
---------------------------------------------------------------- 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
