Merge a (small) subset of #409 into 2.16 (no point merging into 2.15) (#413)

cowtowncoder · web-flow · commit 653f79207130 · 2023-11-02T17:26:23.000-07:00
diff --git a/avro/src/main/java/com/fasterxml/jackson/dataformat/avro/AvroGenerator.java b/avro/src/main/java/com/fasterxml/jackson/dataformat/avro/AvroGenerator.java
@@ -4,6 +4,7 @@
 import java.io.OutputStream;
 import java.math.BigDecimal;
 import java.math.BigInteger;
+import java.nio.charset.StandardCharsets;
 
 import org.apache.avro.io.BinaryEncoder;
 import org.apache.avro.io.EncoderFactory;
@@ -68,7 +69,7 @@ public static int collectDefaults()
             return flags;
         }
 
-        private Feature(boolean defaultState) {
+        Feature(boolean defaultState) {
             _defaultState = defaultState;
             _mask = (1 << ordinal());
         }
@@ -481,7 +482,7 @@ public void writeRawUTF8String(byte[] text, int offset, int len) throws IOExcept
 
     @Override
     public final void writeUTF8String(byte[] text, int offset, int len) throws IOException {
-        writeString(new String(text, offset, len, "UTF-8"));
+        writeString(new String(text, offset, len, StandardCharsets.UTF_8));
     }
 
     /*
@@ -568,12 +569,12 @@ public void writeNull() throws IOException {
 
     @Override
     public void writeNumber(int i) throws IOException {
-        _avroContext.writeValue(Integer.valueOf(i));
+        _avroContext.writeValue(i);
     }
 
     @Override
     public void writeNumber(long l) throws IOException {
-        _avroContext.writeValue(Long.valueOf(l));
+        _avroContext.writeValue(l);
     }
 
     @Override
@@ -588,12 +589,12 @@ public void writeNumber(BigInteger v) throws IOException
 
     @Override
     public void writeNumber(double d) throws IOException {
-        _avroContext.writeValue(Double.valueOf(d));
+        _avroContext.writeValue(d);
     }
 
     @Override
     public void writeNumber(float f) throws IOException {
-        _avroContext.writeValue(Float.valueOf(f));
+        _avroContext.writeValue(f);
     }
 
     @Override
diff --git a/avro/src/main/java/com/fasterxml/jackson/dataformat/avro/AvroParser.java b/avro/src/main/java/com/fasterxml/jackson/dataformat/avro/AvroParser.java
@@ -55,7 +55,7 @@ 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/com/fasterxml/jackson/dataformat/avro/AvroSchema.java b/avro/src/main/java/com/fasterxml/jackson/dataformat/avro/AvroSchema.java
@@ -35,7 +35,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/com/fasterxml/jackson/dataformat/avro/deser/AvroFieldDefaulters.java b/avro/src/main/java/com/fasterxml/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/com/fasterxml/jackson/dataformat/avro/deser/AvroParserImpl.java b/avro/src/main/java/com/fasterxml/jackson/dataformat/avro/deser/AvroParserImpl.java
@@ -320,7 +320,7 @@ protected final void convertNumberToInt() throws IOException
     protected final void convertNumberToLong() throws IOException
     {
         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) {
@@ -370,6 +370,7 @@ protected final void convertNumberToBigInteger() throws IOException
         _numTypesValid |= NR_BIGINT;
     }
 
+    @Override
     protected final void convertNumberToFloat() throws IOException
     {
         // Note: this MUST start with more accurate representations, since we don't know which
@@ -398,13 +399,13 @@ protected final void convertNumberToDouble() throws IOException
         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/com/fasterxml/jackson/dataformat/avro/deser/AvroReaderFactory.java b/avro/src/main/java/com/fasterxml/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<>();
 
     /*
     /**********************************************************************
@@ -306,10 +305,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/com/fasterxml/jackson/dataformat/avro/deser/JacksonAvroParserImpl.java b/avro/src/main/java/com/fasterxml/jackson/dataformat/avro/deser/JacksonAvroParserImpl.java
@@ -368,7 +368,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
@@ -427,7 +427,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/com/fasterxml/jackson/dataformat/avro/schema/AvroSchemaHelper.java b/avro/src/main/java/com/fasterxml/jackson/dataformat/avro/schema/AvroSchemaHelper.java
@@ -65,7 +65,7 @@ public abstract class AvroSchemaHelper
      *
      * @since 2.8.7
      */
-    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
@@ -121,15 +121,15 @@ protected static String getName(JavaType type) {
     protected static String getName(Class<?> cls) {
         String name = cls.getSimpleName();
         // Alas, some characters not accepted...
-        while (name.indexOf("[]") >= 0) {
+        while (name.contains("[]")) {
             name = name.replace("[]", "Array");
         }
         return name;
     }
 
     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
@@ -464,8 +464,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/com/fasterxml/jackson/dataformat/avro/schema/DefinedSchemas.java b/avro/src/main/java/com/fasterxml/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/com/fasterxml/jackson/dataformat/avro/schema/RecordVisitor.java b/avro/src/main/java/com/fasterxml/jackson/dataformat/avro/schema/RecordVisitor.java
@@ -33,7 +33,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/com/fasterxml/jackson/dataformat/avro/schema/SchemaBuilder.java b/avro/src/main/java/com/fasterxml/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/com/fasterxml/jackson/dataformat/avro/schema/VisitorFormatWrapperImpl.java b/avro/src/main/java/com/fasterxml/jackson/dataformat/avro/schema/VisitorFormatWrapperImpl.java
@@ -159,12 +159,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;
             }
         }
@@ -251,9 +246,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/com/fasterxml/jackson/dataformat/avro/ser/AvroWriteContext.java b/avro/src/main/java/com/fasterxml/jackson/dataformat/avro/ser/AvroWriteContext.java
@@ -214,7 +214,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/com/fasterxml/jackson/dataformat/avro/ser/MapWriteContext.java b/avro/src/main/java/com/fasterxml/jackson/dataformat/avro/ser/MapWriteContext.java
@@ -21,7 +21,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/com/fasterxml/jackson/dataformat/avro/ser/RootContext.java b/avro/src/main/java/com/fasterxml/jackson/dataformat/avro/ser/RootContext.java
@@ -136,7 +136,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/com/fasterxml/jackson/dataformat/avro/ArrayTest.java b/avro/src/test/java/com/fasterxml/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 com.fasterxml.jackson.core.*;
@@ -47,7 +48,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/com/fasterxml/jackson/dataformat/avro/AvroTestBase.java b/avro/src/test/java/com/fasterxml/jackson/dataformat/avro/AvroTestBase.java
@@ -137,7 +137,7 @@ public MediaItem(MediaContent c) {
 
         public void addPhoto(Image p) {
             if (_images == null) {
-                _images = new ArrayList<Image>();
+                _images = new ArrayList<>();
             }
             _images.add(p);
         }
@@ -185,7 +185,7 @@ protected MediaContent(MediaContent src) {
 
         public void addPerson(String p) {
             if (_persons == null) {
-                _persons = new ArrayList<String>();
+                _persons = new ArrayList<>();
             }
             _persons.add(p);
         }
@@ -331,7 +331,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/com/fasterxml/jackson/dataformat/avro/BinaryDataTest.java b/avro/src/test/java/com/fasterxml/jackson/dataformat/avro/BinaryDataTest.java
@@ -1,8 +1,8 @@
 package com.fasterxml.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;
 import com.fasterxml.jackson.core.JsonParser;
@@ -16,8 +16,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/com/fasterxml/jackson/dataformat/avro/FileFormatTest.java b/avro/src/test/java/com/fasterxml/jackson/dataformat/avro/FileFormatTest.java
@@ -38,9 +38,9 @@ private void _testFileFormatOutput(boolean useApacheImpl) 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/com/fasterxml/jackson/dataformat/avro/Issue19Test.java b/avro/src/test/java/com/fasterxml/jackson/dataformat/avro/Issue19Test.java
@@ -63,12 +63,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/com/fasterxml/jackson/dataformat/avro/MapTest.java b/avro/src/test/java/com/fasterxml/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;
@@ -197,7 +197,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/com/fasterxml/jackson/dataformat/avro/MapWithUnionTest.java b/avro/src/test/java/com/fasterxml/jackson/dataformat/avro/MapWithUnionTest.java
diff --git a/avro/src/test/java/com/fasterxml/jackson/dataformat/avro/MapperConfigTest.java b/avro/src/test/java/com/fasterxml/jackson/dataformat/avro/MapperConfigTest.java
diff --git a/avro/src/test/java/com/fasterxml/jackson/dataformat/avro/RootSequenceTest.java b/avro/src/test/java/com/fasterxml/jackson/dataformat/avro/RootSequenceTest.java
diff --git a/avro/src/test/java/com/fasterxml/jackson/dataformat/avro/UUIDTest.java b/avro/src/test/java/com/fasterxml/jackson/dataformat/avro/UUIDTest.java
diff --git a/avro/src/test/java/com/fasterxml/jackson/dataformat/avro/interop/annotations/AvroAliasTest.java b/avro/src/test/java/com/fasterxml/jackson/dataformat/avro/interop/annotations/AvroAliasTest.java
diff --git a/avro/src/test/java/com/fasterxml/jackson/dataformat/avro/interop/annotations/AvroEncodeTest.java b/avro/src/test/java/com/fasterxml/jackson/dataformat/avro/interop/annotations/AvroEncodeTest.java

Original file line number	Diff line number	Diff line change
`@@ -55,7 +55,7 @@ public static int collectDefaults()`
`55`	`55`	`return flags;`
`56`	`56`	`}`
`57`	`57`
`58`		`- private Feature(boolean defaultState) {`
	`58`	`+ Feature(boolean defaultState) {`
`59`	`59`	`_defaultState = defaultState;`
`60`	`60`	`_mask = (1 << ordinal());`
`61`	`61`	`}`
Original file line number	Diff line number	Diff line change
`@@ -35,7 +35,7 @@ public class AvroSchema implements FormatSchema`
`35`	`35`	`/**`
`36`	`36`	`* Lazily instantiated value reader for this schema.`
`37`	`37`	`*/`
`38`		`- protected final AtomicReference<AvroStructureReader> _reader = new AtomicReference<AvroStructureReader>();`
	`38`	`+ protected final AtomicReference<AvroStructureReader> _reader = new AtomicReference<>();`
`39`	`39`
`40`	`40`	`public AvroSchema(Schema asch)`
`41`	`41`	`{`
Original file line number	Diff line number	Diff line change
`@@ -42,7 +42,7 @@ public static AvroFieldReader createDefaulter(String name,`
`42`	`42`	`case START_OBJECT:`
`43`	`43`	`{`
`44`	`44`	`Iterator<Map.Entry<String,JsonNode>> it = defaultAsNode.fields();`
`45`		`- List<AvroFieldReader> readers = new ArrayList<AvroFieldReader>();`
	`45`	`+ List<AvroFieldReader> readers = new ArrayList<>();`
`46`	`46`	`while (it.hasNext()) {`
`47`	`47`	`Map.Entry<String,JsonNode> entry = it.next();`
`48`	`48`	`String propName = entry.getKey();`
`@@ -52,7 +52,7 @@ public static AvroFieldReader createDefaulter(String name,`
`52`	`52`	`}`
`53`	`53`	`case START_ARRAY:`
`54`	`54`	`{`
`55`		`- List<AvroFieldReader> readers = new ArrayList<AvroFieldReader>();`
	`55`	`+ List<AvroFieldReader> readers = new ArrayList<>();`
`56`	`56`	`for (JsonNode value : defaultAsNode) {`
`57`	`57`	`readers.add(createDefaulter("", value));`
`58`	`58`	`}`
Original file line number	Diff line number	Diff line change
`@@ -368,7 +368,7 @@ public long decodeLong() throws IOException {`
`368`	`368`	`_inputPtr = ptr;`
`369`	`369`	`// should be ok to zigzag as int, then sign-extend`
`370`	`370`	`i = (i >>> 1) ^ (-(i & 1));`
`371`		`- return (long) i;`
	`371`	`+ return i;`
`372`	`372`	`}`
`373`	`373`
`374`	`374`	`private long _decodeLong2(int ptr, long lo) throws IOException`
`@@ -427,7 +427,7 @@ public long _decodeLongSlow() throws IOException {`
`427`	`427`	`}`
`428`	`428`	`}`
`429`	`429`	`i = (i >>> 1) ^ (-(i & 1));`
`430`		`- return (long) i;`
	`430`	`+ return i;`
`431`	`431`	`}`
`432`	`432`
`433`	`433`	`private long _decodeLongSlow2(long lo) throws IOException`
Original file line number	Diff line number	Diff line change
`@@ -65,7 +65,7 @@ public abstract class AvroSchemaHelper`
`65`	`65`	`*`
`66`	`66`	`* @since 2.8.7`
`67`	`67`	`*/`
`68`		`- protected static final Set<Class<?>> STRINGABLE_CLASSES = new HashSet<Class<?>>(Arrays.asList(`
	`68`	`+ protected static final Set<Class<?>> STRINGABLE_CLASSES = new HashSet<>(Arrays.asList(`
`69`	`69`	`URI.class, URL.class, File.class,`
`70`	`70`	`BigInteger.class, BigDecimal.class,`
`71`	`71`	`String.class`
`@@ -121,15 +121,15 @@ protected static String getName(JavaType type) {`
`121`	`121`	`protected static String getName(Class<?> cls) {`
`122`	`122`	`String name = cls.getSimpleName();`
`123`	`123`	`// Alas, some characters not accepted...`
`124`		`- while (name.indexOf("[]") >= 0) {`
	`124`	`+ while (name.contains("[]")) {`
`125`	`125`	`name = name.replace("[]", "Array");`
`126`	`126`	`}`
`127`	`127`	`return name;`
`128`	`128`	`}`
`129`	`129`
`130`	`130`	`protected static Schema unionWithNull(Schema otherSchema)`
`131`	`131`	`{`
`132`		`- List<Schema> schemas = new ArrayList<Schema>();`
	`132`	`+ List<Schema> schemas = new ArrayList<>();`
`133`	`133`	`schemas.add(Schema.create(Schema.Type.NULL));`
`134`	`134`
`135`	`135`	`// two cases: existing union`
`@@ -464,8 +464,7 @@ public FullNameKey(String namespace, String name) {`
`464`	`464`	`}`
`465`	`465`
`466`	`466`	`public String nameWithSeparator(char sep) {`
`467`		`- final StringBuilder sb = new StringBuilder(1 + _namespace.length() + _name.length());`
`468`		`- return sb.append(_namespace).append(sep).append(_name).toString();`
	`467`	`+ return _namespace + sep + _name;`
`469`	`468`	`}`
`470`	`469`
`471`	`470`	`@Override`
Original file line number	Diff line number	Diff line change
`@@ -13,7 +13,7 @@`
`13`	`13`	`*/`
`14`	`14`	`public class DefinedSchemas`
`15`	`15`	`{`
`16`		`- protected final Map<JavaType, Schema> _schemas = new LinkedHashMap<JavaType, Schema>();`
	`16`	`+ protected final Map<JavaType, Schema> _schemas = new LinkedHashMap<>();`
`17`	`17`
`18`	`18`	`protected SerializerProvider _provider;`
`19`	`19`
Original file line number	Diff line number	Diff line change
`@@ -33,7 +33,7 @@ public class RecordVisitor`
`33`	`33`
`34`	`34`	`protected Schema _avroSchema;`
`35`	`35`
`36`		`- protected List<Schema.Field> _fields = new ArrayList<Schema.Field>();`
	`36`	`+ protected List<Schema.Field> _fields = new ArrayList<>();`
`37`	`37`
`38`	`38`	`public RecordVisitor(SerializerProvider p, JavaType type, VisitorFormatWrapperImpl visitorWrapper)`
`39`	`39`	`{`