RFR (JAXP): 8035577: Xerces Update: impl/xpath/regex/RangeToken.java

huizhe wang huizhe.wang at oracle.com
Wed Mar 5 22:10:59 UTC 2014


On 3/5/2014 12:46 PM, Lance Andersen - Oracle wrote:
> On Mar 5, 2014, at 3:37 PM, Xueming Shen wrote:
>
>> nitpicking,
>>
>> (1) shouldn't the variable at #468 to be updated to "lch" instead of "uch" as well?
> I would agree given you are now calling Character.toLowerCase

Also, at line 463 and 464, it looks like 'uppers' meant to be 'lowers' 
defined at 462.

>> (2) StringBuilder can be used to replace the StringBuffer in toString().
> Agree, but I think this could be considered optional for this putback...

Yes, feel free to replace StringBuffer with StringBuilder, Vector to 
ArrayList and etc.

-Joe

>
>
>> -Sherman
>>
>> On 03/05/2014 12:18 PM, David Li wrote:
>>> Hi,
>>>
>>> This is an update from Xerces for file impl/xpath/regex/TokenRange.java.  For details, please refer to: https://bugs.openjdk.java.net/browse/JDK-8035577.
>>>
>>> Webrevs: http://cr.openjdk.java.net/~joehw/jdk9/8035577/webrev/
>>>
>>> No new tests.  There were none added in Xerces.
>>>
>>> Existing tests: JAXP SQE and unit tests passed.
>>>
>>> Thanks,
>>> David
>
>
> Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037
> Oracle Java Engineering
> 1 Network Drive
> Burlington, MA 01803
> Lance.Andersen at oracle.com
>




More information about the core-libs-dev mailing list