[11] RFR: (JAXP) 8038043: Xerces Update: XInclude update
Joe Wang
huizhe.wang at oracle.com
Mon Feb 26 17:11:56 UTC 2018
Looks good. Thanks!
Best,
Joe
On 2/26/2018 7:34 AM, Aleks Efimov wrote:
> Hi Joe,
>
> Thank you for the review. I've updated XIncludeHandler and
> XIncludeTextReader files per your suggestions.
> XIncludeTextReader:191 line is still shown in Sdiffs as one line, but
> patch contains correct change that splits it.
>
> New webrev: http://cr.openjdk.java.net/~aefimov/8038043/11/01
>
> Best,
> Aleksei
>
>
> On 02/22/2018 08:39 AM, Joe Wang wrote:
>> Hi Aleksei,
>>
>> Thanks for taking the time to work on this!
>>
>> Looks good overall.
>>
>> XIncludeHandler: setupCurrentBaseURI method can be private.
>>
>> XIncludeTextReader: there's a very long line at 191. It would be good
>> to fix it so that Sdiffs looks better the next time.
>>
>> As for the tests, I'm fine with the encoding tests plus passing all
>> existing ones.
>>
>> Best,
>> Joe
>>
>> On 2/16/2018 9:53 AM, Aleks Efimov wrote:
>>> Hi,
>>>
>>> Please, help to review the update of XInclude related classes from
>>> the Apache Xerces 2.11.0 source.
>>> JBS:
>>> https://bugs.openjdk.java.net/browse/JDK-8038043
>>> The webrev:
>>> http://cr.openjdk.java.net/~aefimov/8038043/11/00/
>>>
>>> New regression test has been added to check the updated reporting of
>>> invalid bytes encountered during the parsing and inclusion of XML
>>> documents. New test and other regression tests shows no failures.
>>>
>>> With Best Regards,
>>> Aleksei
>>
>
More information about the core-libs-dev
mailing list