Update JDK

dmlloyd · Dec 4, 2024 · 8eac262 · 8eac262
1 parent 263b20d
commit 8eac262
Show file tree

Hide file tree

Showing 6 changed files with 38 additions and 36 deletions.
diff --git a/jdk b/jdk
diff --git a/src/main/java/io/github/dmlloyd/classfile/AnnotationValue.java b/src/main/java/io/github/dmlloyd/classfile/AnnotationValue.java
@@ -456,9 +456,11 @@ default ClassDesc classSymbol() {
      *
      * @apiNote
      * {@code TAG_}-prefixed constants in this class, such as {@link #TAG_INT},
-     * describe the possible return values of this method.
+     * describe the possible return values of this method.  The return type is
+     * {@code int} for consistency with union indicator items in other union
+     * structures in the {@code class} file format.
      */
-    char tag();
+    int tag();
 
     /**
      * {@return an enum value for an element-value pair}

diff --git a/src/main/java/io/github/dmlloyd/classfile/constantpool/PoolEntry.java b/src/main/java/io/github/dmlloyd/classfile/constantpool/PoolEntry.java
@@ -98,7 +98,7 @@ public sealed interface PoolEntry
      * {@code TAG_}-prefixed constants in this class, such as {@link #TAG_UTF8},
      * describe the possible return values of this method.
      */
-    byte tag();
+    int tag();
 
     /**
      * {@return the index within the constant pool corresponding to this entry}

diff --git a/src/main/java/io/github/dmlloyd/classfile/impl/AbstractPoolEntry.java b/src/main/java/io/github/dmlloyd/classfile/impl/AbstractPoolEntry.java
@@ -106,7 +106,7 @@ public int hashCode() {
         return hash;
     }
 
-    public abstract byte tag();
+    public abstract int tag();
 
     public int width() {
         return 1;
@@ -181,7 +181,7 @@ enum State { RAW, BYTE, CHAR, STRING }
         }
 
         @Override
-        public byte tag() {
+        public int tag() {
             return TAG_UTF8;
         }
 
@@ -522,7 +522,7 @@ public static final class ClassEntryImpl extends AbstractNamedEntry implements C
         }
 
         @Override
-        public byte tag() {
+        public int tag() {
             return TAG_CLASS;
         }
 
@@ -582,7 +582,7 @@ public static final class PackageEntryImpl extends AbstractNamedEntry implements
         }
 
         @Override
-        public byte tag() {
+        public int tag() {
             return TAG_PACKAGE;
         }
 
@@ -613,7 +613,7 @@ public static final class ModuleEntryImpl extends AbstractNamedEntry implements
         }
 
         @Override
-        public byte tag() {
+        public int tag() {
             return TAG_MODULE;
         }
 
@@ -645,7 +645,7 @@ public static final class NameAndTypeEntryImpl extends AbstractRefsEntry<Utf8Ent
         }
 
         @Override
-        public byte tag() {
+        public int tag() {
             return TAG_NAME_AND_TYPE;
         }
 
@@ -719,7 +719,7 @@ public static final class FieldRefEntryImpl extends AbstractMemberRefEntry imple
         }
 
         @Override
-        public byte tag() {
+        public int tag() {
             return TAG_FIELDREF;
         }
 
@@ -737,7 +737,7 @@ public static final class MethodRefEntryImpl extends AbstractMemberRefEntry impl
         }
 
         @Override
-        public byte tag() {
+        public int tag() {
             return TAG_METHODREF;
         }
 
@@ -755,7 +755,7 @@ public static final class InterfaceMethodRefEntryImpl extends AbstractMemberRefE
         }
 
         @Override
-        public byte tag() {
+        public int tag() {
             return TAG_INTERFACE_METHODREF;
         }
 
@@ -849,7 +849,7 @@ public static final class InvokeDynamicEntryImpl
         }
 
         @Override
-        public byte tag() {
+        public int tag() {
             return TAG_INVOKE_DYNAMIC;
         }
 
@@ -874,7 +874,7 @@ public static final class ConstantDynamicEntryImpl extends AbstractDynamicConsta
         }
 
         @Override
-        public byte tag() {
+        public int tag() {
             return TAG_DYNAMIC;
         }
 
@@ -905,7 +905,7 @@ public static final class MethodHandleEntryImpl extends AbstractPoolEntry
         }
 
         @Override
-        public byte tag() {
+        public int tag() {
             return TAG_METHOD_HANDLE;
         }
 
@@ -964,7 +964,7 @@ public static final class MethodTypeEntryImpl
         }
 
         @Override
-        public byte tag() {
+        public int tag() {
             return TAG_METHOD_TYPE;
         }
 
@@ -1002,7 +1002,7 @@ public static final class StringEntryImpl
         }
 
         @Override
-        public byte tag() {
+        public int tag() {
             return TAG_STRING;
         }
 
@@ -1054,7 +1054,7 @@ public static final class IntegerEntryImpl extends AbstractPoolEntry implements
         }
 
         @Override
-        public byte tag() {
+        public int tag() {
             return TAG_INTEGER;
         }
 
@@ -1100,7 +1100,7 @@ public static final class FloatEntryImpl extends AbstractPoolEntry
         }
 
         @Override
-        public byte tag() {
+        public int tag() {
             return TAG_FLOAT;
         }
 
@@ -1145,7 +1145,7 @@ public static final class LongEntryImpl extends AbstractPoolEntry implements Lon
         }
 
         @Override
-        public byte tag() {
+        public int tag() {
             return TAG_LONG;
         }
 
@@ -1195,7 +1195,7 @@ public static final class DoubleEntryImpl extends AbstractPoolEntry implements D
         }
 
         @Override
-        public byte tag() {
+        public int tag() {
             return TAG_DOUBLE;
         }
 

diff --git a/src/main/java/io/github/dmlloyd/classfile/impl/AnnotationImpl.java b/src/main/java/io/github/dmlloyd/classfile/impl/AnnotationImpl.java
@@ -72,7 +72,7 @@ public String toString() {
     public record OfStringImpl(Utf8Entry constant)
             implements AnnotationValue.OfString {
         @Override
-        public char tag() {
+        public int tag() {
             return TAG_STRING;
         }
 
@@ -85,7 +85,7 @@ public String stringValue() {
     public record OfDoubleImpl(DoubleEntry constant)
             implements AnnotationValue.OfDouble {
         @Override
-        public char tag() {
+        public int tag() {
             return TAG_DOUBLE;
         }
 
@@ -98,7 +98,7 @@ public double doubleValue() {
     public record OfFloatImpl(FloatEntry constant)
             implements AnnotationValue.OfFloat {
         @Override
-        public char tag() {
+        public int tag() {
             return TAG_FLOAT;
         }
 
@@ -111,7 +111,7 @@ public float floatValue() {
     public record OfLongImpl(LongEntry constant)
             implements AnnotationValue.OfLong {
         @Override
-        public char tag() {
+        public int tag() {
             return TAG_LONG;
         }
 
@@ -124,7 +124,7 @@ public long longValue() {
     public record OfIntImpl(IntegerEntry constant)
             implements AnnotationValue.OfInt {
         @Override
-        public char tag() {
+        public int tag() {
             return TAG_INT;
         }
 
@@ -137,7 +137,7 @@ public int intValue() {
     public record OfShortImpl(IntegerEntry constant)
             implements AnnotationValue.OfShort {
         @Override
-        public char tag() {
+        public int tag() {
             return TAG_SHORT;
         }
 
@@ -150,7 +150,7 @@ public short shortValue() {
     public record OfCharImpl(IntegerEntry constant)
             implements AnnotationValue.OfChar {
         @Override
-        public char tag() {
+        public int tag() {
             return TAG_CHAR;
         }
 
@@ -163,7 +163,7 @@ public char charValue() {
     public record OfByteImpl(IntegerEntry constant)
             implements AnnotationValue.OfByte {
         @Override
-        public char tag() {
+        public int tag() {
             return TAG_BYTE;
         }
 
@@ -176,7 +176,7 @@ public byte byteValue() {
     public record OfBooleanImpl(IntegerEntry constant)
             implements AnnotationValue.OfBoolean {
         @Override
-        public char tag() {
+        public int tag() {
             return TAG_BOOLEAN;
         }
 
@@ -193,31 +193,31 @@ public record OfArrayImpl(List<AnnotationValue> values)
         }
 
         @Override
-        public char tag() {
+        public int tag() {
             return TAG_ARRAY;
         }
     }
 
     public record OfEnumImpl(Utf8Entry className, Utf8Entry constantName)
             implements AnnotationValue.OfEnum {
         @Override
-        public char tag() {
+        public int tag() {
             return TAG_ENUM;
         }
     }
 
     public record OfAnnotationImpl(Annotation annotation)
             implements AnnotationValue.OfAnnotation {
         @Override
-        public char tag() {
+        public int tag() {
             return TAG_ANNOTATION;
         }
     }
 
     public record OfClassImpl(Utf8Entry className)
             implements AnnotationValue.OfClass {
         @Override
-        public char tag() {
+        public int tag() {
             return TAG_CLASS;
         }
     }

diff --git a/src/main/java/io/github/dmlloyd/classfile/impl/AnnotationReader.java b/src/main/java/io/github/dmlloyd/classfile/impl/AnnotationReader.java
@@ -351,7 +351,7 @@ public static void writeTypeAnnotations(BufWriter buf, List<TypeAnnotation> list
     public static void writeAnnotationValue(BufWriterImpl buf, AnnotationValue value) {
         var tag = value.tag();
         buf.writeU1(tag);
-        switch (value.tag()) {
+        switch (tag) {
             case TAG_BOOLEAN, TAG_BYTE, TAG_CHAR, TAG_DOUBLE, TAG_FLOAT, TAG_INT, TAG_LONG, TAG_SHORT, TAG_STRING ->
                     buf.writeIndex(((AnnotationValue.OfConstant) value).constant());
             case TAG_CLASS -> buf.writeIndex(((AnnotationValue.OfClass) value).className());