please review 7117612: warnings fixes in java.lang
stuart.marks at oracle.com
Thu Dec 8 06:22:09 UTC 2011
On 12/7/11 3:13 PM, Omair Majid wrote:
> On 12/07/2011 05:43 AM, Alan Bateman wrote:
>> 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 to go?
Everything looks good to me. I think Alan was lamenting that adding the local
variable for the sole purpose of adding the @SuppressWarnings annotation makes
the method appear longer and more complex. The alternative is to put
@SuppressWarnings on the entire method, which we've consistently frowned on, so
I don't see the need to change anything.
By the way, I've also run this changeset through our internal multi-platform
build and test system, and everything works fine (with the exception of
intermittent failures unrelated to this change).
You have commit rights, don't you? I'd say it's OK to proceed with the push.
Or, if you prefer, Alan should be online in just a couple hours, and I'm sure
he can give the final go-ahead.
Thanks for working on this!
More information about the core-libs-dev