Merge branch '2.16'

FasterXML · Nov 3, 2023 · 5e2350e · 5e2350e
2 parents 86a728d + bef6905
commit 5e2350e
Show file tree

Hide file tree

Showing 26 changed files with 61 additions and 61 deletions.
diff --git a/avro/src/main/java/tools/jackson/dataformat/avro/AvroParser.java b/avro/src/main/java/tools/jackson/dataformat/avro/AvroParser.java
@@ -52,8 +52,8 @@ public static int collectDefaults()
             }
             return flags;
         }
-        
-        private Feature(boolean defaultState) {
+
+        Feature(boolean defaultState) {
             _defaultState = defaultState;
             _mask = (1 << ordinal());
         }

diff --git a/avro/src/main/java/tools/jackson/dataformat/avro/AvroSchema.java b/avro/src/main/java/tools/jackson/dataformat/avro/AvroSchema.java
@@ -34,7 +34,7 @@ public class AvroSchema implements FormatSchema
     /**
      * Lazily instantiated value reader for this schema.
      */
-    protected final AtomicReference<AvroStructureReader> _reader = new AtomicReference<AvroStructureReader>();
+    protected final AtomicReference<AvroStructureReader> _reader = new AtomicReference<>();
 
     public AvroSchema(Schema asch)
     {

diff --git a/avro/src/main/java/tools/jackson/dataformat/avro/deser/AvroFieldDefaulters.java b/avro/src/main/java/tools/jackson/dataformat/avro/deser/AvroFieldDefaulters.java
@@ -42,7 +42,7 @@ public static AvroFieldReader createDefaulter(String name,
         case START_OBJECT:
             {
                 Iterator<Map.Entry<String,JsonNode>> it = defaultAsNode.fields();
-                List<AvroFieldReader> readers = new ArrayList<AvroFieldReader>();
+                List<AvroFieldReader> readers = new ArrayList<>();
                 while (it.hasNext()) {
                     Map.Entry<String,JsonNode> entry = it.next();
                     String propName = entry.getKey();
@@ -52,7 +52,7 @@ public static AvroFieldReader createDefaulter(String name,
             }
         case START_ARRAY:
         {
-            List<AvroFieldReader> readers = new ArrayList<AvroFieldReader>();
+            List<AvroFieldReader> readers = new ArrayList<>();
             for (JsonNode value : defaultAsNode) {
                 readers.add(createDefaulter("", value));
             }

diff --git a/avro/src/main/java/tools/jackson/dataformat/avro/deser/AvroParserImpl.java b/avro/src/main/java/tools/jackson/dataformat/avro/deser/AvroParserImpl.java
@@ -364,7 +364,7 @@ protected final void convertNumberToInt() throws JacksonException
     protected final void convertNumberToLong() throws JacksonException
     {
         if ((_numTypesValid & NR_INT) != 0) {
-            _numberLong = (long) _numberInt;
+            _numberLong = _numberInt;
         } else if ((_numTypesValid & NR_BIGINT) != 0) {
             if (BI_MIN_LONG.compareTo(_numberBigInt) > 0
                     || BI_MAX_LONG.compareTo(_numberBigInt) < 0) {
@@ -443,13 +443,13 @@ protected final void convertNumberToDouble() throws JacksonException
         if ((_numTypesValid & NR_BIGDECIMAL) != 0) {
             _numberDouble = _numberBigDecimal.doubleValue();
         } else if ((_numTypesValid & NR_FLOAT) != 0) {
-            _numberDouble = (double) _numberFloat;
+            _numberDouble = _numberFloat;
         } else if ((_numTypesValid & NR_BIGINT) != 0) {
             _numberDouble = _numberBigInt.doubleValue();
         } else if ((_numTypesValid & NR_LONG) != 0) {
             _numberDouble = (double) _numberLong;
         } else if ((_numTypesValid & NR_INT) != 0) {
-            _numberDouble = (double) _numberInt;
+            _numberDouble = _numberInt;
         } else {
             _throwInternal();
         }

diff --git a/avro/src/main/java/tools/jackson/dataformat/avro/deser/AvroReaderFactory.java b/avro/src/main/java/tools/jackson/dataformat/avro/deser/AvroReaderFactory.java
@@ -27,8 +27,7 @@ public abstract class AvroReaderFactory
      * To resolve cyclic types, need to keep track of resolved named
      * types.
      */
-    protected final TreeMap<String, AvroStructureReader> _knownReaders
-        = new TreeMap<String, AvroStructureReader>();
+    protected final TreeMap<String, AvroStructureReader> _knownReaders = new TreeMap<>();
 
     /*
     /**********************************************************************
@@ -303,10 +302,10 @@ protected AvroStructureReader createRecordReader(Schema writerSchema, Schema rea
 
             // but first: find fields that only exist in reader-side and need defaults,
             // and remove those from
-            Map<String,Schema.Field> readerFields = new HashMap<String,Schema.Field>();
-            List<Schema.Field> defaultFields = new ArrayList<Schema.Field>();
+            Map<String,Schema.Field> readerFields = new HashMap<>();
+            List<Schema.Field> defaultFields = new ArrayList<>();
             {
-                Set<String> writerNames = new HashSet<String>();
+                Set<String> writerNames = new HashSet<>();
                 for (Schema.Field f : writerFields) {
                     writerNames.add(f.name());
                 }

diff --git a/avro/src/main/java/tools/jackson/dataformat/avro/deser/JacksonAvroParserImpl.java b/avro/src/main/java/tools/jackson/dataformat/avro/deser/JacksonAvroParserImpl.java
@@ -377,7 +377,7 @@ public long decodeLong() throws IOException {
         _inputPtr = ptr;
         // should be ok to zigzag as int, then sign-extend
         i = (i >>> 1) ^ (-(i & 1));
-        return (long) i;
+        return i;
     }
 
     private long _decodeLong2(int ptr, long lo) throws IOException
@@ -436,7 +436,7 @@ public long _decodeLongSlow() throws IOException {
             }
         }
         i = (i >>> 1) ^ (-(i & 1));
-        return (long) i;
+        return i;
     }
 
     private long _decodeLongSlow2(long lo) throws IOException

diff --git a/avro/src/main/java/tools/jackson/dataformat/avro/schema/AvroSchemaHelper.java b/avro/src/main/java/tools/jackson/dataformat/avro/schema/AvroSchemaHelper.java
@@ -58,7 +58,7 @@ public abstract class AvroSchemaHelper
     /**
      * Default stringable classes
      */
-    protected static final Set<Class<?>> STRINGABLE_CLASSES = new HashSet<Class<?>>(Arrays.asList(
+    protected static final Set<Class<?>> STRINGABLE_CLASSES = new HashSet<>(Arrays.asList(
             URI.class, URL.class, File.class,
             BigInteger.class, BigDecimal.class,
             String.class
@@ -118,7 +118,7 @@ protected static String getTypeName(JavaType type) {
 
     protected static Schema unionWithNull(Schema otherSchema)
     {
-        List<Schema> schemas = new ArrayList<Schema>();
+        List<Schema> schemas = new ArrayList<>();
         schemas.add(Schema.create(Schema.Type.NULL));
 
         // two cases: existing union
@@ -437,8 +437,7 @@ public FullNameKey(String namespace, String name) {
         }
 
         public String nameWithSeparator(char sep) {
-            final StringBuilder sb = new StringBuilder(1 + _namespace.length() + _name.length());
-            return sb.append(_namespace).append(sep).append(_name).toString();
+            return _namespace + sep + _name;
         }
 
         @Override

diff --git a/avro/src/main/java/tools/jackson/dataformat/avro/schema/DefinedSchemas.java b/avro/src/main/java/tools/jackson/dataformat/avro/schema/DefinedSchemas.java
@@ -13,7 +13,7 @@
  */
 public class DefinedSchemas
 {
-    protected final Map<JavaType, Schema> _schemas = new LinkedHashMap<JavaType, Schema>();
+    protected final Map<JavaType, Schema> _schemas = new LinkedHashMap<>();
 
     protected SerializerProvider _provider;
 

diff --git a/avro/src/main/java/tools/jackson/dataformat/avro/schema/RecordVisitor.java b/avro/src/main/java/tools/jackson/dataformat/avro/schema/RecordVisitor.java
@@ -38,7 +38,7 @@ public class RecordVisitor
 
     protected Schema _avroSchema;
 
-    protected List<Schema.Field> _fields = new ArrayList<Schema.Field>();
+    protected List<Schema.Field> _fields = new ArrayList<>();
 
     public RecordVisitor(SerializerProvider p, JavaType type, VisitorFormatWrapperImpl visitorWrapper)
     {

diff --git a/avro/src/main/java/tools/jackson/dataformat/avro/schema/SchemaBuilder.java b/avro/src/main/java/tools/jackson/dataformat/avro/schema/SchemaBuilder.java
@@ -3,5 +3,5 @@
 import org.apache.avro.Schema;
 
 public interface SchemaBuilder {
-    public Schema builtAvroSchema();
+    Schema builtAvroSchema();
 }
diff --git a/avro/src/main/java/tools/jackson/dataformat/avro/schema/VisitorFormatWrapperImpl.java b/avro/src/main/java/tools/jackson/dataformat/avro/schema/VisitorFormatWrapperImpl.java
@@ -151,12 +151,7 @@ public JsonArrayFormatVisitor expectArrayFormat(final JavaType convertedType) {
         if (convertedType.isArrayType()) {
             JavaType vt = convertedType.getContentType();
             if (vt.hasRawClass(Byte.TYPE)) {
-                _builder = new SchemaBuilder() {
-                    @Override
-                    public Schema builtAvroSchema() {
-                        return AvroSchemaHelper.typedSchema(Schema.Type.BYTES, convertedType);
-                    }
-                };
+                _builder = () -> AvroSchemaHelper.typedSchema(Schema.Type.BYTES, convertedType);
                 return null;
             }
         }
@@ -243,9 +238,6 @@ protected <T> T _throwUnsupported(String msg) {
     }
 
     private boolean _isDateTimeType(JavaType type) {
-        if (Temporal.class.isAssignableFrom(type.getRawClass())) {
-            return true;
-        }
-        return false;
+        return Temporal.class.isAssignableFrom(type.getRawClass());
     }
 }
diff --git a/avro/src/main/java/tools/jackson/dataformat/avro/ser/AvroWriteContext.java b/avro/src/main/java/tools/jackson/dataformat/avro/ser/AvroWriteContext.java
@@ -207,7 +207,7 @@ protected GenericArray<Object> _createArray(Schema schema)
             }
             schema = schema.getTypes().get(arraySchemaIndex);
         }
-        return new GenericData.Array<Object>(8, schema);
+        return new GenericData.Array<>(8, schema);
     }
 
     // Removed from 2.10, should not be needed any more

diff --git a/avro/src/main/java/tools/jackson/dataformat/avro/ser/MapWriteContext.java b/avro/src/main/java/tools/jackson/dataformat/avro/ser/MapWriteContext.java
@@ -20,7 +20,7 @@ public MapWriteContext(AvroWriteContext parent, AvroGenerator generator,
             Schema schema, Object currValue)
     {
         super(parent, generator, schema, currValue);
-        _data = new HashMap<String,Object>();
+        _data = new HashMap<>();
     }
 
     @Override

diff --git a/avro/src/main/java/tools/jackson/dataformat/avro/ser/RootContext.java b/avro/src/main/java/tools/jackson/dataformat/avro/ser/RootContext.java
@@ -134,7 +134,7 @@ protected void _reportError() {
     private final NonBSGenericDatumWriter<Object> _writer() {
         NonBSGenericDatumWriter<Object> w = _writer;
         if (w == null){
-            w = new NonBSGenericDatumWriter<Object>(_schema);
+            w = new NonBSGenericDatumWriter<>(_schema);
             _writer = w;
         }
         return w;

diff --git a/avro/src/test/java/tools/jackson/dataformat/avro/ArrayTest.java b/avro/src/test/java/tools/jackson/dataformat/avro/ArrayTest.java
@@ -3,6 +3,7 @@
 import java.io.ByteArrayInputStream;
 import java.io.ByteArrayOutputStream;
 import java.util.Arrays;
+import java.util.Collections;
 import java.util.List;
 
 import tools.jackson.core.*;
@@ -48,7 +49,7 @@ public void testStringArraySequence() throws Exception
         AvroSchema schema = getStringArraySchema();
         List<String> input1 = Arrays.asList("foo", "bar",
                 "...........................................................!");
-        List<String> input2 = Arrays.asList("foobar");
+        List<String> input2 = Collections.singletonList("foobar");
         String[] input3 = new String[] { "a",
 "Something very much longer than the first entry: and with \u00DCnicod\u00E9 -- at least "
 +"2 lines full of stuff... 12235u4039680346 -346-0436 34-6 -43609 4363469 436-09",

diff --git a/avro/src/test/java/tools/jackson/dataformat/avro/AvroTestBase.java b/avro/src/test/java/tools/jackson/dataformat/avro/AvroTestBase.java
@@ -139,7 +139,7 @@ public MediaItem(MediaContent c) {
 
         public void addPhoto(Image p) {
             if (_images == null) {
-                _images = new ArrayList<Image>();
+                _images = new ArrayList<>();
             }
             _images.add(p);
         }
@@ -187,7 +187,7 @@ protected MediaContent(MediaContent src) {
 
         public void addPerson(String p) {
             if (_persons == null) {
-                _persons = new ArrayList<String>();
+                _persons = new ArrayList<>();
             }
             _persons.add(p);
         }
@@ -337,7 +337,7 @@ public static void verifyException(Throwable e, String... matches)
         String lmsg = (msg == null) ? "" : msg.toLowerCase();
         for (String match : matches) {
             String lmatch = match.toLowerCase();
-            if (lmsg.indexOf(lmatch) >= 0) {
+            if (lmsg.contains(lmatch)) {
                 return;
             }
         }

diff --git a/avro/src/test/java/tools/jackson/dataformat/avro/BinaryDataTest.java b/avro/src/test/java/tools/jackson/dataformat/avro/BinaryDataTest.java
@@ -1,8 +1,8 @@
 package tools.jackson.dataformat.avro;
 
 import java.io.ByteArrayInputStream;
-import java.io.IOException;
 import java.math.BigInteger;
+import java.nio.charset.StandardCharsets;
 
 import com.fasterxml.jackson.annotation.JsonPropertyOrder;
 
@@ -17,8 +17,8 @@ public class BinaryDataTest extends AvroTestBase
     @JsonPropertyOrder({ "filename", "data", "size" })
     static class FilePojo {
         protected FilePojo() { }
-        public FilePojo(String text) throws IOException {
-            data = text.getBytes("UTF-8");
+        public FilePojo(String text) {
+            data = text.getBytes(StandardCharsets.UTF_8);
             size = data.length;
         }
 

diff --git a/avro/src/test/java/tools/jackson/dataformat/avro/FileFormatTest.java b/avro/src/test/java/tools/jackson/dataformat/avro/FileFormatTest.java
@@ -34,9 +34,9 @@ private void _testFileFormatOutput(AvroFactory af) throws Exception
         assertNotNull(bytes);
         assertEquals(301, bytes.length);
 
-        DatumReader<GenericRecord> datumReader = new GenericDatumReader<GenericRecord>(schema.getAvroSchema());
+        DatumReader<GenericRecord> datumReader = new GenericDatumReader<>(schema.getAvroSchema());
         @SuppressWarnings("resource")
-        DataFileReader<GenericRecord> dataFileReader = new DataFileReader<GenericRecord>(new SeekableByteArrayInput(bytes),
+        DataFileReader<GenericRecord> dataFileReader = new DataFileReader<>(new SeekableByteArrayInput(bytes),
                 datumReader);
         GenericRecord output = dataFileReader.next();
 

diff --git a/avro/src/test/java/tools/jackson/dataformat/avro/Issue19Test.java b/avro/src/test/java/tools/jackson/dataformat/avro/Issue19Test.java
@@ -62,12 +62,12 @@ public EventLog(Integer version, Byte eventCount, List<Event> events, List<Probl
 
     public void testIssue19() throws Exception
     {
-        List<Event> sampleEvents = new ArrayList<Event>();
+        List<Event> sampleEvents = new ArrayList<>();
         sampleEvents.add(new Event(10, new EventID("sample1", 1, 2)));
         sampleEvents.add(new Event(20, new EventID("sample2", 10, 20)));
         sampleEvents.add(new Event(30, new EventID("sample3", 100, 200)));
 
-        List<Problem> sampleProblems = new ArrayList<Problem>();
+        List<Problem> sampleProblems = new ArrayList<>();
         sampleProblems.add(new Problem(800, 801));
         sampleProblems.add(new Problem(900, 901));
 

diff --git a/avro/src/test/java/tools/jackson/dataformat/avro/MapTest.java b/avro/src/test/java/tools/jackson/dataformat/avro/MapTest.java
@@ -31,7 +31,7 @@ public class MapTest extends AvroTestBase
             +"]}"
             ;
     static class Container {
-        public Map<String,String> stuff = new LinkedHashMap<String,String>();
+        public Map<String,String> stuff = new LinkedHashMap<>();
 
         public void setStuff(Map<String,String> arg) {
             stuff = arg;
@@ -194,7 +194,7 @@ public void testRootMapSequence() throws Exception
     }
 
     private Map<String,String> _map(String... stuff) {
-        Map<String,String> map = new LinkedHashMap<String,String>();
+        Map<String,String> map = new LinkedHashMap<>();
         for (int i = 0, end = stuff.length; i < end; i += 2) {
             map.put(stuff[i], stuff[i+1]);
         }

diff --git a/avro/src/test/java/tools/jackson/dataformat/avro/MapWithUnionTest.java b/avro/src/test/java/tools/jackson/dataformat/avro/MapWithUnionTest.java
@@ -99,9 +99,9 @@ public void testRootMapWithUnionSequence() throws Exception
 
     public void testMapContainerWithNested() throws IOException
     {
-        Map<String,Object> map = new LinkedHashMap<String,Object>();
+        Map<String,Object> map = new LinkedHashMap<>();
         map.put("hello", "world");
-        Map<String,String> otherMap = new LinkedHashMap<String,String>();
+        Map<String,String> otherMap = new LinkedHashMap<>();
         otherMap.put("foo", "bar");
         otherMap.put("zap", "bing");
         map.put("otherMap", otherMap);
@@ -125,7 +125,7 @@ public void testMapContainerWithNested() throws IOException
     }
 
     private Map<String,Object> _map(Object... args) {
-        Map<String,Object> m = new LinkedHashMap<String,Object>();
+        Map<String,Object> m = new LinkedHashMap<>();
         for (int i = 0; i < args.length; i += 2) {
             m.put((String) args[i], args[i+1]);
         }

diff --git a/avro/src/test/java/tools/jackson/dataformat/avro/RootSequenceTest.java b/avro/src/test/java/tools/jackson/dataformat/avro/RootSequenceTest.java
@@ -25,9 +25,9 @@ public void testReadWriteIntSequence() throws Exception
         // First: write a sequence of 3 root-level ints
 
         SequenceWriter sw = MAPPER.writer(schema).writeValues(b);
-        sw.write(Integer.valueOf(1));
-        sw.write(Integer.valueOf(123456));
-        sw.write(Integer.valueOf(-999));
+        sw.write(1);
+        sw.write(123456);
+        sw.write(-999);
         sw.close();
 
         byte[] bytes = b.toByteArray();

diff --git a/avro/src/test/java/tools/jackson/dataformat/avro/UUIDTest.java b/avro/src/test/java/tools/jackson/dataformat/avro/UUIDTest.java
@@ -1,5 +1,6 @@
 package tools.jackson.dataformat.avro;
 
+import java.nio.charset.StandardCharsets;
 import java.util.UUID;
 
 // @since 2.10
@@ -20,7 +21,7 @@ protected UUIDWrapper() { }
     public void testUUIDRoundtrip() throws Exception
     {
         final AvroSchema schema = MAPPER.schemaFor(UUIDWrapper.class);
-        UUIDWrapper input = new UUIDWrapper(UUID.nameUUIDFromBytes("BOGUS".getBytes("UTF-8")));
+        UUIDWrapper input = new UUIDWrapper(UUID.nameUUIDFromBytes("BOGUS".getBytes(StandardCharsets.UTF_8)));
         byte[] avro = MAPPER.writer(schema).writeValueAsBytes(input);
 
         UUIDWrapper output = MAPPER.readerFor(UUIDWrapper.class)

diff --git a/avro/src/test/java/tools/jackson/dataformat/avro/interop/annotations/AvroAliasTest.java b/avro/src/test/java/tools/jackson/dataformat/avro/interop/annotations/AvroAliasTest.java
@@ -41,19 +41,19 @@ public static class AliasedNameEmployee {
     }
 
     @AvroAlias(alias = "Size", space = "tools.jackson.dataformat.avro.AvroTestBase$")
-    public static enum NewSize {
+    public enum NewSize {
         SMALL,
         LARGE;
     }
 
     @AvroAlias(alias = "NewestSize")
-    public static enum NewerSize {
+    public enum NewerSize {
         SMALL,
         LARGE;
     }
 
     @AvroAlias(alias = "NewerSize")
-    public static enum NewestSize {
+    public enum NewestSize {
         SMALL,
         LARGE;
     }