Add caching for expensive methods.

raphw · Mar 25, 2020 · 1eba9dd · 1eba9dd
1 parent 6193017
commit 1eba9dd
Show file tree

Hide file tree

Showing 7 changed files with 21 additions and 4 deletions.
diff --git a/byte-buddy-agent/pom.xml b/byte-buddy-agent/pom.xml
@@ -69,7 +69,7 @@
         <dependency>
             <groupId>net.bytebuddy</groupId>
             <artifactId>byte-buddy</artifactId>
-            <version>1.10.7</version>
+            <version>1.10.8</version>
             <scope>test</scope>
         </dependency>
     </dependencies>

diff --git a/byte-buddy-dep/pom.xml b/byte-buddy-dep/pom.xml
@@ -101,7 +101,7 @@
             <plugin>
                 <groupId>net.bytebuddy</groupId>
                 <artifactId>byte-buddy-maven-plugin</artifactId>
-                <version>1.10.7</version>
+                <version>1.10.8</version>
                 <executions>
                     <execution>
                         <phase>compile</phase>
@@ -116,13 +116,13 @@
                                 <transformation>
                                     <groupId>net.bytebuddy</groupId>
                                     <artifactId>byte-buddy</artifactId>
-                                    <version>1.10.7</version>
+                                    <version>1.10.8</version>
                                     <plugin>net.bytebuddy.build.HashCodeAndEqualsPlugin$WithNonNullableFields</plugin>
                                 </transformation>
                                 <transformation>
                                     <groupId>net.bytebuddy</groupId>
                                     <artifactId>byte-buddy</artifactId>
-                                    <version>1.10.7</version>
+                                    <version>1.10.8</version>
                                     <plugin>net.bytebuddy.build.CachedReturnPlugin</plugin>
                                 </transformation>
                             </transformations>

diff --git a/byte-buddy-dep/src/main/java/net/bytebuddy/description/field/FieldDescription.java b/byte-buddy-dep/src/main/java/net/bytebuddy/description/field/FieldDescription.java
@@ -192,6 +192,7 @@ public SignatureToken asSignatureToken() {
         }
 
         @Override
+        @CachedReturnPlugin.Enhance
         public int hashCode() {
             return getDeclaringType().hashCode() + 31 * (17 + getName().hashCode());
         }

diff --git a/byte-buddy-dep/src/main/java/net/bytebuddy/description/method/MethodDescription.java b/byte-buddy-dep/src/main/java/net/bytebuddy/description/method/MethodDescription.java
@@ -863,6 +863,7 @@ public boolean isBridgeCompatible(TypeToken typeToken) {
         }
 
         @Override
