Corrections after review + one more test added

idea/src/org/jetbrains/jet/plugin/completion/handlers/KotlinCallableInsertHandler.kt

@@ -153,8 +153,7 @@ public class KotlinFunctionInsertHandler(val caretPosition : CaretPosition, val
         if (completionChar == Lookup.REPLACE_SELECT_CHAR) {
             offset = skipSpaces(chars, offset)
             if (offset < document.getTextLength()) {
-                val c = chars[offset]
+                if (chars[offset] == '<') {
-                if (c == '<') {
                     PsiDocumentManager.getInstance(context.getProject()).commitDocument(document)
                     val psiFile = context.getFile()
                     val token = psiFile.findElementAt(offset)

idea/src/org/jetbrains/jet/plugin/completion/handlers/KotlinClassInsertHandler.kt

@@ -29,7 +29,6 @@ import com.intellij.psi.PsiClass
 import org.jetbrains.jet.lang.psi.JetNameReferenceExpression
 import org.jetbrains.jet.plugin.caches.resolve.getResolutionFacade
 import org.jetbrains.jet.lang.resolve.BindingContext
-import org.jetbrains.jet.lang.resolve.DescriptorUtils
 import org.jetbrains.jet.lang.resolve.lazy.BodyResolveMode
 import org.jetbrains.jet.lang.descriptors.ClassKind
 
@@ -53,13 +52,7 @@ public object KotlinClassInsertHandler : BaseDeclarationInsertHandler() {
                 if (nameRef != null) {
                     val bindingContext = nameRef.getResolutionFacade().analyze(nameRef, BodyResolveMode.PARTIAL)
                     val target = bindingContext[BindingContext.REFERENCE_TARGET, nameRef] as? ClassDescriptor
-                    if (target != null) {
+                    if (target != null && qualifiedNameForSourceCode(target) == qualifiedName) return
-                        val realTarget = if (target.getKind() == ClassKind.CLASS_OBJECT)
-                            target.getContainingDeclaration() as ClassDescriptor
-                        else
-                            target
-                        if (DescriptorUtils.getFqName(realTarget).asString() == qualifiedName) return
-                    }
                 }
 
                 val tempPrefix = if (nameRef != null)

idea/testData/completion/handlers/basic/DoNotUseParenthesisOnNextLine.kt

@@ -0,0 +1,8 @@
+fun foo(p: Int){}
+
+fun f() {
+    f<caret>
+    (a + b).x()
+}
+
+// ELEMENT: foo

idea/testData/completion/handlers/basic/DoNotUseParenthesisOnNextLine.kt.after

@@ -0,0 +1,8 @@
+fun foo(p: Int){}
+
+fun f() {
+    foo(<caret>)
+    (a + b).x()
+}
+
+// ELEMENT: foo

idea/tests/org/jetbrains/jet/completion/handlers/BasicCompletionHandlerTestGenerated.java

@@ -42,6 +42,12 @@ public class BasicCompletionHandlerTestGenerated extends AbstractBasicCompletion
         doTest(fileName);
     }
 
+    @TestMetadata("DoNotUseParenthesisOnNextLine.kt")
+    public void testDoNotUseParenthesisOnNextLine() throws Exception {
+        String fileName = JetTestUtils.navigationMetadata("idea/testData/completion/handlers/basic/DoNotUseParenthesisOnNextLine.kt");
+        doTest(fileName);
+    }
+
     @TestMetadata("ExtensionReceiverTypeArg.kt")
     public void testExtensionReceiverTypeArg() throws Exception {
         String fileName = JetTestUtils.navigationMetadata("idea/testData/completion/handlers/basic/ExtensionReceiverTypeArg.kt");