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

Refactoring: HasActualMarker (use not-null declarations)

Browse Source
This commit is contained in:
Mikhail Glukhikh
2018-08-20 18:20:25 +03:00
parent 9b29651371
commit 8ef2c479e4
2 changed files with 6 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files
idea/src/org/jetbrains/kotlin/idea/highlighter/markers/HasActualMarker.kt
+4 -4
View File
@@ -27,8 +27,8 @@ import org.jetbrains.kotlin.resolve.descriptorUtil.module
import org.jetbrains.kotlin.resolve.getMultiTargetPlatform
import java.awt.event.MouseEvent
fun getPlatformActualTooltip(declaration: KtDeclaration?): String? {
val actualDeclarations = declaration?.actualsForExpected() ?: return null
fun getPlatformActualTooltip(declaration: KtDeclaration): String? {
val actualDeclarations = declaration.actualsForExpected()
if (actualDeclarations.isEmpty()) return null
return actualDeclarations.asSequence()
@@ -49,8 +49,8 @@ fun getPlatformActualTooltip(declaration: KtDeclaration?): String? {
}
}
fun navigateToPlatformActual(e: MouseEvent?, declaration: KtDeclaration?) {
val actualDeclarations = declaration?.actualsForExpected() ?: return
fun navigateToPlatformActual(e: MouseEvent?, declaration: KtDeclaration) {
val actualDeclarations = declaration.actualsForExpected()
if (actualDeclarations.isEmpty()) return
val renderer = object : DefaultPsiElementCellRenderer() {
idea/src/org/jetbrains/kotlin/idea/highlighter/markers/KotlinLineMarkerProvider.kt
+2 -2
View File
@@ -200,10 +200,10 @@ private val PsiElement.markerDeclaration
private val PLATFORM_ACTUAL = MarkerType(
"PLATFORM_ACTUAL",
{ it?.let { getPlatformActualTooltip(it.markerDeclaration) } },
{ element -> element?.markerDeclaration?.let { getPlatformActualTooltip(it) } },
object : LineMarkerNavigator() {
override fun browse(e: MouseEvent?, element: PsiElement?) {
element?.let { navigateToPlatformActual(e, it.markerDeclaration) }
element?.markerDeclaration?.let { navigateToPlatformActual(e, it) }
}
}
)