Merge pull request #969 from k163377/organize-deprecateds

Cleanup of deprecated contents

Author: k163377

pom.xml

@@ -245,6 +245,11 @@
                         <breakBuildOnBinaryIncompatibleModifications>true</breakBuildOnBinaryIncompatibleModifications>
                         <breakBuildOnSourceIncompatibleModifications>true</breakBuildOnSourceIncompatibleModifications>
                         <excludes>
+                            <!-- public -->
+                            <!-- removed -->
+                            <exclude>
+                                com.fasterxml.jackson.module.kotlin.MissingKotlinParameterException#MissingKotlinParameterException(kotlin.reflect.KParameter,java.io.Closeable,java.lang.String)
+                            </exclude>
                         </excludes>
                     </parameter>
                 </configuration>

release-notes/CREDITS-2.x

@@ -18,6 +18,7 @@ Contributors:
 # 2.20.0 (not yet released)
 
 WrongWrong (@k163377)
+* #969: Cleanup of deprecated contents
 * #967: Update settings for 2.20
 
 # 2.19.1 (not yet released)

release-notes/VERSION-2.x

@@ -18,6 +18,7 @@ Co-maintainers:
 
 2.20.0 (not yet released)
 
+#969: Deprecated content has been cleaned up with the version upgrade.
 #967: Kotlin has been upgraded to 2.0.21.
 
 2.19.1 (not yet released)

src/main/kotlin/com/fasterxml/jackson/module/kotlin/Exceptions.kt

@@ -3,7 +3,6 @@ package com.fasterxml.jackson.module.kotlin
 import com.fasterxml.jackson.core.JsonParser
 import com.fasterxml.jackson.databind.JsonMappingException
 import com.fasterxml.jackson.databind.exc.InvalidNullException
-import java.io.Closeable
 import kotlin.reflect.KParameter
 
 /**
@@ -31,15 +30,4 @@ class MissingKotlinParameterException(
     val parameter: KParameter,
     processor: JsonParser? = null,
     msg: String
-) : InvalidNullException(processor, msg, null) {
-    @Deprecated(
-        "Use main constructor, ",
-        ReplaceWith("MissingKotlinParameterException(KParameter, JsonParser?, String)"),
-        DeprecationLevel.ERROR,
-    )
-    constructor(
-            parameter: KParameter,
-            processor: Closeable? = null,
-            msg: String
-    ) : this(parameter, processor as JsonParser, msg)
-}
+) : InvalidNullException(processor, msg, null)

src/main/kotlin/com/fasterxml/jackson/module/kotlin/KotlinFeature.kt

@@ -43,7 +43,7 @@ enum class KotlinFeature(internal val enabledByDefault: Boolean) {
      * Enabling it protects against this but has significant performance impact.
      */
     @Deprecated(
-        level = DeprecationLevel.WARNING,
+        level = DeprecationLevel.ERROR,
         message = "This option will be migrated to the new backend in 2.21.",
         replaceWith = ReplaceWith("NewStrictNullChecks")
     )

src/main/kotlin/com/fasterxml/jackson/module/kotlin/KotlinModule.kt

@@ -43,6 +43,7 @@ class KotlinModule private constructor(
     val nullToEmptyMap: Boolean = NullToEmptyMap.enabledByDefault,
     val nullIsSameAsDefault: Boolean = NullIsSameAsDefault.enabledByDefault,
     val singletonSupport: Boolean = SingletonSupport.enabledByDefault,
+    @Suppress("DEPRECATION_ERROR")
     strictNullChecks: Boolean = StrictNullChecks.enabledByDefault,
     val kotlinPropertyNameAsImplicitName: Boolean = KotlinPropertyNameAsImplicitName.enabledByDefault,
     val useJavaDurationConversion: Boolean = UseJavaDurationConversion.enabledByDefault,
@@ -59,7 +60,7 @@ class KotlinModule private constructor(
      * Now that 2 is complete, deprecation is in progress for 3.
      */
     @Deprecated(
-        level = DeprecationLevel.WARNING,
+        level = DeprecationLevel.ERROR,
         message = "This property is scheduled to be removed in 2.21 or later" +
                 " in order to unify the use of KotlinFeature.",
         replaceWith = ReplaceWith("singletonSupport")
@@ -96,6 +97,7 @@ class KotlinModule private constructor(
         builder.isEnabled(NullToEmptyMap),
         builder.isEnabled(NullIsSameAsDefault),
         builder.isEnabled(SingletonSupport),
+        @Suppress("DEPRECATION_ERROR")
         builder.isEnabled(StrictNullChecks),
         builder.isEnabled(KotlinPropertyNameAsImplicitName),
         builder.isEnabled(UseJavaDurationConversion),