please review 7117612: warnings fixes in java.lang
omajid at redhat.com
Wed Dec 7 23:13:56 UTC 2011
On 12/07/2011 05:43 AM, Alan Bateman wrote:
> On 05/12/2011 22:09, Omair Majid wrote:
>> I have posted a new webrev:
>> This includes all the changes from the feedback so far, as well as
>> changes to ThreadLocal.
>> I will not be posting a patch for ClassValue since (as you kindly
>> pointed out), others are making changes to it and removing some
>> warnings too . I suppose I will revisit it later (if warning fixes
>> are still being accepted :D ).
> I looked through the latest webrev. In
> EnumConstantNotPresentException.java then the
> @SuppressWarnings("rawtypes") should probably have a comment to explain
> why it is needed. In ThreadLocal.get then it's a pity that an additional
> local is needed to increase the size of the method. Otherwise the
> changes look okay to me.
Updated webrev at:
Is there something I should do to address the extra local, or is it good
More information about the core-libs-dev