stevedlawrence commented on code in PR #1314:
URL: https://github.com/apache/daffodil/pull/1314#discussion_r1773803872
##########
daffodil-core/src/main/scala/org/apache/daffodil/core/runtime1/ElementBaseRuntime1Mixin.scala:
##########
@@ -191,6 +191,7 @@ trait ElementBaseRuntime1Mixin { self: ElementBase =>
minimizedScope,
defaultBitOrder,
optPrimType,
+ targetNamespace,
Review Comment:
This was removed by the original patch the avoid duplicating
`targetNamespace` in the runtime data here:
https://github.com/apache/daffodil/commit/2590f54ce93e262d02660bd9d57bc7979a7d06d9#diff-65004632a05e8533fdb90c6f2fc0d9e840ffa93293ae171e54e1cd99591bc540L657
This PR replaces `targetNamespace` with `noPrefixNamespace` but we still
need the original `targetNamespace`, so this change essentially reverts the
original change back to where `targetNamespace` was.
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
