Hey,

I'm currently getting familiar with the source code to eventually contribute
something more in the future. While doing so I noticed some smaller
enhancements where I don't know if they even justify a mail. Please let me
know how you handle such tiny improvements based on the following: 

One of the arguably small improvements was Class.isAnonymousClass() which
checks for emptiness with "".equals(getSimpleName()) instead of
getSimpleName().isEmpty() and I see no way of getSimpleName() returning null
(I might miss something though). Anyhow, the latter is slightly faster and a
bit more verbose:

MyBenchmark.testEmpty   thrpt   20  364479649,385 ± 5805392,007  ops/s
MyBenchmark.testEquals   thrpt   20  287935443,484 ± 2895104,850  ops/s  

Again - if this is too small please let me know and excuse the disturbance.

Cheers,
Christoph

=========== PATCH ============
# User Christoph Dreis <christoph.dr...@freenet.de>
Small enhancement for Class.isAnonymousClass()

diff --git a/src/java.base/share/classes/java/lang/Class.java
b/src/java.base/share/classes/java/lang/Class.java
--- a/src/java.base/share/classes/java/lang/Class.java
+++ b/src/java.base/share/classes/java/lang/Class.java
@@ -1596,7 +1596,7 @@
      * @since 1.5
      */
     public boolean isAnonymousClass() {
-        return "".equals(getSimpleName());
+        return getSimpleName().isEmpty();
     }

Reply via email to