Can be parameter inspection: usages in all functions are now handled as property usage
This commit is contained in:
@@ -55,13 +55,13 @@ class CanBeParameterInspection : AbstractKotlinInspection() {
|
|||||||
do {
|
do {
|
||||||
parameterUser = PsiTreeUtil.getParentOfType(parameterUser, KtProperty::class.java, KtPropertyAccessor::class.java,
|
parameterUser = PsiTreeUtil.getParentOfType(parameterUser, KtProperty::class.java, KtPropertyAccessor::class.java,
|
||||||
KtClassInitializer::class.java, KtSecondaryConstructor::class.java,
|
KtClassInitializer::class.java, KtSecondaryConstructor::class.java,
|
||||||
KtFunctionLiteral::class.java, KtObjectDeclaration::class.java) ?: return true
|
KtFunction::class.java, KtObjectDeclaration::class.java) ?: return true
|
||||||
} while (parameterUser is KtProperty && parameterUser.isLocal)
|
} while (parameterUser is KtProperty && parameterUser.isLocal)
|
||||||
return when (parameterUser) {
|
return when (parameterUser) {
|
||||||
is KtProperty -> parameterUser.containingClassOrObject !== klass
|
is KtProperty -> parameterUser.containingClassOrObject !== klass
|
||||||
is KtClassInitializer -> parameterUser.containingDeclaration !== klass
|
is KtClassInitializer -> parameterUser.containingDeclaration !== klass
|
||||||
is KtSecondaryConstructor -> parameterUser.getContainingClassOrObject() !== klass
|
is KtSecondaryConstructor -> parameterUser.getContainingClassOrObject() !== klass
|
||||||
is KtFunctionLiteral, is KtObjectDeclaration, is KtPropertyAccessor -> true
|
is KtFunction, is KtObjectDeclaration, is KtPropertyAccessor -> true
|
||||||
else -> true
|
else -> true
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -126,4 +126,13 @@ class UsedInObjectLiteral(val x: Int) {
|
|||||||
val y = object: Any() {
|
val y = object: Any() {
|
||||||
fun bar() = x
|
fun bar() = x
|
||||||
}
|
}
|
||||||
|
}
|
||||||
|
// NO
|
||||||
|
class UsedInLocalFunction(val x: Int) {
|
||||||
|
init {
|
||||||
|
fun local() {
|
||||||
|
val y = x
|
||||||
|
}
|
||||||
|
local()
|
||||||
|
}
|
||||||
}
|
}
|
||||||
Reference in New Issue
Block a user