Update release notes wrt #3445; improve TypeFactory Javadocs /cc @kostislav

Author: cowtowncoder

release-notes/CREDITS-2.x

@@ -1445,3 +1445,7 @@ Gary Morgan (morganga@github)
   * Contributed #3419: Improve performance of `UnresolvedForwardReference` for
     forward reference resolution
   (2.14.0)
+
+Jan Judas (kostislav@github)
+  * Contributed #3445: Do not strip generic type from `Class<C>` when resolving `JavaType`
+  (2.14.0)

release-notes/VERSION-2.x

@@ -14,6 +14,8 @@ Project: jackson-databind
 (contributed by Gary M)
 #3421: Implement `JsonNodeFeature.READ_NULL_PROPERTIES` to allow skipping of
   JSON `null` values on reading
+#3445: Do not strip generic type from `Class<C>` when resolving `JavaType`
+ (contributed by Jan J)
 
 2.13.3 (not yet released)
 

src/main/java/com/fasterxml/jackson/databind/type/TypeFactory.java

@@ -31,6 +31,33 @@
  *<pre>
  *   JavaType stringCollection = mapper.getTypeFactory().constructCollectionType(List.class, String.class);
  *</pre>
+ *<p>
+ * Note on optimizations: generic type parameters are resolved for all types, with following
+ * exceptions:
+ *<ul>
+ *  <li>For optimization purposes, type resolution is skipped for following commonly seen
+ * types that do have type parameters, but ones that are rarely needed:
+ *     <ul>
+ *       <li>{@link java.lang.Enum}: Self-referential type reference is simply dropped and
+ *       Class is exposed as a simple, non-parameterized {@link SimpleType}
+ *        </li>
+ *       <li>{@link java.lang.Comparable}: Type parameter is simply dropped and and
+ *       interface is exposed as a simple, non-parameterized {@link SimpleType}
+ *        </li>
+ *       <li>Up until Jackson 2.13, {@link java.lang.Class} type parameter was dropped; resolution
+ *         was added back in Jackson 2.14.
+ *        </li>
+ *     </ul>
+ *   </li>
+ *  <li>For {@link java.util.Collection} subtypes, resolved type is ALWAYS the parameter for
+ *     {link java.util.Collection} and not that of actually resolved subtype.
+ *     This is usually (but not always) same parameter.
+ *   </li>
+ *  <li>For {@link java.util.Map} subtypes, resolved type is ALWAYS the parameter for
+ *     {link java.util.Map} and not that of actually resolved subtype.
+ *     These are usually (but not always) same parameters.
+ *   </li>
+ *</ul>
  */
 @SuppressWarnings({"rawtypes" })
 public class TypeFactory // note: was final in 2.9, removed from 2.10
@@ -63,7 +90,6 @@ public class TypeFactory // note: was final in 2.9, removed from 2.10
     private final static Class<?> CLS_OBJECT = Object.class;
 
     private final static Class<?> CLS_COMPARABLE = Comparable.class;
-    private final static Class<?> CLS_CLASS = Class.class;
     private final static Class<?> CLS_ENUM = Enum.class;
     private final static Class<?> CLS_JSON_NODE = JsonNode.class; // since 2.10