Minor. More proper short-circuit check.

Evgeny Gerashchenko · Evgeny Gerashchenko · commit 01b8718ac7b3 · 2015-04-08T15:33:49.000+03:00
diff --git a/idea/src/org/jetbrains/kotlin/idea/highlighter/KotlinRecursiveCallLineMarkerProvider.kt b/idea/src/org/jetbrains/kotlin/idea/highlighter/KotlinRecursiveCallLineMarkerProvider.kt
@@ -62,8 +62,7 @@ public class KotlinRecursiveCallLineMarkerProvider() : LineMarkerProvider {
     }
 
     private fun isRecursiveCall(element: JetElement): Boolean {
-        if (!(element is JetSimpleNameExpression && element.getParent() is JetCallExpression || element is JetArrayAccessExpression))
-            return false
+        if (element.getParent() is JetCallableReferenceExpression) return false
         val enclosingFunction = getEnclosingFunction(element) ?: return false
 
         val bindingContext = element.analyze()

Original file line number	Diff line number	Diff line change
`@@ -62,8 +62,7 @@ public class KotlinRecursiveCallLineMarkerProvider() : LineMarkerProvider {`
`62`	`62`	`}`
`63`	`63`
`64`	`64`	`private fun isRecursiveCall(element: JetElement): Boolean {`
`65`		`- if (!(element is JetSimpleNameExpression && element.getParent() is JetCallExpression \|\| element is JetArrayAccessExpression))`
`66`		`- return false`
	`65`	`+ if (element.getParent() is JetCallableReferenceExpression) return false`
`67`	`66`	`val enclosingFunction = getEnclosingFunction(element) ?: return false`
`68`	`67`
`69`	`68`	`val bindingContext = element.analyze()`