Improve code style

Kotlin · Dec 20, 2021 · 472a4da · 472a4da
1 parent 848a8b0
commit 472a4da
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 11 deletions.
diff --git a/...ase/src/main/kotlin/transformers/documentables/DocumentableVisibilityFilterTransformer.kt b/...ase/src/main/kotlin/transformers/documentables/DocumentableVisibilityFilterTransformer.kt
@@ -81,9 +81,12 @@ class DocumentableVisibilityFilterTransformer(val context: DokkaContext) : PreMe
             return Pair(packagesListChanged, filteredPackages)
         }
 
+        @Suppress("UNUSED_PARAMETER")
         private fun <T : WithVisibility> alwaysTrue(a: T, p: DokkaSourceSet) = true
+        @Suppress("UNUSED_PARAMETER")
         private fun <T : WithVisibility> alwaysFalse(a: T, p: DokkaSourceSet) = false
-        private fun <T> alwaysNoModify(a: T, sourceSets: Set<DokkaSourceSet>) = Pair(false, a)
+        @Suppress("UNUSED_PARAMETER")
+        private fun <T> alwaysNoModify(a: T, sourceSets: Set<DokkaSourceSet>) = false to a
 
         private fun WithVisibility.visibilityForPlatform(data: DokkaSourceSet): Visibility? = visibility[data]
 
@@ -159,16 +162,14 @@ class DocumentableVisibilityFilterTransformer(val context: DokkaContext) : PreMe
 
                     val modified = setter?.first == true || getter?.first == true
 
-                    Pair(modified,
+                    val property =
                         if (modified)
-                            with(original) {
-                                copy(
-                                    setter = setter?.second?.firstOrNull(),
-                                    getter = getter?.second?.firstOrNull()
-                                )
-                            }
+                            original.copy(
+                                setter = setter?.second?.firstOrNull(),
+                                getter = getter?.second?.firstOrNull()
+                            )
                         else original
-                    )
+                    modified to property
                 }
 
             return properties.transform(

diff --git a/plugins/base/src/test/kotlin/transformers/SuppressTagFilterTest.kt b/plugins/base/src/test/kotlin/transformers/SuppressTagFilterTest.kt
@@ -2,7 +2,6 @@ package transformers
 
 import org.jetbrains.dokka.base.testApi.testRunner.BaseAbstractTest
 import org.jetbrains.dokka.model.DEnum
-import org.jetbrains.dokka.model.DProperty
 import org.jetbrains.dokka.model.WithCompanion
 import org.junit.jupiter.api.Test
 import kotlin.test.assertEquals
@@ -101,7 +100,7 @@ class SuppressTagFilterTest : BaseAbstractTest() {
         ) {
             preMergeDocumentablesTransformationStage = { modules ->
                 val prop = modules.flatMap { it.packages }.flatMap { it.properties }
-                    .firstOrNull { it.name == "property" } as? DProperty
+                    .find { it.name == "property" }
                 assertNotNull(prop)
                 assertNotNull(prop.getter)
                 assertNull(prop.setter)