RFR (JDK10/JAXP) 8181150: Fix lint warnings in JAXP repo: rawtypes and unchecked
Joe Wang
huizhe.wang at oracle.com
Wed Oct 18 20:15:41 UTC 2017
Thanks Roger!
We'll go through a bunch of much smaller patches once this one is out of
the way. After BCEL update and this, we'll go from over 5000 warnings
down to 365 that will be fixed in three separate categories.
-Joe
On 10/18/17, 10:57 AM, Roger Riggs wrote:
> Hi Joe,
>
> The changes look fine.
>
> Maybe do them in smaller batches next time with a consistent kind of
> change. Just thought.
>
> Thanks, Roger
>
>
> On 10/12/17 11:24 AM, Joe Wang 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/
>>
>> Thanks,
>> Joe
>>
>
More information about the core-libs-dev
mailing list