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

Martijn Verburg martijnverburg at gmail.com
Mon Jun 25 12:54:53 UTC 2012


Hi all,

Apologies for the delay.  So it was simply a case of human error in
missing that last fallthrough (we wanted to double check that our
warnings script wasn't failing, hence the delay in getting back to
you). I've respun the patch with the extra SuppressWarning.

Hopefully the patch is complete now.

Cheers,
Martijn

On 20 June 2012 17:18, Martijn Verburg <martijnverburg at gmail.com> wrote:
> We'll look into it, hopefully have an answer for you shortly - M
>
> On 20 June 2012 17:07, Kurchi Subhra Hazra
> <kurchi.subhra.hazra at oracle.com> wrote:
>> 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