Hi Frank,
Should you skip '\r' if it's not followed by '\n'?
best regards,
-- daniel
On 14/02/17 10:33, Frank Yuan wrote:
Hi Joe
As you suggested, I made pretty-print a little better based on the fix. That is
when adding indentation, just check the beginning
character(s), in case of '\n' or '\r' then, ignore it/them.
Please check the new webrev:
http://cr.openjdk.java.net/~fyuan/8174025/webrev.01/
Thanks
Frank
-----Original Message-----
From: huizhe wang [mailto:huizhe.w...@oracle.com]
Subject: should have been 8174025 -> Re: RFR [JAXP] JDK-8170192 Regression in
XML Transform caused by JDK-8087303
Note that the bug id was incorrect, it should have been 8174025. 8170192
was a test bug fix.
-Joe
On 2/13/2017 1:35 AM, Frank Yuan wrote:
Hi Joe and Daniel
Thank you very much for your review!
Frank
-----Original Message-----
From: huizhe wang [mailto:huizhe.w...@oracle.com]
Subject: Re: RFR [JAXP] JDK-8170192 Regression in XML Transform caused by
JDK-8087303
+1 from me too.
Thanks,
Joe
On 2/10/2017 5:25 AM, Daniel Fuchs wrote:
Hi Frank,
Thanks for fixing this!
I imported your patch and played with it a bit.
Also ran the jaxp test.
Both issues reported have indeed disappeared.
So that's a +1 from me.
best regards,
-- daniel
On 10/02/17 11:03, Frank Yuan wrote:
Hi All
Would you like to review
http://cr.openjdk.java.net/~fyuan/8174025/webrev.00/?
Bug: https://bugs.openjdk.java.net/browse/JDK-8174025
JDK-8087303 introduced 2 issues:
1. Flaw when xlst uses disable-output-escaping attribute
2. Eat the whitespace between html inline elements
This patch fixed the issues.
To fix the second issue, we decide to keep the compatibility with JDK 8
on the whitespace handling, that is only LSSerializer cleans the extra
whitespaces in if pretty-print is on, but XSLT doesn't.
I modified the behavior of getIndent() method in class ToStream, to make
LSSerializer be sensitive of current state from ToStream. This should be
safe because ToStream is an internal class and getIndent() method is
never used before.
Thanks
Frank
