RFR [JAXP] JDK-8174025 Regression in XML Transform caused by JDK-8087303

Frank Yuan frank.yuan at oracle.com
Tue Feb 14 13:43:23 UTC 2017


> -----Original Message-----
> From: Daniel Fuchs [mailto:daniel.fuchs at oracle.com]
> Subject: Re: RFR [JAXP] JDK-8174025 Regression in XML Transform caused by JDK-8087303
> 
> Hi Frank,
> 
> Should you skip '\r' if it's not followed by '\n'?
> 
Does it matter? Since XML processor should normalize the newline.

Thanks
Frank
> 
> 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.wang at 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.wang at 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
> >>>>
> >>>>
> >>>>
> >>
> >
> >




More information about the core-libs-dev mailing list