stevedlawrence opened a new pull request #244: Pass findPropertyOption through a ChoiceBranchImpliedSequence to the underlying Term URL: https://github.com/apache/incubator-daffodil/pull/244 The ChoiceCombinator asks all of its child Terms for the choiceBranchKey property (and other properties related to direct dispatch) to build the dispatch mappings. This means that property lookup results are cached on the implied sequence instead of on the Term that the properties actually appear. This leads to the Term thinking the property wasn't used because its access didn't appear it its cache, and leads it an incorrect warning. Instead, pass through the findPropertyOption function to the underlying Term so that look ups happen and are cached on the Term, getting rid of the false warnings. DAFFODIL-2162
---------------------------------------------------------------- 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: us...@infra.apache.org With regards, Apache Git Services
