diff --git a/detekt-rules-complexity/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/complexity/NamedArguments.kt b/detekt-rules-complexity/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/complexity/NamedArguments.kt index 2fc021d9fe26..f624d45a74bd 100644 --- a/detekt-rules-complexity/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/complexity/NamedArguments.kt +++ b/detekt-rules-complexity/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/complexity/NamedArguments.kt @@ -49,7 +49,6 @@ class NamedArguments(config: Config = Config.empty) : Rule(config) { private val ignoreArgumentsMatchingNames: Boolean by config(defaultValue = false) override fun visitCallExpression(expression: KtCallExpression) { - if (bindingContext == BindingContext.EMPTY) return val valueArguments = expression.valueArguments.filterNot { it is KtLambdaArgument } if (valueArguments.size > threshold && expression.canNameArguments()) { val message = "This function call has ${valueArguments.size} arguments. To call a function with more " + diff --git a/detekt-rules-complexity/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/complexity/NestedScopeFunctions.kt b/detekt-rules-complexity/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/complexity/NestedScopeFunctions.kt index 81a0dc39d7f6..b31fb702b540 100644 --- a/detekt-rules-complexity/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/complexity/NestedScopeFunctions.kt +++ b/detekt-rules-complexity/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/complexity/NestedScopeFunctions.kt @@ -66,7 +66,6 @@ class NestedScopeFunctions(config: Config = Config.empty) : Rule(config) { } override fun visitNamedFunction(function: KtNamedFunction) { - if (bindingContext == BindingContext.EMPTY) return function.accept(FunctionDepthVisitor()) } diff --git a/detekt-rules-complexity/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/complexity/ReplaceSafeCallChainWithRun.kt b/detekt-rules-complexity/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/complexity/ReplaceSafeCallChainWithRun.kt index 4d0209811e40..c29252475bfd 100644 --- a/detekt-rules-complexity/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/complexity/ReplaceSafeCallChainWithRun.kt +++ b/detekt-rules-complexity/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/complexity/ReplaceSafeCallChainWithRun.kt @@ -52,8 +52,6 @@ class ReplaceSafeCallChainWithRun(config: Config = Config.empty) : Rule(config) override fun visitSafeQualifiedExpression(expression: KtSafeQualifiedExpression) { super.visitSafeQualifiedExpression(expression) - if (bindingContext == BindingContext.EMPTY) return - /* We want the last safe qualified expression in the chain, so if there are more in this chain then there's no need to run checks on this one */ if (expression.parent is KtSafeQualifiedExpression) return diff --git a/detekt-rules-complexity/src/test/kotlin/io/gitlab/arturbosch/detekt/rules/complexity/NestedScopeFunctionsSpec.kt b/detekt-rules-complexity/src/test/kotlin/io/gitlab/arturbosch/detekt/rules/complexity/NestedScopeFunctionsSpec.kt index 583ab5dd918f..a6c60916fd22 100644 --- a/detekt-rules-complexity/src/test/kotlin/io/gitlab/arturbosch/detekt/rules/complexity/NestedScopeFunctionsSpec.kt +++ b/detekt-rules-complexity/src/test/kotlin/io/gitlab/arturbosch/detekt/rules/complexity/NestedScopeFunctionsSpec.kt @@ -4,7 +4,6 @@ import io.gitlab.arturbosch.detekt.api.Finding import io.gitlab.arturbosch.detekt.rules.KotlinCoreEnvironmentTest import io.gitlab.arturbosch.detekt.test.TestConfig import io.gitlab.arturbosch.detekt.test.assertThat -import io.gitlab.arturbosch.detekt.test.compileAndLint import io.gitlab.arturbosch.detekt.test.compileAndLintWithContext import org.jetbrains.kotlin.cli.jvm.compiler.KotlinCoreEnvironment import org.junit.jupiter.api.Test @@ -94,27 +93,10 @@ class NestedScopeFunctionsSpec(private val env: KotlinCoreEnvironment) { expectNoFindings() } - @Test - fun `should not report when binding context is empty`() { - givenCode = """ - fun f() { - 1.run { - 1.run { } - } - } - """ - whenLintRunsWithoutContext() - expectNoFindings() - } - private fun whenLintRuns() { actual = subject.compileAndLintWithContext(env, givenCode) } - private fun whenLintRunsWithoutContext() { - actual = subject.compileAndLint(givenCode) - } - private fun expectSourceLocation(location: Pair) { assertThat(actual).hasStartSourceLocation(location.first, location.second) } diff --git a/detekt-rules-coroutines/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/coroutines/InjectDispatcher.kt b/detekt-rules-coroutines/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/coroutines/InjectDispatcher.kt index a3453a61c1f6..d863c21cc5df 100644 --- a/detekt-rules-coroutines/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/coroutines/InjectDispatcher.kt +++ b/detekt-rules-coroutines/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/coroutines/InjectDispatcher.kt @@ -16,7 +16,6 @@ import org.jetbrains.kotlin.name.FqName import org.jetbrains.kotlin.psi.KtParameter import org.jetbrains.kotlin.psi.KtSimpleNameExpression import org.jetbrains.kotlin.psi.psiUtil.getStrictParentOfType -import org.jetbrains.kotlin.resolve.BindingContext.EMPTY import org.jetbrains.kotlin.resolve.calls.util.getType import org.jetbrains.kotlin.types.typeUtil.supertypes @@ -56,7 +55,7 @@ class InjectDispatcher(config: Config) : Rule(config) { override fun visitSimpleNameExpression(expression: KtSimpleNameExpression) { super.visitSimpleNameExpression(expression) - if (bindingContext == EMPTY || expression.getReferencedName() !in dispatcherNames) return + if (expression.getReferencedName() !in dispatcherNames) return val type = expression.getType(bindingContext) ?: return val isCoroutineDispatcher = type.fqNameOrNull() == COROUTINE_DISPATCHER_FQCN || type.supertypes().any { it.fqNameOrNull() == COROUTINE_DISPATCHER_FQCN } diff --git a/detekt-rules-coroutines/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/coroutines/RedundantSuspendModifier.kt b/detekt-rules-coroutines/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/coroutines/RedundantSuspendModifier.kt index 0005021245cc..95ae114332a0 100644 --- a/detekt-rules-coroutines/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/coroutines/RedundantSuspendModifier.kt +++ b/detekt-rules-coroutines/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/coroutines/RedundantSuspendModifier.kt @@ -66,7 +66,6 @@ class RedundantSuspendModifier(config: Config) : Rule(config) { ) override fun visitNamedFunction(function: KtNamedFunction) { - if (bindingContext == BindingContext.EMPTY) return val suspendModifier = function.modifierList?.getModifier(KtTokens.SUSPEND_KEYWORD) ?: return if (!function.hasBody()) return if (function.hasModifier(KtTokens.OVERRIDE_KEYWORD)) return diff --git a/detekt-rules-coroutines/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/coroutines/SleepInsteadOfDelay.kt b/detekt-rules-coroutines/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/coroutines/SleepInsteadOfDelay.kt index de15f72bcf25..20ffc6c40459 100644 --- a/detekt-rules-coroutines/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/coroutines/SleepInsteadOfDelay.kt +++ b/detekt-rules-coroutines/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/coroutines/SleepInsteadOfDelay.kt @@ -50,7 +50,6 @@ class SleepInsteadOfDelay(config: Config = Config.empty) : Rule(config) { ) override fun visitNamedFunction(function: KtNamedFunction) { - if (bindingContext == BindingContext.EMPTY) return if (function.modifierList?.hasSuspendModifier() == true) { function.checkDescendants(SUSPEND_FUN_MESSAGE) } diff --git a/detekt-rules-coroutines/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/coroutines/SuspendFunWithCoroutineScopeReceiver.kt b/detekt-rules-coroutines/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/coroutines/SuspendFunWithCoroutineScopeReceiver.kt index 5b18e5837560..0a56c4d415f0 100644 --- a/detekt-rules-coroutines/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/coroutines/SuspendFunWithCoroutineScopeReceiver.kt +++ b/detekt-rules-coroutines/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/coroutines/SuspendFunWithCoroutineScopeReceiver.kt @@ -60,7 +60,6 @@ class SuspendFunWithCoroutineScopeReceiver(config: Config) : Rule(config) { ) override fun visitNamedFunction(function: KtNamedFunction) { - if (bindingContext == BindingContext.EMPTY) return checkReceiver(function) } diff --git a/detekt-rules-coroutines/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/coroutines/SuspendFunWithFlowReturnType.kt b/detekt-rules-coroutines/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/coroutines/SuspendFunWithFlowReturnType.kt index 45b5688cda5d..0067684f4d54 100644 --- a/detekt-rules-coroutines/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/coroutines/SuspendFunWithFlowReturnType.kt +++ b/detekt-rules-coroutines/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/coroutines/SuspendFunWithFlowReturnType.kt @@ -73,7 +73,6 @@ class SuspendFunWithFlowReturnType(config: Config) : Rule(config) { ) override fun visitNamedFunction(function: KtNamedFunction) { - if (bindingContext == BindingContext.EMPTY) return val suspendModifier = function.modifierList?.getModifier(KtTokens.SUSPEND_KEYWORD) ?: return bindingContext[BindingContext.FUNCTION, function] ?.returnType diff --git a/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/AvoidReferentialEquality.kt b/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/AvoidReferentialEquality.kt index 1b249949f4af..d01abc18d6c0 100644 --- a/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/AvoidReferentialEquality.kt +++ b/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/AvoidReferentialEquality.kt @@ -62,7 +62,6 @@ class AvoidReferentialEquality(config: Config) : Rule(config) { } private fun checkBinaryExpression(expression: KtBinaryExpression) { - if (bindingContext == BindingContext.EMPTY) return if (expression.operationToken != EQEQEQ && expression.operationToken != EXCLEQEQEQ) return val checkedType = expression.left?.getType(bindingContext)?.fqNameOrNull() ?: return diff --git a/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/Deprecation.kt b/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/Deprecation.kt index 615f1cf77c3c..43cfb4e2d422 100644 --- a/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/Deprecation.kt +++ b/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/Deprecation.kt @@ -30,7 +30,6 @@ class Deprecation(config: Config) : Rule(config) { override val defaultRuleIdAliases = setOf("DEPRECATION") override fun visitElement(element: PsiElement) { - if (bindingContext == BindingContext.EMPTY) return if (hasDeprecationCompilerWarnings(element)) { val entity = if (element is KtNamedDeclaration) Entity.atName(element) else Entity.from(element) report(CodeSmell(issue, entity, "${element.text} is deprecated.")) diff --git a/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/DontDowncastCollectionTypes.kt b/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/DontDowncastCollectionTypes.kt index 3310b269f4af..0d0ce4579669 100644 --- a/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/DontDowncastCollectionTypes.kt +++ b/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/DontDowncastCollectionTypes.kt @@ -50,7 +50,6 @@ class DontDowncastCollectionTypes(config: Config) : Rule(config) { override fun visitIsExpression(expression: KtIsExpression) { super.visitIsExpression(expression) - if (bindingContext == BindingContext.EMPTY) return checkForDowncast(expression, expression.leftHandSide, expression.typeReference) } @@ -58,8 +57,6 @@ class DontDowncastCollectionTypes(config: Config) : Rule(config) { override fun visitBinaryWithTypeRHSExpression(expression: KtBinaryExpressionWithTypeRHS) { super.visitBinaryWithTypeRHSExpression(expression) - if (bindingContext == BindingContext.EMPTY) return - checkForDowncast(expression, expression.left, expression.right) } diff --git a/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/DoubleMutabilityForCollection.kt b/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/DoubleMutabilityForCollection.kt index 57d6f8fb2f37..50501ffba336 100644 --- a/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/DoubleMutabilityForCollection.kt +++ b/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/DoubleMutabilityForCollection.kt @@ -65,8 +65,6 @@ class DoubleMutabilityForCollection(config: Config = Config.empty) : Rule(config override fun visitProperty(property: KtProperty) { super.visitProperty(property) - if (bindingContext == BindingContext.EMPTY) return - val type = (bindingContext[BindingContext.VARIABLE, property])?.type ?: return val standardType = type.fqNameOrNull() if (property.isVar && standardType in mutableTypes) { diff --git a/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/ElseCaseInsteadOfExhaustiveWhen.kt b/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/ElseCaseInsteadOfExhaustiveWhen.kt index 59d710006078..6323a4e13064 100644 --- a/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/ElseCaseInsteadOfExhaustiveWhen.kt +++ b/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/ElseCaseInsteadOfExhaustiveWhen.kt @@ -63,7 +63,6 @@ class ElseCaseInsteadOfExhaustiveWhen(config: Config = Config.empty) : Rule(conf override fun visitWhenExpression(whenExpression: KtWhenExpression) { super.visitWhenExpression(whenExpression) - if (bindingContext == BindingContext.EMPTY) return checkForElseCaseInsteadOfExhaustiveWhenExpression(whenExpression) } diff --git a/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/ExitOutsideMain.kt b/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/ExitOutsideMain.kt index d40c3c347a55..ae5ed566644d 100644 --- a/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/ExitOutsideMain.kt +++ b/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/ExitOutsideMain.kt @@ -58,8 +58,6 @@ class ExitOutsideMain(config: Config = Config.empty) : Rule(config) { override fun visitCallExpression(expression: KtCallExpression) { super.visitCallExpression(expression) - if (bindingContext == BindingContext.EMPTY) return - if (expression.getStrictParentOfType()?.isMainFunction() == true) return val fqName = expression.getResolvedCall(bindingContext)?.resultingDescriptor?.fqNameOrNull() ?: return diff --git a/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/HasPlatformType.kt b/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/HasPlatformType.kt index 80f8088c26fe..0e6e892bed8a 100644 --- a/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/HasPlatformType.kt +++ b/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/HasPlatformType.kt @@ -51,7 +51,6 @@ class HasPlatformType(config: Config) : Rule(config) { override fun visitKtElement(element: KtElement) { super.visitKtElement(element) - if (bindingContext == BindingContext.EMPTY) return if (element is KtCallableDeclaration && element.hasImplicitPlatformType()) { report( diff --git a/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/IgnoredReturnValue.kt b/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/IgnoredReturnValue.kt index b5eb9c48df43..fa2f51ef0535 100644 --- a/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/IgnoredReturnValue.kt +++ b/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/IgnoredReturnValue.kt @@ -75,7 +75,6 @@ class IgnoredReturnValue(config: Config = Config.empty) : Rule(config) { @Suppress("ReturnCount") override fun visitCallExpression(expression: KtCallExpression) { super.visitCallExpression(expression) - if (bindingContext == BindingContext.EMPTY) return if (expression.isUsedAsExpression(bindingContext)) return diff --git a/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/ImplicitUnitReturnType.kt b/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/ImplicitUnitReturnType.kt index e4d4c0a5f56e..fc5791917e01 100644 --- a/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/ImplicitUnitReturnType.kt +++ b/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/ImplicitUnitReturnType.kt @@ -58,9 +58,6 @@ class ImplicitUnitReturnType(config: Config) : Rule(config) { @Suppress("ReturnCount") override fun visitNamedFunction(function: KtNamedFunction) { super.visitNamedFunction(function) - if (BindingContext.EMPTY == bindingContext) { - return - } if (allowExplicitReturnType && function.hasDeclaredReturnType()) { return diff --git a/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/MissingWhenCase.kt b/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/MissingWhenCase.kt index e91dcd1a7b04..0f23759fceec 100644 --- a/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/MissingWhenCase.kt +++ b/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/MissingWhenCase.kt @@ -83,7 +83,6 @@ class MissingWhenCase(config: Config = Config.empty) : Rule(config) { override fun visitWhenExpression(expression: KtWhenExpression) { super.visitWhenExpression(expression) - if (bindingContext == BindingContext.EMPTY) return if (allowElseExpression && expression.elseExpression != null) return checkMissingWhenExpression(expression) } diff --git a/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/NullCheckOnMutableProperty.kt b/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/NullCheckOnMutableProperty.kt index 1a153e64491b..6d7798b7e532 100644 --- a/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/NullCheckOnMutableProperty.kt +++ b/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/NullCheckOnMutableProperty.kt @@ -62,7 +62,6 @@ class NullCheckOnMutableProperty(config: Config) : Rule(config) { ) override fun visitKtFile(file: KtFile) { - if (bindingContext == BindingContext.EMPTY) return super.visitKtFile(file) NullCheckVisitor().visitKtFile(file) } diff --git a/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/NullableToStringCall.kt b/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/NullableToStringCall.kt index b6d4eb7b86d0..833772e13d5f 100644 --- a/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/NullableToStringCall.kt +++ b/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/NullableToStringCall.kt @@ -61,7 +61,6 @@ class NullableToStringCall(config: Config = Config.empty) : Rule(config) { override fun visitSimpleNameExpression(expression: KtSimpleNameExpression) { super.visitSimpleNameExpression(expression) - if (bindingContext == BindingContext.EMPTY) return val simpleOrCallExpression = expression.parent.safeAs() ?: expression val targetExpression = simpleOrCallExpression.targetExpression() ?: return diff --git a/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/RedundantElseInWhen.kt b/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/RedundantElseInWhen.kt index e4a2b80d806b..5bc8fb8abdb5 100644 --- a/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/RedundantElseInWhen.kt +++ b/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/RedundantElseInWhen.kt @@ -72,7 +72,6 @@ class RedundantElseInWhen(config: Config = Config.empty) : Rule(config) { override fun visitWhenExpression(whenExpression: KtWhenExpression) { super.visitWhenExpression(whenExpression) - if (bindingContext == BindingContext.EMPTY) return val elseEntry = whenExpression.entries.lastOrNull { it.isElse } ?: return val compilerReports = bindingContext.diagnostics.forElement(elseEntry) if (compilerReports.any { it.factory == Errors.REDUNDANT_ELSE_IN_WHEN }) { diff --git a/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/UnnecessaryNotNullOperator.kt b/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/UnnecessaryNotNullOperator.kt index d6c60d4319bd..0f232aee2e98 100644 --- a/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/UnnecessaryNotNullOperator.kt +++ b/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/UnnecessaryNotNullOperator.kt @@ -39,7 +39,6 @@ class UnnecessaryNotNullOperator(config: Config = Config.empty) : Rule(config) { override fun visitUnaryExpression(expression: KtUnaryExpression) { super.visitUnaryExpression(expression) - if (bindingContext == BindingContext.EMPTY) return val compilerReports = bindingContext.diagnostics.forElement(expression.operationReference) if (compilerReports.any { it.factory == Errors.UNNECESSARY_NOT_NULL_ASSERTION }) { diff --git a/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/UnnecessarySafeCall.kt b/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/UnnecessarySafeCall.kt index 290723135eb3..b6929da1be1d 100644 --- a/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/UnnecessarySafeCall.kt +++ b/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/UnnecessarySafeCall.kt @@ -44,8 +44,6 @@ class UnnecessarySafeCall(config: Config = Config.empty) : Rule(config) { override fun visitSafeQualifiedExpression(expression: KtSafeQualifiedExpression) { super.visitSafeQualifiedExpression(expression) - if (bindingContext == BindingContext.EMPTY) return - val safeAccessElement = expression.getChildOfType() if (safeAccessElement == null || safeAccessElement.elementType != KtTokens.SAFE_ACCESS) { return diff --git a/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/UnreachableCatchBlock.kt b/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/UnreachableCatchBlock.kt index 6846f42cd13a..9a00e0662ac0 100644 --- a/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/UnreachableCatchBlock.kt +++ b/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/UnreachableCatchBlock.kt @@ -60,7 +60,6 @@ class UnreachableCatchBlock(config: Config = Config.empty) : Rule(config) { @Suppress("ReturnCount") override fun visitCatchSection(catchClause: KtCatchClause) { super.visitCatchSection(catchClause) - if (bindingContext == BindingContext.EMPTY) return val tryExpression = catchClause.getStrictParentOfType() ?: return val prevCatchClauses = tryExpression.catchClauses.takeWhile { it != catchClause } diff --git a/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/UnreachableCode.kt b/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/UnreachableCode.kt index 5dac3f7bf149..c874709ef561 100644 --- a/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/UnreachableCode.kt +++ b/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/UnreachableCode.kt @@ -46,9 +46,7 @@ class UnreachableCode(config: Config = Config.empty) : Rule(config) { override fun visitExpression(expression: KtExpression) { super.visitExpression(expression) - if (bindingContext != BindingContext.EMPTY && - bindingContext.diagnostics.forElement(expression).any { it.factory == Errors.UNREACHABLE_CODE } - ) { + if (bindingContext.diagnostics.forElement(expression).any { it.factory == Errors.UNREACHABLE_CODE }) { report( CodeSmell( issue, diff --git a/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/UnsafeCallOnNullableType.kt b/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/UnsafeCallOnNullableType.kt index f05ff1ec2e98..492d49ab1c99 100644 --- a/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/UnsafeCallOnNullableType.kt +++ b/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/UnsafeCallOnNullableType.kt @@ -46,7 +46,6 @@ class UnsafeCallOnNullableType(config: Config = Config.empty) : Rule(config) { override fun visitPostfixExpression(expression: KtPostfixExpression) { super.visitPostfixExpression(expression) - if (bindingContext == BindingContext.EMPTY) return if (expression.operationToken == KtTokens.EXCLEXCL && expression.baseExpression?.getType(bindingContext)?.nullability() == TypeNullability.NULLABLE ) { diff --git a/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/UnsafeCast.kt b/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/UnsafeCast.kt index b80155cfa46d..c2a9cb875e66 100644 --- a/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/UnsafeCast.kt +++ b/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/UnsafeCast.kt @@ -46,8 +46,6 @@ class UnsafeCast(config: Config = Config.empty) : Rule(config) { ) override fun visitBinaryWithTypeRHSExpression(expression: KtBinaryExpressionWithTypeRHS) { - if (bindingContext == BindingContext.EMPTY) return - if (bindingContext.diagnostics.forElement(expression.operationReference) .any { it.factory == Errors.CAST_NEVER_SUCCEEDS } ) { diff --git a/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/UnusedUnaryOperator.kt b/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/UnusedUnaryOperator.kt index da3b6e64487f..ae9a9dc52576 100644 --- a/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/UnusedUnaryOperator.kt +++ b/detekt-rules-errorprone/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/bugs/UnusedUnaryOperator.kt @@ -51,7 +51,6 @@ class UnusedUnaryOperator(config: Config = Config.empty) : Rule(config) { @Suppress("ReturnCount") override fun visitPrefixExpression(expression: KtPrefixExpression) { super.visitPrefixExpression(expression) - if (bindingContext == BindingContext.EMPTY) return if (expression.baseExpression == null) return val operationToken = expression.operationToken diff --git a/detekt-rules-exceptions/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/exceptions/ObjectExtendsThrowable.kt b/detekt-rules-exceptions/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/exceptions/ObjectExtendsThrowable.kt index e07a9f94466c..3643d8bdc621 100644 --- a/detekt-rules-exceptions/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/exceptions/ObjectExtendsThrowable.kt +++ b/detekt-rules-exceptions/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/exceptions/ObjectExtendsThrowable.kt @@ -55,7 +55,6 @@ class ObjectExtendsThrowable(config: Config = Config.empty) : Rule(config) { override fun visitObjectDeclaration(declaration: KtObjectDeclaration) { super.visitObjectDeclaration(declaration) - if (bindingContext == BindingContext.EMPTY) return if (!declaration.isObjectLiteral() && declaration.isSubtypeOfThrowable()) { report( CodeSmell( diff --git a/detekt-rules-exceptions/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/exceptions/ReturnFromFinally.kt b/detekt-rules-exceptions/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/exceptions/ReturnFromFinally.kt index 6e92d8cab796..0bc533e1b87e 100644 --- a/detekt-rules-exceptions/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/exceptions/ReturnFromFinally.kt +++ b/detekt-rules-exceptions/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/exceptions/ReturnFromFinally.kt @@ -57,7 +57,6 @@ class ReturnFromFinally(config: Config = Config.empty) : Rule(config) { override fun visitTryExpression(expression: KtTryExpression) { super.visitTryExpression(expression) - if (bindingContext == BindingContext.EMPTY) return val finallyBlock = expression.finallyBlock ?: return diff --git a/detekt-rules-naming/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/naming/BooleanPropertyNaming.kt b/detekt-rules-naming/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/naming/BooleanPropertyNaming.kt index 2bb5c283402d..42eea1006aaf 100644 --- a/detekt-rules-naming/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/naming/BooleanPropertyNaming.kt +++ b/detekt-rules-naming/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/naming/BooleanPropertyNaming.kt @@ -62,10 +62,6 @@ class BooleanPropertyNaming(config: Config = Config.empty) : Rule(config) { } private fun validateDeclaration(declaration: KtCallableDeclaration) { - if (bindingContext == BindingContext.EMPTY) { - return - } - val name = declaration.identifierName() val typeName = getTypeName(declaration) val isBooleanType = diff --git a/detekt-rules-naming/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/naming/NoNameShadowing.kt b/detekt-rules-naming/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/naming/NoNameShadowing.kt index e66e0321345c..56cc0eab9307 100644 --- a/detekt-rules-naming/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/naming/NoNameShadowing.kt +++ b/detekt-rules-naming/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/naming/NoNameShadowing.kt @@ -76,16 +76,13 @@ class NoNameShadowing(config: Config = Config.empty) : Rule(config) { private fun checkNameShadowing(declaration: KtNamedDeclaration) { val nameIdentifier = declaration.nameIdentifier ?: return - if (bindingContext != BindingContext.EMPTY && - bindingContext.diagnostics.forElement(declaration).any { it.factory == Errors.NAME_SHADOWING } - ) { + if (bindingContext.diagnostics.forElement(declaration).any { it.factory == Errors.NAME_SHADOWING }) { report(CodeSmell(issue, Entity.from(nameIdentifier), "Name shadowed: ${nameIdentifier.text}")) } } override fun visitLambdaExpression(lambdaExpression: KtLambdaExpression) { super.visitLambdaExpression(lambdaExpression) - if (bindingContext == BindingContext.EMPTY) return val implicitParameter = lambdaExpression.implicitParameter(bindingContext) ?: return if (lambdaExpression.hasImplicitParameterReference(implicitParameter, bindingContext) && lambdaExpression.hasParentImplicitParameterLambda() diff --git a/detekt-rules-naming/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/naming/NonBooleanPropertyPrefixedWithIs.kt b/detekt-rules-naming/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/naming/NonBooleanPropertyPrefixedWithIs.kt index e8c17808be66..4ea8688fd807 100644 --- a/detekt-rules-naming/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/naming/NonBooleanPropertyPrefixedWithIs.kt +++ b/detekt-rules-naming/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/naming/NonBooleanPropertyPrefixedWithIs.kt @@ -59,10 +59,6 @@ class NonBooleanPropertyPrefixedWithIs(config: Config = Config.empty) : Rule(con } private fun validateDeclaration(declaration: KtCallableDeclaration) { - if (bindingContext == BindingContext.EMPTY) { - return - } - val name = declaration.identifierName() if (name.startsWith("is") && name.length > 2 && !name[2].isLowerCase()) { diff --git a/detekt-rules-performance/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/performance/ArrayPrimitive.kt b/detekt-rules-performance/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/performance/ArrayPrimitive.kt index 95348b92ee15..b8ff7b21196c 100644 --- a/detekt-rules-performance/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/performance/ArrayPrimitive.kt +++ b/detekt-rules-performance/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/performance/ArrayPrimitive.kt @@ -52,11 +52,6 @@ class ArrayPrimitive(config: Config = Config.empty) : Rule(config) { Debt.FIVE_MINS ) - override fun visitKtFile(file: KtFile) { - if (bindingContext == BindingContext.EMPTY) return - super.visitKtFile(file) - } - override fun visitCallExpression(expression: KtCallExpression) { super.visitCallExpression(expression) if (expression.calleeExpression?.text !in factoryMethodNames) return diff --git a/detekt-rules-performance/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/performance/CouldBeSequence.kt b/detekt-rules-performance/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/performance/CouldBeSequence.kt index 178cded16eb2..9b394fd500e5 100644 --- a/detekt-rules-performance/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/performance/CouldBeSequence.kt +++ b/detekt-rules-performance/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/performance/CouldBeSequence.kt @@ -51,8 +51,6 @@ class CouldBeSequence(config: Config = Config.empty) : Rule(config) { override fun visitCallExpression(expression: KtCallExpression) { super.visitCallExpression(expression) - if (bindingContext == BindingContext.EMPTY) return - if (visitedCallExpressions.contains(expression)) return if (!expression.isCalling(operationsFqNames)) return diff --git a/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/ExplicitCollectionElementAccessMethod.kt b/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/ExplicitCollectionElementAccessMethod.kt index 58e7eaeb37a4..8b00226bc07f 100644 --- a/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/ExplicitCollectionElementAccessMethod.kt +++ b/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/ExplicitCollectionElementAccessMethod.kt @@ -52,7 +52,6 @@ class ExplicitCollectionElementAccessMethod(config: Config = Config.empty) : Rul override fun visitDotQualifiedExpression(expression: KtDotQualifiedExpression) { super.visitDotQualifiedExpression(expression) - if (bindingContext == BindingContext.EMPTY) return val call = expression.selectorExpression as? KtCallExpression ?: return if (isIndexGetterRecommended(call) || isIndexSetterRecommended(call)) { report(CodeSmell(issue, Entity.from(expression), issue.description)) diff --git a/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/ForbiddenMethodCall.kt b/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/ForbiddenMethodCall.kt index a49cfb05eb21..8f7a89d927e2 100644 --- a/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/ForbiddenMethodCall.kt +++ b/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/ForbiddenMethodCall.kt @@ -105,8 +105,6 @@ class ForbiddenMethodCall(config: Config = Config.empty) : Rule(config) { } private fun check(expression: KtExpression) { - if (bindingContext == BindingContext.EMPTY) return - val descriptors = expression.getResolvedCall(bindingContext)?.resultingDescriptor?.let { val foundDescriptors = if (it is PropertyDescriptor) { listOfNotNull(it.unwrappedGetMethod, it.unwrappedSetMethod) diff --git a/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/ForbiddenVoid.kt b/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/ForbiddenVoid.kt index 6e67b33309a0..d98aa27f24e4 100644 --- a/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/ForbiddenVoid.kt +++ b/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/ForbiddenVoid.kt @@ -57,7 +57,6 @@ class ForbiddenVoid(config: Config = Config.empty) : Rule(config) { @Suppress("ReturnCount") override fun visitTypeReference(typeReference: KtTypeReference) { - if (bindingContext == BindingContext.EMPTY) return val kotlinType = typeReference.getAbbreviatedTypeOrType(bindingContext) ?: return if (kotlinType.fqNameOrNull() == VOID_FQ_NAME) { diff --git a/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/LibraryCodeMustSpecifyReturnType.kt b/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/LibraryCodeMustSpecifyReturnType.kt index d63904568d40..711bb43bbcd7 100644 --- a/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/LibraryCodeMustSpecifyReturnType.kt +++ b/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/LibraryCodeMustSpecifyReturnType.kt @@ -54,9 +54,6 @@ class LibraryCodeMustSpecifyReturnType(config: Config = Config.empty) : Rule(con ) override fun visitProperty(property: KtProperty) { - if (bindingContext == BindingContext.EMPTY) { - return - } if (property.explicitReturnTypeRequired()) { report( CodeSmell( diff --git a/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/MaxChainedCallsOnSameLine.kt b/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/MaxChainedCallsOnSameLine.kt index f64f09a2974b..10040c68ff93 100644 --- a/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/MaxChainedCallsOnSameLine.kt +++ b/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/MaxChainedCallsOnSameLine.kt @@ -47,8 +47,6 @@ class MaxChainedCallsOnSameLine(config: Config = Config.empty) : Rule(config) { override fun visitQualifiedExpression(expression: KtQualifiedExpression) { super.visitQualifiedExpression(expression) - if (bindingContext == BindingContext.EMPTY) return - val parent = expression.parent // skip if the parent is also a call on the same line to avoid duplicated warnings diff --git a/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/MultilineLambdaItParameter.kt b/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/MultilineLambdaItParameter.kt index 3c6d898bd6c7..efc7ecfe7b63 100644 --- a/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/MultilineLambdaItParameter.kt +++ b/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/MultilineLambdaItParameter.kt @@ -75,7 +75,6 @@ class MultilineLambdaItParameter(val config: Config) : Rule(config) { override fun visitLambdaExpression(lambdaExpression: KtLambdaExpression) { super.visitLambdaExpression(lambdaExpression) - if (bindingContext == BindingContext.EMPTY) return val size = lambdaExpression.bodyExpression?.statements?.size if (size == null || size <= 1) return diff --git a/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/ObjectLiteralToLambda.kt b/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/ObjectLiteralToLambda.kt index c4bbb2625272..bddc5ebe654f 100644 --- a/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/ObjectLiteralToLambda.kt +++ b/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/ObjectLiteralToLambda.kt @@ -96,7 +96,6 @@ class ObjectLiteralToLambda(config: Config = Config.empty) : Rule(config) { override fun visitObjectLiteralExpression(expression: KtObjectLiteralExpression) { super.visitObjectLiteralExpression(expression) - if (bindingContext == BindingContext.EMPTY) return val declaration = expression.objectDeclaration if ( diff --git a/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/RedundantExplicitType.kt b/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/RedundantExplicitType.kt index 8b28cd4f1231..5c709b2898bc 100644 --- a/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/RedundantExplicitType.kt +++ b/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/RedundantExplicitType.kt @@ -56,7 +56,6 @@ class RedundantExplicitType(config: Config) : Rule(config) { @Suppress("ReturnCount", "ComplexMethod") override fun visitProperty(property: KtProperty) { - if (bindingContext == BindingContext.EMPTY) return if (!property.isLocal) return val typeReference = property.typeReference ?: return val type = diff --git a/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/RedundantHigherOrderMapUsage.kt b/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/RedundantHigherOrderMapUsage.kt index 75f53400749c..cc9568e7338f 100644 --- a/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/RedundantHigherOrderMapUsage.kt +++ b/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/RedundantHigherOrderMapUsage.kt @@ -86,7 +86,6 @@ class RedundantHigherOrderMapUsage(config: Config = Config.empty) : Rule(config) override fun visitCallExpression(expression: KtCallExpression) { super.visitCallExpression(expression) - if (bindingContext == BindingContext.EMPTY) return val calleeExpression = expression.calleeExpression if (calleeExpression?.text != "map") return diff --git a/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/UnnecessaryApply.kt b/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/UnnecessaryApply.kt index dc9a4647843b..4d53753bf375 100644 --- a/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/UnnecessaryApply.kt +++ b/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/UnnecessaryApply.kt @@ -56,8 +56,6 @@ class UnnecessaryApply(config: Config) : Rule(config) { override fun visitCallExpression(expression: KtCallExpression) { super.visitCallExpression(expression) - if (bindingContext == BindingContext.EMPTY) return - if (expression.isApplyExpr() && expression.hasOnlyOneMemberAccessStatement() && !expression.receiverIsUsed(bindingContext) diff --git a/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/UnnecessaryFilter.kt b/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/UnnecessaryFilter.kt index 9fdf56e03e55..41fc67eb13f0 100644 --- a/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/UnnecessaryFilter.kt +++ b/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/UnnecessaryFilter.kt @@ -57,7 +57,6 @@ class UnnecessaryFilter(config: Config = Config.empty) : Rule(config) { @Suppress("ReturnCount") override fun visitCallExpression(expression: KtCallExpression) { super.visitCallExpression(expression) - if (bindingContext == BindingContext.EMPTY) return if (!expression.isCalling(filterFqNames)) return val lambdaArgumentText = expression.lambda()?.text ?: return diff --git a/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/UnnecessaryInnerClass.kt b/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/UnnecessaryInnerClass.kt index 3b4ea4c6f0a3..ccbb65cc3d01 100644 --- a/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/UnnecessaryInnerClass.kt +++ b/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/UnnecessaryInnerClass.kt @@ -54,11 +54,6 @@ class UnnecessaryInnerClass(config: Config = Config.empty) : Rule(config) { Debt.FIVE_MINS ) - override fun visit(root: KtFile) { - if (bindingContext == BindingContext.EMPTY) return - super.visit(root) - } - override fun visitClass(klass: KtClass) { classChain.add(klass) if (klass.isInner()) { diff --git a/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/UnnecessaryLet.kt b/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/UnnecessaryLet.kt index b6698e4e6af3..531d7503de22 100644 --- a/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/UnnecessaryLet.kt +++ b/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/UnnecessaryLet.kt @@ -60,8 +60,6 @@ class UnnecessaryLet(config: Config) : Rule(config) { override fun visitCallExpression(expression: KtCallExpression) { super.visitCallExpression(expression) - if (bindingContext == BindingContext.EMPTY) return - if (!expression.isLetExpr()) return val lambdaExpr = expression.firstLambdaArg diff --git a/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/UseAnyOrNoneInsteadOfFind.kt b/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/UseAnyOrNoneInsteadOfFind.kt index 87be6f8cf936..b4ab77663223 100644 --- a/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/UseAnyOrNoneInsteadOfFind.kt +++ b/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/UseAnyOrNoneInsteadOfFind.kt @@ -45,7 +45,6 @@ class UseAnyOrNoneInsteadOfFind(config: Config = Config.empty) : Rule(config) { @Suppress("ReturnCount") override fun visitCallExpression(expression: KtCallExpression) { super.visitCallExpression(expression) - if (bindingContext == BindingContext.EMPTY) return val functionName = expression.calleeExpression?.text ?: return val qualifiedOrThis = expression.getStrictParentOfType() ?: expression diff --git a/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/UseCheckNotNull.kt b/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/UseCheckNotNull.kt index 9b64c40222fa..775a234c2505 100644 --- a/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/UseCheckNotNull.kt +++ b/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/UseCheckNotNull.kt @@ -37,7 +37,6 @@ class UseCheckNotNull(config: Config = Config.empty) : Rule(config) { override fun visitCallExpression(expression: KtCallExpression) { super.visitCallExpression(expression) - if (bindingContext == BindingContext.EMPTY) return if (expression.isCallingWithNonNullCheckArgument(checkFunctionFqName, bindingContext)) { report(CodeSmell(issue, Entity.from(expression), issue.description)) } diff --git a/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/UseIfEmptyOrIfBlank.kt b/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/UseIfEmptyOrIfBlank.kt index 8b0c14352b5c..7d20cc464831 100644 --- a/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/UseIfEmptyOrIfBlank.kt +++ b/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/UseIfEmptyOrIfBlank.kt @@ -59,7 +59,6 @@ class UseIfEmptyOrIfBlank(config: Config = Config.empty) : Rule(config) { override fun visitIfExpression(expression: KtIfExpression) { super.visitIfExpression(expression) - if (bindingContext == BindingContext.EMPTY) return if (expression.isElseIf()) return val thenExpression = expression.then ?: return diff --git a/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/UseIsNullOrEmpty.kt b/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/UseIsNullOrEmpty.kt index b10efcc64aef..d91ffcf543e1 100644 --- a/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/UseIsNullOrEmpty.kt +++ b/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/UseIsNullOrEmpty.kt @@ -59,7 +59,6 @@ class UseIsNullOrEmpty(config: Config = Config.empty) : Rule(config) { @Suppress("ReturnCount") override fun visitBinaryExpression(expression: KtBinaryExpression) { super.visitBinaryExpression(expression) - if (bindingContext == BindingContext.EMPTY) return if (expression.operationToken != KtTokens.OROR) return val left = expression.left as? KtBinaryExpression ?: return diff --git a/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/UseOrEmpty.kt b/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/UseOrEmpty.kt index 8f4e7a697982..6c89b0bb5619 100644 --- a/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/UseOrEmpty.kt +++ b/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/UseOrEmpty.kt @@ -57,7 +57,6 @@ class UseOrEmpty(config: Config = Config.empty) : Rule(config) { @Suppress("ReturnCount") override fun visitBinaryExpression(expression: KtBinaryExpression) { super.visitBinaryExpression(expression) - if (bindingContext == BindingContext.EMPTY) return if (expression.operationToken != KtTokens.ELVIS) return val left = expression.left ?: return diff --git a/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/UseRequireNotNull.kt b/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/UseRequireNotNull.kt index d2eec9f51c03..0f85c2a6bbb7 100644 --- a/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/UseRequireNotNull.kt +++ b/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/UseRequireNotNull.kt @@ -37,7 +37,6 @@ class UseRequireNotNull(config: Config = Config.empty) : Rule(config) { override fun visitCallExpression(expression: KtCallExpression) { super.visitCallExpression(expression) - if (bindingContext == BindingContext.EMPTY) return if (expression.isCallingWithNonNullCheckArgument(requireFunctionFqName, bindingContext)) { report(CodeSmell(issue, Entity.from(expression), issue.description)) } diff --git a/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/UselessCallOnNotNull.kt b/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/UselessCallOnNotNull.kt index 8f6e0eb6a920..09c5a121c9f3 100644 --- a/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/UselessCallOnNotNull.kt +++ b/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/UselessCallOnNotNull.kt @@ -61,7 +61,6 @@ class UselessCallOnNotNull(config: Config = Config.empty) : Rule(config) { @Suppress("ReturnCount") override fun visitQualifiedExpression(expression: KtQualifiedExpression) { super.visitQualifiedExpression(expression) - if (bindingContext == BindingContext.EMPTY) return val safeExpression = expression as? KtSafeQualifiedExpression val notNullType = expression.receiverExpression.getType(bindingContext)?.isNullable() == false diff --git a/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/VarCouldBeVal.kt b/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/VarCouldBeVal.kt index 8bc2fc7c2f48..da8c7c62ba81 100644 --- a/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/VarCouldBeVal.kt +++ b/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/VarCouldBeVal.kt @@ -76,7 +76,6 @@ class VarCouldBeVal(config: Config = Config.empty) : Rule(config) { override fun visitKtFile(file: KtFile) { super.visitKtFile(file) - if (bindingContext == BindingContext.EMPTY) return val assignmentVisitor = AssignmentVisitor(bindingContext, ignoreLateinitVar) file.accept(assignmentVisitor) diff --git a/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/optional/PreferToOverPairSyntax.kt b/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/optional/PreferToOverPairSyntax.kt index 37228a57b851..9e795b3999f2 100644 --- a/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/optional/PreferToOverPairSyntax.kt +++ b/detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/optional/PreferToOverPairSyntax.kt @@ -42,7 +42,6 @@ class PreferToOverPairSyntax(config: Config = Config.empty) : Rule(config) { override fun visitCallExpression(expression: KtCallExpression) { super.visitCallExpression(expression) - if (bindingContext == BindingContext.EMPTY) return if (expression.isPairConstructor()) { val arg = expression.valueArguments.joinToString(" to ") { it.text } report(