Also ignore UnusedPrivateMember for actual classes, objects and const…

…ructors
detekt · Apr 10, 2021 · a654113 · a654113
1 parent e4938c6
commit a654113
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/...ules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/UnusedPrivateMember.kt b/...ules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/UnusedPrivateMember.kt
@@ -205,7 +205,7 @@ private class UnusedParameterVisitor(allowedNames: Regex) : UnusedMemberVisitor(
     }
 
     override fun visitClassOrObject(klassOrObject: KtClassOrObject) {
-        if (klassOrObject.isExpect()) return
+        if (klassOrObject.isExpect() || klassOrObject.isActual()) return
 
         super.visitClassOrObject(klassOrObject)
     }
@@ -294,7 +294,11 @@ private class UnusedPropertyVisitor(allowedNames: Regex) : UnusedMemberVisitor(a
     override fun visitPrimaryConstructor(constructor: KtPrimaryConstructor) {
         super.visitPrimaryConstructor(constructor)
         constructor.valueParameters
-            .filter { (it.isPrivate() || !it.hasValOrVar()) && it.containingClassOrObject?.isExpect() == false }
+            .filter {
+                (it.isPrivate() || !it.hasValOrVar()) &&
+                    it.containingClassOrObject?.isExpect() == false &&
+                    it.containingClassOrObject?.isActual() == false
+            }
             .forEach { maybeAddUnusedProperty(it) }
     }