Minor: readability improvements after code review
This commit is contained in:
+1
-3
@@ -31,7 +31,6 @@ import org.jetbrains.kotlin.codegen.pseudoInsns.PseudoInsn
|
|||||||
import org.jetbrains.kotlin.codegen.pseudoInsns.isPseudo
|
import org.jetbrains.kotlin.codegen.pseudoInsns.isPseudo
|
||||||
import org.jetbrains.kotlin.resolve.jvm.AsmTypes
|
import org.jetbrains.kotlin.resolve.jvm.AsmTypes
|
||||||
import org.jetbrains.kotlin.utils.SmartList
|
import org.jetbrains.kotlin.utils.SmartList
|
||||||
import org.jetbrains.kotlin.utils.addToStdlib.cast
|
|
||||||
import org.jetbrains.org.objectweb.asm.Label
|
import org.jetbrains.org.objectweb.asm.Label
|
||||||
import org.jetbrains.org.objectweb.asm.Opcodes
|
import org.jetbrains.org.objectweb.asm.Opcodes
|
||||||
import org.jetbrains.org.objectweb.asm.Type
|
import org.jetbrains.org.objectweb.asm.Type
|
||||||
@@ -397,12 +396,11 @@ class RedundantNullCheckMethodTransformer : MethodTransformer() {
|
|||||||
// <...> -- v is not null here because codegen told us so
|
// <...> -- v is not null here because codegen told us so
|
||||||
val previous = insn.previous
|
val previous = insn.previous
|
||||||
if (previous.opcode != Opcodes.ASTORE) return
|
if (previous.opcode != Opcodes.ASTORE) return
|
||||||
val varIndex = previous.cast<VarInsnNode>().`var`
|
|
||||||
|
|
||||||
methodNode.instructions.run {
|
methodNode.instructions.run {
|
||||||
insert(insn, listOfSynthetics {
|
insert(insn, listOfSynthetics {
|
||||||
anew(varType)
|
anew(varType)
|
||||||
store(varIndex, varType)
|
store((previous as VarInsnNode).`var`, varType)
|
||||||
})
|
})
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -366,7 +366,7 @@ object KeywordCompletion {
|
|||||||
}
|
}
|
||||||
val modifierTargets = ModifierCheckerCore.possibleTargetMap[keywordTokenType]?.intersect(possibleTargets)
|
val modifierTargets = ModifierCheckerCore.possibleTargetMap[keywordTokenType]?.intersect(possibleTargets)
|
||||||
if (modifierTargets != null && possibleTargets.isNotEmpty() &&
|
if (modifierTargets != null && possibleTargets.isNotEmpty() &&
|
||||||
!modifierTargets.any {
|
modifierTargets.none {
|
||||||
isModifierTargetSupportedAtLanguageLevel(keywordTokenType, it, languageVersionSettings)
|
isModifierTargetSupportedAtLanguageLevel(keywordTokenType, it, languageVersionSettings)
|
||||||
}) return false
|
}) return false
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user