RFR: 8170595: Optimize Class.isAnonymousClass

Mandy Chung mandy.chung at oracle.com
Thu Dec 8 03:30:14 UTC 2016


> On Dec 2, 2016, at 6:29 AM, Claes Redestad <claes.redestad at oracle.com> wrote:
> 
> :
> http://cr.openjdk.java.net/~redestad/8170595/webrev.03/
> 
> This brings significant improvements to some variants:

Good to see the significant improvements.

I think what isLocalOrAnonymousClass needs probably is something like this:

  private boolean hasEnclosingMethodInfo() {
      Object[] enclosingInfo = getEnclosingMethod0();
      if (enclosingInfo != null) {
          EnclosingMethodInfo.validate(enclosingInfo);
      }
      return enclosingInfo != null;
  }

Then EnclosingMethodInfo doesn’t seem the need to change.  The overall change would be simpler.

As Joe suggests, this worths adding test cases for these methods, if not present.

Mandy



More information about the core-libs-dev mailing list