RFR (JAXP): 8162598 XSLTC transformer swallows empty namespace declaration which is needed to undeclare default namespace

[Langer, Christoph]

Hi Joe,

here is the webrev after merging: http://cr.openjdk.java.net/~clanger/webrevs/8162598.2/

Let me know when you have done your tests - then I'll push it.

@Daniel: Thanks for your help regarding Stack<?>. You were right, I could remove that "<?>".

Thanks & Best regards
Christoph


From: huizhe wang [mailto:huizhe.wang at oracle.com]
Sent: Freitag, 29. Juli 2016 08:04
To: Langer, Christoph <christoph.langer at sap.com>; Daniel Fuchs <daniel.fuchs at oracle.com>
Cc: core-libs-dev at openjdk.java.net
Subject: Re: RFR (JAXP): 8162598 XSLTC transformer swallows empty namespace declaration which is needed to undeclare default namespace

Hi Christoph,
On 7/28/2016 6:10 AM, Langer, Christoph wrote:
Hi,

please review my change for the XSLT namespace issue.

Webrev: http://cr.openjdk.java.net/~clanger/webrevs/8162598.1/<http://cr.openjdk.java.net/%7Eclanger/webrevs/8162598.1/>
Bug: https://bugs.openjdk.java.net/browse/JDK-8162598

The issue has already been discussed in this thread: http://mail.openjdk.java.net/pipermail/core-libs-dev/2016-July/042525.html

Apart from the real fix in LiteralElement.java, method translate(), I've done some further cleanups in a few places. @Joe: The cleanups collide with some places of your proposed change for https://bugs.openjdk.java.net/browse/JDK-8158084 where we'd like to do the same things. So we'll have to synchronize on pushing.

My patch has just pushed. Could you merge the changes and re-generate webrev?  The change looks to be sensitive. I'll build and run all other tests for you.

Best,
Joe



I've also enhanced the test case "TransformerTest" and added a method which does a regression test for the bug reported.

Thanks in advance for reviewing.

Best regards
Christoph