Add explicit return values to Builder methods

Author: TWiStErRob

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

@@ -129,32 +129,33 @@ class KotlinModule @Deprecated(level = DeprecationLevel.WARNING, message = "Use
             KotlinFeature.values().filter { it.enabledByDefault }.forEach { or(it.bitSet) }
         }
 
-        fun withReflectionCacheSize(reflectionCacheSize: Int) = apply {
+        fun withReflectionCacheSize(reflectionCacheSize: Int): Builder = apply {
             this.reflectionCacheSize = reflectionCacheSize
         }
 
-        fun enable(feature: KotlinFeature) = apply {
+        fun enable(feature: KotlinFeature): Builder = apply {
             bitSet.or(feature.bitSet)
         }
 
-        fun disable(feature: KotlinFeature) = apply {
+        fun disable(feature: KotlinFeature): Builder = apply {
             bitSet.andNot(feature.bitSet)
         }
 
-        fun configure(feature: KotlinFeature, enabled: Boolean) = when {
-            enabled -> enable(feature)
-            else -> disable(feature)
-        }
+        fun configure(feature: KotlinFeature, enabled: Boolean): Builder =
+            when {
+                enabled -> enable(feature)
+                else -> disable(feature)
+            }
 
-        fun isEnabled(feature: KotlinFeature): Boolean = bitSet.intersects(feature.bitSet)
+        fun isEnabled(feature: KotlinFeature): Boolean =
+            bitSet.intersects(feature.bitSet)
 
         @Deprecated(
             message = "Deprecated, use withReflectionCacheSize(reflectionCacheSize) instead.",
             replaceWith = ReplaceWith("withReflectionCacheSize(reflectionCacheSize)")
         )
-        fun reflectionCacheSize(reflectionCacheSize: Int) = apply {
-            this.reflectionCacheSize = reflectionCacheSize
-        }
+        fun reflectionCacheSize(reflectionCacheSize: Int): Builder =
+            withReflectionCacheSize(reflectionCacheSize)
 
         @Deprecated(
             message = "Deprecated, use isEnabled(NullToEmptyCollection) instead.",
@@ -163,7 +164,8 @@ class KotlinModule @Deprecated(level = DeprecationLevel.WARNING, message = "Use
                 "com.fasterxml.jackson.module.kotlin.KotlinFeature"
             )
         )
-        fun getNullToEmptyCollection() = isEnabled(NullToEmptyCollection)
+        fun getNullToEmptyCollection(): Boolean =
+            isEnabled(NullToEmptyCollection)
 
         @Deprecated(
             message = "Deprecated, use configure(NullToEmptyCollection, enabled) instead.",
@@ -172,7 +174,7 @@ class KotlinModule @Deprecated(level = DeprecationLevel.WARNING, message = "Use
                 "com.fasterxml.jackson.module.kotlin.KotlinFeature"
             )
         )
-        fun nullToEmptyCollection(nullToEmptyCollection: Boolean) =
+        fun nullToEmptyCollection(nullToEmptyCollection: Boolean): Builder =
             configure(NullToEmptyCollection, nullToEmptyCollection)
 
         @Deprecated(
@@ -182,7 +184,8 @@ class KotlinModule @Deprecated(level = DeprecationLevel.WARNING, message = "Use
                 "com.fasterxml.jackson.module.kotlin.KotlinFeature"
             )
         )
-        fun getNullToEmptyMap() = isEnabled(NullToEmptyMap)
+        fun getNullToEmptyMap(): Boolean =
+            isEnabled(NullToEmptyMap)
 
         @Deprecated(
             message = "Deprecated, use configure(NullToEmptyMap, enabled) instead.",
@@ -191,7 +194,8 @@ class KotlinModule @Deprecated(level = DeprecationLevel.WARNING, message = "Use
                 "com.fasterxml.jackson.module.kotlin.KotlinFeature"
             )
         )
-        fun nullToEmptyMap(nullToEmptyMap: Boolean) = configure(NullToEmptyMap, nullToEmptyMap)
+        fun nullToEmptyMap(nullToEmptyMap: Boolean): Builder =
+            configure(NullToEmptyMap, nullToEmptyMap)
 
         @Deprecated(
             message = "Deprecated, use isEnabled(NullIsSameAsDefault) instead.",
@@ -200,7 +204,8 @@ class KotlinModule @Deprecated(level = DeprecationLevel.WARNING, message = "Use
                 "com.fasterxml.jackson.module.kotlin.KotlinFeature"
             )
         )
-        fun getNullIsSameAsDefault() = isEnabled(NullIsSameAsDefault)
+        fun getNullIsSameAsDefault(): Boolean =
+            isEnabled(NullIsSameAsDefault)
 
         @Deprecated(
             message = "Deprecated, use configure(NullIsSameAsDefault, enabled) instead.",
@@ -209,7 +214,8 @@ class KotlinModule @Deprecated(level = DeprecationLevel.WARNING, message = "Use
                 "com.fasterxml.jackson.module.kotlin.KotlinFeature"
             )
         )
-        fun nullIsSameAsDefault(nullIsSameAsDefault: Boolean) = configure(NullIsSameAsDefault, nullIsSameAsDefault)
+        fun nullIsSameAsDefault(nullIsSameAsDefault: Boolean): Builder =
+            configure(NullIsSameAsDefault, nullIsSameAsDefault)
 
         @Deprecated(
             message = "Deprecated, use isEnabled(SingletonSupport) instead.",
@@ -218,10 +224,11 @@ class KotlinModule @Deprecated(level = DeprecationLevel.WARNING, message = "Use
                 "com.fasterxml.jackson.module.kotlin.KotlinFeature"
             )
         )
-        fun getSingletonSupport() = when {
-            isEnabled(KotlinFeature.SingletonSupport) -> CANONICALIZE
-            else -> DISABLED
-        }
+        fun getSingletonSupport(): SingletonSupport =
+            when {
+                isEnabled(KotlinFeature.SingletonSupport) -> CANONICALIZE
+                else -> DISABLED
+            }
 
         @Deprecated(
             message = "Deprecated, use configure(SingletonSupport, enabled) instead.",
@@ -230,10 +237,11 @@ class KotlinModule @Deprecated(level = DeprecationLevel.WARNING, message = "Use
                 "com.fasterxml.jackson.module.kotlin.KotlinFeature"
             )
         )
-        fun singletonSupport(singletonSupport: SingletonSupport) = when (singletonSupport) {
-            CANONICALIZE -> enable(KotlinFeature.SingletonSupport)
-            else -> disable(KotlinFeature.SingletonSupport)
-        }
+        fun singletonSupport(singletonSupport: SingletonSupport): Builder =
+            when (singletonSupport) {
+                CANONICALIZE -> enable(KotlinFeature.SingletonSupport)
+                else -> disable(KotlinFeature.SingletonSupport)
+            }
 
         @Deprecated(
             message = "Deprecated, use isEnabled(StrictNullChecks) instead.",
@@ -242,7 +250,8 @@ class KotlinModule @Deprecated(level = DeprecationLevel.WARNING, message = "Use
                 "com.fasterxml.jackson.module.kotlin.KotlinFeature"
             )
         )
-        fun getStrictNullChecks() = isEnabled(StrictNullChecks)
+        fun getStrictNullChecks(): Boolean =
+            isEnabled(StrictNullChecks)
 
         @Deprecated(
             message = "Deprecated, use configure(StrictNullChecks, enabled) instead.",
@@ -251,8 +260,10 @@ class KotlinModule @Deprecated(level = DeprecationLevel.WARNING, message = "Use
                 "com.fasterxml.jackson.module.kotlin.KotlinFeature"
             )
         )
-        fun strictNullChecks(strictNullChecks: Boolean) = configure(StrictNullChecks, strictNullChecks)
+        fun strictNullChecks(strictNullChecks: Boolean): Builder =
+            configure(StrictNullChecks, strictNullChecks)
 
-        fun build() = KotlinModule(this)
+        fun build(): KotlinModule =
+            KotlinModule(this)
     }
 }