+        @CachedReturnPlugin.Enhance
         public int hashCode() {
             int hashCode = 17 + getDeclaringType().hashCode();
             hashCode = 31 * hashCode + getInternalName().hashCode();

diff --git a/byte-buddy-dep/src/main/java/net/bytebuddy/description/method/ParameterDescription.java b/byte-buddy-dep/src/main/java/net/bytebuddy/description/method/ParameterDescription.java
@@ -16,6 +16,7 @@
 package net.bytebuddy.description.method;
 
 import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
+import net.bytebuddy.build.CachedReturnPlugin;
 import net.bytebuddy.build.HashCodeAndEqualsPlugin;
 import net.bytebuddy.description.ByteCodeElement;
 import net.bytebuddy.description.ModifierReviewable;
@@ -186,6 +187,7 @@ public Token asToken(ElementMatcher<? super TypeDescription> matcher) {
         }
 
         @Override
+        @CachedReturnPlugin.Enhance
         public int hashCode() {
             return getDeclaringMethod().hashCode() ^ getIndex();
         }

diff --git a/byte-buddy-dep/src/main/java/net/bytebuddy/description/type/TypeDescription.java b/byte-buddy-dep/src/main/java/net/bytebuddy/description/type/TypeDescription.java
@@ -3773,6 +3773,7 @@ public Iterator<TypeDefinition> iterator() {
             }
 
             @Override
+            @CachedReturnPlugin.Enhance
             public int hashCode() {
                 return asErasure().hashCode();
             }
@@ -4313,6 +4314,7 @@ public StackSize getStackSize() {
             }
 
             @Override
+            @CachedReturnPlugin.Enhance
             public int hashCode() {
                 return getSort().isNonGeneric()
                         ? asErasure().hashCode()
@@ -4591,6 +4593,7 @@ public StackSize getStackSize() {
             }
 
             @Override
+            @CachedReturnPlugin.Enhance
             public int hashCode() {
                 int lowerHash = 1, upperHash = 1;
                 for (Generic lowerBound : getLowerBounds()) {
@@ -5017,6 +5020,7 @@ public StackSize getStackSize() {
             }
 
             @Override
+            @CachedReturnPlugin.Enhance
             public int hashCode() {
                 int result = 1;
                 for (Generic typeArgument : getTypeArguments()) {
@@ -5601,6 +5605,7 @@ public Iterator<TypeDefinition> iterator() {
             }
 
             @Override
+            @CachedReturnPlugin.Enhance
             public int hashCode() {
                 return getTypeVariableSource().hashCode() ^ getSymbol().hashCode();
             }
@@ -6153,6 +6158,7 @@ public boolean represents(java.lang.reflect.Type type) {
             }
 
             @Override
+            @CachedReturnPlugin.Enhance
             public int hashCode() {
                 return resolve().hashCode();
             }
@@ -7932,6 +7938,7 @@ public Iterator<TypeDefinition> iterator() {
         }
 
         @Override
+        @CachedReturnPlugin.Enhance
         public int hashCode() {
             return getName().hashCode();
         }

diff --git a/byte-buddy-dep/src/main/java/net/bytebuddy/dynamic/scaffold/InstrumentedType.java b/byte-buddy-dep/src/main/java/net/bytebuddy/dynamic/scaffold/InstrumentedType.java
@@ -15,6 +15,7 @@
  */
 package net.bytebuddy.dynamic.scaffold;
 
+import net.bytebuddy.build.CachedReturnPlugin;
 import net.bytebuddy.description.annotation.AnnotationDescription;
 import net.bytebuddy.description.annotation.AnnotationList;
 import net.bytebuddy.description.annotation.AnnotationValue;
@@ -1123,6 +1124,7 @@ public TypeDescription getDeclaringType() {
         /**
          * {@inheritDoc}
          */
+        @CachedReturnPlugin.Enhance
         public Generic getSuperClass() {
             return superClass == null
                     ? Generic.UNDEFINED
@@ -1132,27 +1134,31 @@ public Generic getSuperClass() {
         /**
          * {@inheritDoc}
          */
+        @CachedReturnPlugin.Enhance
         public TypeList.Generic getInterfaces() {
             return new TypeList.Generic.ForDetachedTypes.WithResolvedErasure(interfaceTypes, TypeDescription.Generic.Visitor.Substitutor.ForAttachment.of(this));
         }
 
         /**
          * {@inheritDoc}
          */
+        @CachedReturnPlugin.Enhance
         public FieldList<FieldDescription.InDefinedShape> getDeclaredFields() {
             return new FieldList.ForTokens(this, fieldTokens);
         }
 
         /**
          * {@inheritDoc}
          */
+        @CachedReturnPlugin.Enhance
         public MethodList<MethodDescription.InDefinedShape> getDeclaredMethods() {
             return new MethodList.ForTokens(this, methodTokens);
         }
 
         /**
          * {@inheritDoc}
          */
+        @CachedReturnPlugin.Enhance
         public TypeList.Generic getTypeVariables() {
             return TypeList.Generic.ForDetachedTypes.attachVariables(this, typeVariables);
         }