tuxji opened a new pull request #385: URL: https://github.com/apache/incubator-daffodil/pull/385
An element with both initiatedContent="yes" and an initiator which accepts the empty string should be a schema definition error. Fix the bug and add TDML delimiter tests to check that it is fixed. Change ContantExpression.isKnownNonEmpty in CompiledExpression1 to check against "%ES;" and "%WSP*;" as well as "". Note, however, that checking for "%WSP*;" by itself breaks three other delimiter tests since isKnownNonEmpty gets called for dfdl:terminator as well as dfdl:initiator. Adding a check for qn.toQName == "dfdl:initiator" as well as value == "%WSP*;" avoids breaking the other tests, but this fix will need review to determine if it makes sense and is complete enough. DAFFODIL-2199 ---------------------------------------------------------------- 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]
