Bug 7176907 - Patches for javac warnings cleanup (text and util) from Adopt OpenJDK

Kurchi Subhra Hazra kurchi.subhra.hazra at oracle.com
Wed Jun 20 16:07:53 UTC 2012


Hi,

   I was just going through the patches, there are some more fallthrough 
cases in
src/share/classes/java/util/regex/Pattern.java.(for example in line 2247).
Are these not generating warnings?

- Kurchi

On 6/20/12 7:30 AM, Martijn Verburg wrote:

>>> Hi all,
>>>
>>> Apologies, I didn't check that attachments were stripped.  The patches
>>> can be found at:
>>>
>>>
>>> https://raw.github.com/AdoptOpenJDK/PatchReview/master/submitted/core_java_text.patch
>>>
>>> https://raw.github.com/AdoptOpenJDK/PatchReview/master/submitted/core_java_util.patch
>>>
>>> Cheers,
>>> Martijn
>>
>> Hi Martijn,
>> the two patches looks good.
>>
>> A minor nit, why is there a space between the '(' and the readUByte() in
>> readUShort.
> Thanks for the quick review! No reason on the whitespace, I've fixed that now.
>
> Quick question.  Is there a checkstyle or jcheck that we should be
> applying to any corelib patches going forwards?
>
> Cheers,
> Martijn




More information about the core-libs-dev mailing list