JDK 8 code review request for JDK-8014249 Add Modifer.parameterModifiers() (original) (raw)

Joe Darcy joe.darcy at oracle.com
Wed May 8 20:20:49 UTC 2013


Hello,

Please review my changes to address

 JDK-8014249 Add Modifer.parameterModifiers()

Diff below.

The motivation is that since we now have a reflective model of parameters (JDK-8004729), the java.lang.reflect.Modifier class should provide a parameterModifiers method similar to other "fooModifers" methods already in the class (JDK-6261502).

Thanks,

-Joe

diff -r 2fba6ae13ed8 src/share/classes/java/lang/reflect/Modifier.java --- a/src/share/classes/java/lang/reflect/Modifier.java Tue Apr 30 12:32:49 2013 -0700 +++ b/src/share/classes/java/lang/reflect/Modifier.java Wed May 08 13:16:22 2013 -0700 @@ -351,7 +351,7 @@ }

  /**

Modifier.PRIVATE | @@ -359,7 +359,7 @@ Modifier.STRICT;

  /**

Modifier.PRIVATE | @@ -367,13 +367,13 @@

  /**

Modifier.PRIVATE;

  /**

Modifier.PRIVATE | @@ -381,7 +381,7 @@ Modifier.SYNCHRONIZED | Modifier.NATIVE | Modifier.STRICT;

  /**

Modifier.PRIVATE | @@ -389,6 +389,12 @@ Modifier.VOLATILE;

  /**


More information about the core-libs-dev mailing list