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
<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
