RFR (JDK10/JAXP) 8181150: Fix lint warnings in JAXP repo: rawtypes and unchecked

[Joe Wang]

Thanks Lance!

Yes, comments are added to NameNodeMapImpl. Those are among a few cases 
where Vector is still used for compatibility.

By the way, Sean also okay-ed the change to FuncHere.java in the 
java.xml.crypto module.

Best,
Joe

On 10/18/17, 12:18 PM, Lance Andersen wrote:
> Hi Joe,
>
> Overall looks good.
>
> As we discussed, I would add a comment to NameNodeMapImpl.java 
> read/writeObject describing the use of Vector for backward 
> compatibility.  No need for another webrev :-)
>
> Best
> Lance
>> On Oct 12, 2017, at 2:24 PM, Joe Wang <huizhe.wang at oracle.com 
>> <mailto:huizhe.wang at oracle.com>> wrote:
>>
>> Hi,
>>
>> Please review a cleanup of rawtypes and unchecked warnings from JAXP 
>> sources.
>>
>> The patch involved a good number of classes (278). However, the 
>> majority of the changes are straight-forward replacement, e.g. 
>> s/Vector/List[ArrayList] and their relevant methods e.g. 
>> s/elementAt/get and etc. In a few cases, 
>> ArrayIndexOutOfBoundsException was substituted with 
>> IndexOutOfBoundsException as they were caught.
>>
>> A few cleanups unrelated to rawtypes and unchecked were made before 
>> Roger's suggestion to stay within the scope of the request, that is, 
>> a couple of StringBuffer substitution, other than that this changeset 
>> is almost rawtypes and unchecked cleanup only.
>>
>> JBS: https://bugs.openjdk.java.net/browse/JDK-8181150
>> webrevs: http://cr.openjdk.java.net/~joehw/jdk10/8181150/webrev/ 
>> <http://cr.openjdk.java.net/%7Ejoehw/jdk10/8181150/webrev/>
>>
>> Thanks,
>> Joe
>>
>
> <http://oracle.com/us/design/oracle-email-sig-198324.gif>
> <http://oracle.com/us/design/oracle-email-sig-198324.gif><http://oracle.com/us/design/oracle-email-sig-198324.gif>
> <http://oracle.com/us/design/oracle-email-sig-198324.gif>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 <mailto:Lance.Andersen at oracle.com>
>
>
>