Review Request for 7116997 (warning cleanup java.util.PropertyPermission)

Stuart Marks stuart.marks at
Fri Dec 2 00:06:53 UTC 2011

On 12/1/11 1:18 PM, Brandon Passanisi wrote:
> I have a webrev pertaining to the warning cleanup changes for
> java.util.PropertyPermission ready for review. Here's the info:
> Bug: 7116997
> Webrev:
> <>

Hi Brandon,

Can you take out the @SuppressWarnings from the getMask() method, and change 
the /*FALLTHROUGH*/ comment at line 309 to "break" instead? I think this is 
equivalent code, and it will get rid of the warning message without having to 
suppress warnings.



More information about the core-libs-dev mailing list