Merge pull request #37 from k163377/feature

Fine-grained optimization.

Author: k163377

build.gradle.kts

@@ -6,7 +6,7 @@ plugins {
 }
 
 group = "com.mapk"
-version = "0.29"
+version = "0.30"
 
 java {
     sourceCompatibility = JavaVersion.VERSION_1_8
@@ -30,7 +30,7 @@ repositories {
 dependencies {
     implementation("org.jetbrains.kotlin:kotlin-stdlib-jdk8")
     implementation(kotlin("reflect"))
-    api("com.github.ProjectMapK:Shared:0.15")
+    api("com.github.ProjectMapK:Shared:0.16")
 
     // https://mvnrepository.com/artifact/org.junit.jupiter/junit-jupiter
     testImplementation(group = "org.junit.jupiter", name = "junit-jupiter", version = "5.6.2") {

src/main/kotlin/com/mapk/kmapper/BoundKMapper.kt

@@ -16,13 +16,13 @@ import kotlin.reflect.jvm.jvmName
 class BoundKMapper<S : Any, D : Any> private constructor(
     private val function: KFunctionForCall<D>,
     src: KClass<S>,
-    parameterNameConverter: (String) -> String
+    parameterNameConverter: ((String) -> String)?
 ) {
-    constructor(function: KFunction<D>, src: KClass<S>, parameterNameConverter: (String) -> String = { it }) : this(
+    constructor(function: KFunction<D>, src: KClass<S>, parameterNameConverter: ((String) -> String)? = null) : this(
         KFunctionForCall(function, parameterNameConverter), src, parameterNameConverter
     )
 
-    constructor(clazz: KClass<D>, src: KClass<S>, parameterNameConverter: (String) -> String = { it }) : this(
+    constructor(clazz: KClass<D>, src: KClass<S>, parameterNameConverter: ((String) -> String)? = null) : this(
         clazz.toKConstructor(parameterNameConverter), src, parameterNameConverter
     )
 

src/main/kotlin/com/mapk/kmapper/BoundParameterForMap.kt

@@ -68,7 +68,7 @@ internal sealed class BoundParameterForMap<S> {
         fun <S : Any> newInstance(
             param: ValueParameter<*>,
             property: KProperty1<S, *>,
-            parameterNameConverter: (String) -> String
+            parameterNameConverter: ((String) -> String)?
         ): BoundParameterForMap<S> {
             // ゲッターが無いならエラー
             val propertyGetter = property.javaGetter

src/main/kotlin/com/mapk/kmapper/KMapper.kt

@@ -16,13 +16,13 @@ import kotlin.reflect.jvm.javaGetter
 
 class KMapper<T : Any> private constructor(
     private val function: KFunctionForCall<T>,
-    parameterNameConverter: (String) -> String
+    parameterNameConverter: ((String) -> String)?
 ) {
-    constructor(function: KFunction<T>, parameterNameConverter: (String) -> String = { it }) : this(
+    constructor(function: KFunction<T>, parameterNameConverter: ((String) -> String)? = null) : this(
         KFunctionForCall(function, parameterNameConverter), parameterNameConverter
     )
 
-    constructor(clazz: KClass<T>, parameterNameConverter: (String) -> String = { it }) : this(
+    constructor(clazz: KClass<T>, parameterNameConverter: ((String) -> String)? = null) : this(
         clazz.toKConstructor(parameterNameConverter), parameterNameConverter
     )
 

src/main/kotlin/com/mapk/kmapper/ParameterForMap.kt

@@ -10,7 +10,7 @@ import kotlin.reflect.full.isSuperclassOf
 
 internal class ParameterForMap<T : Any>(
     param: ValueParameter<T>,
-    private val parameterNameConverter: (String) -> String
+    private val parameterNameConverter: ((String) -> String)?
 ) {
     val name: String = param.name
     private val clazz: KClass<T> = param.requiredClazz
@@ -61,7 +61,7 @@ private sealed class ParameterProcessor {
     abstract fun process(value: Any): Any?
 
     object Plain : ParameterProcessor() {
-        override fun process(value: Any): Any? = value
+        override fun process(value: Any): Any = value
     }
 
     class UseConverter(private val converter: KFunction<*>) : ParameterProcessor() {
@@ -82,6 +82,6 @@ private sealed class ParameterProcessor {
     }
 
     object ToString : ParameterProcessor() {
-        override fun process(value: Any): Any? = value.toString()
+        override fun process(value: Any): Any = value.toString()
     }
 }

src/main/kotlin/com/mapk/kmapper/PlainKMapper.kt

@@ -14,13 +14,13 @@ import kotlin.reflect.jvm.javaGetter
 
 class PlainKMapper<T : Any> private constructor(
     private val function: KFunctionForCall<T>,
-    parameterNameConverter: (String) -> String
+    parameterNameConverter: ((String) -> String)?
 ) {
-    constructor(function: KFunction<T>, parameterNameConverter: (String) -> String = { it }) : this(
+    constructor(function: KFunction<T>, parameterNameConverter: ((String) -> String)? = null) : this(
         KFunctionForCall(function, parameterNameConverter), parameterNameConverter
     )
 
-    constructor(clazz: KClass<T>, parameterNameConverter: (String) -> String = { it }) : this(
+    constructor(clazz: KClass<T>, parameterNameConverter: ((String) -> String)? = null) : this(
         clazz.toKConstructor(parameterNameConverter), parameterNameConverter
     )
 

src/main/kotlin/com/mapk/kmapper/PlainParameterForMap.kt

@@ -8,7 +8,7 @@ import kotlin.reflect.full.isSuperclassOf
 
 internal class PlainParameterForMap<T : Any>(
     param: ValueParameter<T>,
-    private val parameterNameConverter: (String) -> String
+    private val parameterNameConverter: ((String) -> String)?
 ) {
     private val clazz: KClass<T> = param.requiredClazz