Merge branch '2.19' into 2.x

Author: cowtowncoder

src/main/java/com/fasterxml/jackson/databind/deser/std/EnumSetDeserializer.java

@@ -139,12 +139,9 @@ public EnumSetDeserializer withResolved(JsonDeserializer<?> deser,
     @Override
     public boolean isCachable() {
         // One caveat: content deserializer should prevent caching
-        if (_enumType.getValueHandler() != null) {
-            return false;
-        }
-        return true;
+        return _enumType.getValueHandler() == null;
     }
-    
+
     @Override // since 2.12
     public LogicalType logicalType() {
         return LogicalType.Collection;

src/main/java/com/fasterxml/jackson/databind/deser/std/ObjectArrayDeserializer.java

@@ -201,7 +201,6 @@ public Object deserialize(JsonParser p, DeserializationContext ctxt)
         Object[] chunk = buffer.resetAndStart();
         int ix = 0;
         JsonToken t;
-        final TypeDeserializer typeDeser = _elementTypeDeserializer;
 
         try {
             while ((t = p.nextToken()) != JsonToken.END_ARRAY) {
@@ -213,10 +212,8 @@ public Object deserialize(JsonParser p, DeserializationContext ctxt)
                         continue;
                     }
                     value = _nullProvider.getNullValue(ctxt);
-                } else if (typeDeser == null) {
-                    value = _elementDeserializer.deserialize(p, ctxt);
                 } else {
-                    value = _elementDeserializer.deserializeWithType(p, ctxt, typeDeser);
+                    value = _deserializeNoNullChecks(p, ctxt);
                 }
                 if (ix >= chunk.length) {
                     chunk = buffer.appendCompletedChunk(chunk);
@@ -269,7 +266,6 @@ public Object deserialize(JsonParser p, DeserializationContext ctxt,
         int ix = intoValue.length;
         Object[] chunk = buffer.resetAndStart(intoValue, ix);
         JsonToken t;
-        final TypeDeserializer typeDeser = _elementTypeDeserializer;
 
         try {
             while ((t = p.nextToken()) != JsonToken.END_ARRAY) {
@@ -280,10 +276,8 @@ public Object deserialize(JsonParser p, DeserializationContext ctxt,
                         continue;
                     }
                     value = _nullProvider.getNullValue(ctxt);
-                } else if (typeDeser == null) {
-                    value = _elementDeserializer.deserialize(p, ctxt);
                 } else {
-                    value = _elementDeserializer.deserializeWithType(p, ctxt, typeDeser);
+                    value = _deserializeNoNullChecks(p, ctxt);
                 }
                 if (ix >= chunk.length) {
                     chunk = buffer.appendCompletedChunk(chunk);
@@ -320,7 +314,7 @@ protected Byte[] deserializeFromBase64(JsonParser p, DeserializationContext ctxt
         // But then need to convert to wrappers
         Byte[] result = new Byte[b.length];
         for (int i = 0, len = b.length; i < len; ++i) {
-            result[i] = Byte.valueOf(b[i]);
+            result[i] = b[i];
         }
         return result;
     }
@@ -375,11 +369,7 @@ protected Object handleNonArray(JsonParser p, DeserializationContext ctxt)
                 // if coercion failed, we can still add it to a list
             }
 
-            if (_elementTypeDeserializer == null) {
-                value = _elementDeserializer.deserialize(p, ctxt);
-            } else {
-                value = _elementDeserializer.deserializeWithType(p, ctxt, _elementTypeDeserializer);
-            }
+            value = _deserializeNoNullChecks(p, ctxt);
         }
         // Ok: bit tricky, since we may want T[], not just Object[]
         Object[] result;
@@ -392,5 +382,21 @@ protected Object handleNonArray(JsonParser p, DeserializationContext ctxt)
         result[0] = value;
         return result;
     }
+
+    /**
+     * Deserialize the content of the map.
+     * If _elementTypeDeserializer is null, use _elementDeserializer.deserialize; if non-null,
+     * use _elementDeserializer.deserializeWithType to deserialize value.
+     * This method only performs deserialization and does not consider _skipNullValues, _nullProvider, etc.
+     * @since 2.19.2
+     */
+    protected Object _deserializeNoNullChecks(JsonParser p, DeserializationContext ctxt)
+            throws IOException
+    {
+        if (_elementTypeDeserializer == null) {
+            return _elementDeserializer.deserialize(p, ctxt);
+        }
+        return _elementDeserializer.deserializeWithType(p, ctxt, _elementTypeDeserializer);
+    }
 }
 

src/main/java/com/fasterxml/jackson/databind/deser/std/StringArrayDeserializer.java

@@ -40,14 +40,14 @@ public final class StringArrayDeserializer
     /**
      * Value serializer to use, if not the standard one (which is inlined)
      */
-    protected JsonDeserializer<String> _elementDeserializer;
+    private final JsonDeserializer<String> _elementDeserializer;
 
     /**
      * Handler we need for dealing with null values as elements
      *
      * @since 2.9
      */
-    protected final NullValueProvider _nullProvider;
+    private final NullValueProvider _nullProvider;
 
     /**
      * Specific override for this instance (from proper, or global per-type overrides)
@@ -56,22 +56,22 @@ public final class StringArrayDeserializer
      *
      * @since 2.7
      */
-    protected final Boolean _unwrapSingle;
+    private final Boolean _unwrapSingle;
 
     /**
      * Marker flag set if the <code>_nullProvider</code> indicates that all null
      * content values should be skipped (instead of being possibly converted).
      *
      * @since 2.9
      */
-    protected final boolean _skipNullValues;
+    private final boolean _skipNullValues;
 
     public StringArrayDeserializer() {
         this(null, null, null);
     }
 
     @SuppressWarnings("unchecked")
-    protected StringArrayDeserializer(JsonDeserializer<?> deser,
+    private StringArrayDeserializer(JsonDeserializer<?> deser,
             NullValueProvider nuller, Boolean unwrapSingle) {
         super(String[].class);
         _elementDeserializer = (JsonDeserializer<String>) deser;
@@ -184,7 +184,7 @@ public String[] deserialize(JsonParser p, DeserializationContext ctxt) throws IO
     /**
      * Offlined version used when we do not use the default deserialization method.
      */
-    protected final String[] _deserializeCustom(JsonParser p, DeserializationContext ctxt,
+    private String[] _deserializeCustom(JsonParser p, DeserializationContext ctxt,
             String[] old) throws IOException
     {
         final ObjectBuffer buffer = ctxt.leaseObjectBuffer();

src/main/java/com/fasterxml/jackson/databind/deser/std/StringCollectionDeserializer.java

@@ -37,20 +37,20 @@ public final class StringCollectionDeserializer
      * Value deserializer to use, if NOT the standard one
      * (if it is, will be null).
      */
-    protected final JsonDeserializer<String> _valueDeserializer;
+    private final JsonDeserializer<String> _valueDeserializer;
 
     // // Instance construction settings:
 
     /**
      * Instantiator used in case custom handling is needed for creation.
      */
-    protected final ValueInstantiator _valueInstantiator;
+    private final ValueInstantiator _valueInstantiator;
 
     /**
      * Deserializer that is used iff delegate-based creator is
      * to be used for deserializing from JSON Object.
      */
-    protected final JsonDeserializer<Object> _delegateDeserializer;
+    private final JsonDeserializer<Object> _delegateDeserializer;
 
     // NOTE: no PropertyBasedCreator, as JSON Arrays have no properties
 
@@ -67,7 +67,7 @@ public StringCollectionDeserializer(JavaType collectionType,
     }
 
     @SuppressWarnings("unchecked")
-    protected StringCollectionDeserializer(JavaType collectionType,
+    private StringCollectionDeserializer(JavaType collectionType,
             ValueInstantiator valueInstantiator, JsonDeserializer<?> delegateDeser,
             JsonDeserializer<?> valueDeser,
             NullValueProvider nuller, Boolean unwrapSingle)
@@ -78,7 +78,7 @@ protected StringCollectionDeserializer(JavaType collectionType,
         _delegateDeserializer = (JsonDeserializer<Object>) delegateDeser;
     }
 
-    protected StringCollectionDeserializer withResolved(JsonDeserializer<?> delegateDeser,
+    private StringCollectionDeserializer withResolved(JsonDeserializer<?> delegateDeser,
             JsonDeserializer<?> valueDeser,
             NullValueProvider nuller, Boolean unwrapSingle)
     {
@@ -107,6 +107,7 @@ public LogicalType logicalType() {
     /* Validation, post-processing
     /**********************************************************
      */
+
     @Override
     public JsonDeserializer<?> createContextual(DeserializationContext ctxt,
             BeanProperty property) throws JsonMappingException