Re: RFR (12): 8191053: Provide a mechanism to make system's security manager immutable

[Sean Mullan]

On 9/13/18 11:01 PM, Weijun Wang wrote:


On Sep 14, 2018, at 4:02 AM, Sean Mullan <sean.mul...@oracle.com> wrote:

webrev: http://cr.openjdk.java.net/~mullan/webrevs/8191053/webrev.00/

      public static SecurityManager getSecurityManager() {
+        if (allowSecurityManager()) {
          return security;
+        } else {
+            return null;
+        }
      }

Is this change really necessary? You have to call a method here.

I'll defer to a compiler expert, but I believe this is necessary to 
cause the constant-folding of this method since the allowSecurityManager 
field has an @Stable annotation.

CSR: https://bugs.openjdk.java.net/browse/JDK-8203316

Still having @Deprecated(since="11").

Fixed. Good catch.

--Sean