[
https://issues.apache.org/jira/browse/DAFFODIL-2550?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17413327#comment-17413327
]
Steve Lawrence commented on DAFFODIL-2550:
------------------------------------------
The more I think about it, the more I think it is slightly different than
DAFFODIL-2282. In this particular case, whether we choose "a" or "c" shouldn't
matter. Either way should come back with the same namespace minimization--"a"
and "c" have the exact same namespace. I think the issue here is that if it
picks an enclosing element that doesn't go to the root, then it gets an
incorrect answer because that root is not a Term/Element.
I think the only way DAFFODIL-2282 will fix this is it completely removes the
concept of enclosingElements so we can't possible go back to the wrong
root--like if the algorithm is purely top-down or something. I haven't read the
design of that thoroughly to know if that's the case or not though, so maybe
implementing DAFFODIL-2282 will fix this?
> xmlns="" not minimized away in two CLI tests.
> ---------------------------------------------
>
> Key: DAFFODIL-2550
> URL: https://issues.apache.org/jira/browse/DAFFODIL-2550
> Project: Daffodil
> Issue Type: Bug
> Components: Middle "End"
> Affects Versions: 3.2.0
> Reporter: Mike Beckerle
> Assignee: Steve Lawrence
> Priority: Major
>
> Changes in [https://github.com/apache/daffodil/pull/600] clean up things a
> lot, but there is one mystery change in behavior.
> Two CLI tests now add an xmlns="" to an inner locally-declared element which
> has a unqualified name.
> This seems like incorrect namespace binding minimization, possibly.
> Rather than hold up the merge of PR 600 (which is a big change set, so needs
> to be merged real soon now) this ticket is to investigate this issue of the
> two CLI tests and the xmlns="" that seems unnecessary.
> Note that we've suspected there could be bugs in namespace binding
> minimization for a while now.
> The two CLI tests are:
> * test_CLI_Debugger_InfoHidden_1
> * test_CLI_Debugger_InfoHidden_2
>
>
>
>
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
