Remove unnecessary property and check

from: FasterXML/jackson-module-kotlin#542

Author: k163377

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

@@ -14,7 +14,6 @@ import com.fasterxml.jackson.module.kotlin.deser.value_instantiator.KotlinInstan
 import com.fasterxml.jackson.module.kotlin.ser.serializers.KotlinKeySerializers
 import com.fasterxml.jackson.module.kotlin.ser.serializers.KotlinSerializers
 import java.util.*
-import kotlin.reflect.KClass
 
 /**
  * @param reflectionCacheSize     Default: 512.  Size, in items, of the caches used for mapping objects.
@@ -52,8 +51,6 @@ public class KotlinModule private constructor(
         private const val serialVersionUID = 1L
     }
 
-    private val ignoredClassesForImplyingJsonCreator = emptySet<KClass<*>>()
-
     override fun setupModule(context: SetupContext) {
         super.setupModule(context)
 
@@ -70,7 +67,7 @@ public class KotlinModule private constructor(
         }
 
         context.insertAnnotationIntrospector(KotlinAnnotationIntrospector(context, cache, nullToEmptyCollection, nullToEmptyMap, nullIsSameAsDefault))
-        context.appendAnnotationIntrospector(KotlinNamesAnnotationIntrospector(this, cache, ignoredClassesForImplyingJsonCreator))
+        context.appendAnnotationIntrospector(KotlinNamesAnnotationIntrospector(this, cache))
 
         context.addDeserializers(KotlinDeserializers())
         context.addKeyDeserializers(KotlinKeyDeserializers)

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

@@ -24,7 +24,7 @@ import kotlin.reflect.full.memberProperties
 import kotlin.reflect.full.primaryConstructor
 import kotlin.reflect.jvm.javaType
 
-internal class KotlinNamesAnnotationIntrospector(val module: KotlinModule, val cache: ReflectionCache, val ignoredClassesForImplyingJsonCreator: Set<KClass<*>>) : NopAnnotationIntrospector() {
+internal class KotlinNamesAnnotationIntrospector(val module: KotlinModule, val cache: ReflectionCache) : NopAnnotationIntrospector() {
     // since 2.4
     override fun findImplicitPropertyName(member: AnnotatedMember): String? = when (member) {
         is AnnotatedMethod -> member.annotated.declaringClass.toKmClass()?.let { kmClass ->
@@ -71,7 +71,6 @@ internal class KotlinNamesAnnotationIntrospector(val module: KotlinModule, val c
         // don't add a JsonCreator to any constructor if one is declared already
 
         val kClass = cache.kotlinFromJava(member.declaringClass as Class<Any>)
-            .apply { if (this in ignoredClassesForImplyingJsonCreator) return false }
         val kConstructor = cache.kotlinFromJava(member.annotated as Constructor<Any>) ?: return false
 
         // TODO:  should we do this check or not?  It could cause failures if we miss another way a property could be set