azalea/kotlin-fork
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Reformat UnusedReceiverParameterInspection

Browse Source
This commit is contained in:
Toshiaki Kameyama
2018-04-27 13:50:08 +09:00
committed by Nikolay Krasko
parent 68dbeeac7a
commit d32b53362d
1 changed files with 10 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files
idea/src/org/jetbrains/kotlin/idea/inspections/UnusedReceiverParameterInspection.kt
+10 -7
View File
@@ -59,7 +59,8 @@ class UnusedReceiverParameterInspection : AbstractKotlinInspection() {
if (callableDeclaration.isOverridable() || if (callableDeclaration.isOverridable() ||
callableDeclaration.hasModifier(KtTokens.OVERRIDE_KEYWORD) || callableDeclaration.hasModifier(KtTokens.OVERRIDE_KEYWORD) ||
callableDeclaration.hasModifier(KtTokens.OPERATOR_KEYWORD) || callableDeclaration.hasModifier(KtTokens.OPERATOR_KEYWORD) ||
callableDeclaration.hasModifier(KtTokens.INFIX_KEYWORD)) return callableDeclaration.hasModifier(KtTokens.INFIX_KEYWORD)
) return
if (callableDeclaration is KtProperty && callableDeclaration.accessors.isEmpty()) return if (callableDeclaration is KtProperty && callableDeclaration.accessors.isEmpty()) return
if (callableDeclaration is KtNamedFunction && !callableDeclaration.hasBody()) return if (callableDeclaration is KtNamedFunction && !callableDeclaration.hasBody()) return
@@ -80,7 +81,8 @@ class UnusedReceiverParameterInspection : AbstractKotlinInspection() {
if (isUsageOfReceiver(resolvedCall, bindingContext)) { if (isUsageOfReceiver(resolvedCall, bindingContext)) {
used = true used = true
} else if (resolvedCall is VariableAsFunctionResolvedCall } else if (resolvedCall is VariableAsFunctionResolvedCall
&& isUsageOfReceiver(resolvedCall.variableCall, bindingContext)) { && isUsageOfReceiver(resolvedCall.variableCall, bindingContext)
) {
used = true used = true
} }
} }
@@ -88,7 +90,8 @@ class UnusedReceiverParameterInspection : AbstractKotlinInspection() {
private fun isUsageOfReceiver(resolvedCall: ResolvedCall<*>, bindingContext: BindingContext): Boolean { private fun isUsageOfReceiver(resolvedCall: ResolvedCall<*>, bindingContext: BindingContext): Boolean {
// As receiver of call // As receiver of call
if (resolvedCall.dispatchReceiver.getThisReceiverOwner(bindingContext) == callable || if (resolvedCall.dispatchReceiver.getThisReceiverOwner(bindingContext) == callable ||
resolvedCall.extensionReceiver.getThisReceiverOwner(bindingContext) == callable) { resolvedCall.extensionReceiver.getThisReceiverOwner(bindingContext) == callable
) {
return true return true
} }
// As explicit "this" // As explicit "this"
@@ -101,10 +104,10 @@ class UnusedReceiverParameterInspection : AbstractKotlinInspection() {
if (!used) { if (!used) {
holder.registerProblem( holder.registerProblem(
receiverTypeReference, receiverTypeReference,
KotlinBundle.message("unused.receiver.parameter"), KotlinBundle.message("unused.receiver.parameter"),
ProblemHighlightType.LIKE_UNUSED_SYMBOL, ProblemHighlightType.LIKE_UNUSED_SYMBOL,
MyQuickFix() MyQuickFix()
) )
} }
} }