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

Remove unused function parameter: minor improvement

Browse Source
This commit is contained in:
Mikhail Glukhikh
2018-01-22 11:13:38 +03:00
parent 1298ef7fed
commit 2472966749
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
idea/src/org/jetbrains/kotlin/idea/quickfix/RemoveUnusedFunctionParameterFix.kt
+2 -2
View File
@@ -38,15 +38,15 @@ class RemoveUnusedFunctionParameterFix(parameter: KtParameter) : KotlinQuickFixA
override fun invoke(project: Project, editor: Editor?, file: KtFile) { override fun invoke(project: Project, editor: Editor?, file: KtFile) {
val element = element ?: return val element = element ?: return
val primaryConstructor = element.parent?.parent as? KtPrimaryConstructor
val parameterList = element.parent as? KtParameterList
val context = element.analyze() val context = element.analyze()
val parameterDescriptor = context[BindingContext.VALUE_PARAMETER, element] as? ValueParameterDescriptor ?: return val parameterDescriptor = context[BindingContext.VALUE_PARAMETER, element] as? ValueParameterDescriptor ?: return
ChangeFunctionSignatureFix.runRemoveParameter(parameterDescriptor, element) ChangeFunctionSignatureFix.runRemoveParameter(parameterDescriptor, element)
val parameterList = element.parent as? KtParameterList
val nextParameter = parameterList?.parameters?.getOrNull(parameterDescriptor.index) val nextParameter = parameterList?.parameters?.getOrNull(parameterDescriptor.index)
if (nextParameter != null) { if (nextParameter != null) {
editor?.caretModel?.moveToOffset(nextParameter.startOffset) editor?.caretModel?.moveToOffset(nextParameter.startOffset)
} }
val primaryConstructor = element.parent?.parent as? KtPrimaryConstructor
if (primaryConstructor != null) { if (primaryConstructor != null) {
val removeConstructorIntention = RemoveEmptyPrimaryConstructorIntention() val removeConstructorIntention = RemoveEmptyPrimaryConstructorIntention()
if (removeConstructorIntention.isApplicableTo(primaryConstructor)) { if (removeConstructorIntention.isApplicableTo(primaryConstructor)) {