Optimization.

app/src/main/java/net/thauvin/erik/android/tesremoteprogrammer/StepsActivity.kt

@@ -43,12 +43,8 @@ class StepsActivity : FragmentActivity() {
                                      steps: ArrayList<String>) : FragmentStatePagerAdapter(fm) {
         private val steps = ArrayList<String>(steps)
 
-        override fun getItem(position: Int): Fragment {
+        override fun getItem(position: Int): Fragment = StepsFragment.create(position, steps)
-            return StepsFragment.create(position, steps)
-        }
 
-        override fun getCount(): Int {
+        override fun getCount(): Int = steps.size
-            return steps.size
-        }
     }
 }

app/src/main/java/net/thauvin/erik/android/tesremoteprogrammer/StepsFragment.kt

@@ -52,9 +52,8 @@ class StepsFragment : Fragment() {
     }
 
     override fun onCreateView(inflater: LayoutInflater, container: ViewGroup?,
-                              savedInstanceState: Bundle?): View? {
+                              savedInstanceState: Bundle?): View? =
-        return inflater.inflate(R.layout.fragment_steps, container, false) as ViewGroup
+            inflater.inflate(R.layout.fragment_steps, container, false) as ViewGroup
-    }
 
     override fun onViewCreated(view: View?, savedInstanceState: Bundle?) {
         super.onViewCreated(view, savedInstanceState)

app/src/main/java/net/thauvin/erik/android/tesremoteprogrammer/filters/AlphaFilter.kt

@@ -39,7 +39,7 @@ class AlphaFilter(private val extras: String) : InputFilter {
             return source
         } else {
             val sb = StringBuilder()
-            (start..end - 1)
+            (start until end)
                     .map { source[it] }
                     .filter { it.isLetterOrDigit() || extras.contains(it) }
                     .forEach { sb.append(it) }

app/src/main/java/net/thauvin/erik/android/tesremoteprogrammer/filters/MinMaxFilter.kt

@@ -39,10 +39,10 @@ class MinMaxFilter(private val min: Int, private val max: Int, private val size:
 
             if (zeros) {
                 if (size > 1 && len != size) {
-                    if (i == 0) {
+                    i = if (i == 0) {
                         return true
                     } else {
-                        i = s.padEnd(size, '0').toInt()
+                        s.padEnd(size, '0').toInt()
                     }
                 }
             } else {

app/src/main/java/net/thauvin/erik/android/tesremoteprogrammer/filters/NumberFilter.kt

@@ -44,7 +44,7 @@ class NumberFilter(allowed: String, alt: String) : InputFilter, AnkoLogger {
             return source
         } else {
             val sb = StringBuilder()
-            (start..end - 1)
+            (start until end)
                     .map { source[it] }
                     .filter { allowed.contains(it) }
                     .forEach { sb.append(it.toUpperCase()) }

app/src/main/java/net/thauvin/erik/android/tesremoteprogrammer/models/Config.kt

@@ -37,9 +37,7 @@ data class Config(var params: Params, var opts: List<Option>) : Parcelable, Seri
             source.readParcelable<Params>(Params::class.java.classLoader),
             source.createTypedArrayList(Option.CREATOR))
 
-    override fun compareTo(other: Config): Int {
+    override fun compareTo(other: Config): Int = params.name.compareTo(other.params.name)
-        return params.name.compareTo(other.params.name)
-    }
 
     override fun describeContents() = 0
 

app/src/main/java/net/thauvin/erik/android/tesremoteprogrammer/models/Option.kt

@@ -45,9 +45,7 @@ data class Option(var title: String,
             1 == source.readInt(),
             source.readString())
 
-    override fun compareTo(other: Option): Int {
+    override fun compareTo(other: Option): Int = title.compareTo(other.title)
-        return title.compareTo(other.title)
-    }
 
     override fun describeContents() = 0
 

app/src/main/java/net/thauvin/erik/android/tesremoteprogrammer/util/Dtmf.kt

@@ -87,9 +87,7 @@ class Dtmf {
             return result.toString()
         }
 
-        fun isValidType(type: String): Boolean {
+        fun isValidType(type: String): Boolean = type.equals(DKS, true) || type.equals(LINEAR, true)
-            return type.equals(DKS, true) || type.equals(LINEAR, true)
-        }
 
 
         private fun linearAlphaToDigits(text: String): String {
@@ -189,8 +187,7 @@ class Dtmf {
         fun mock(option: Option, blank: String): String {
             val replace = arrayListOf(Pair(DTMF_MASTER, blank))
 
-            @Suppress("UNUSED_PARAMETER")
+            option.fields.forEachIndexed { i, _ ->
-            option.fields.forEachIndexed { i, field ->
                 replace.add(Pair(DTMF_FIELD.format(i + 1), blank))
             }
 

app/src/main/java/net/thauvin/erik/android/tesremoteprogrammer/util/Strings.kt

@@ -31,13 +31,9 @@ fun String.isDigits(): Boolean {
     return true
 }
 
-fun String.isDKS(): Boolean {
+fun String.isDKS(): Boolean = equals(Dtmf.DKS, true)
-    return equals(Dtmf.DKS, true)
-}
 
-fun String.isLinear(): Boolean {
+fun String.isLinear(): Boolean = equals(Dtmf.LINEAR, true)
-    return equals(Dtmf.LINEAR, true)
-}
 
 fun String.replaceAll(replace: Array<Pair<String, String>>): String {
     val result = StringBuilder(this)