From 182b0e399a42b9d5decdc48ea2c79e99f86f60a9 Mon Sep 17 00:00:00 2001
From: anna <Anna.Kozlova@jetbrains.com>
Date: Tue, 14 Sep 2010 19:47:33 +0400
Subject: [PATCH] change signature: do not add comma if only one parameter
 added

---
 .../changeSignature/ChangeSignatureUtil.java  | 28 ++++++++++---------
 1 file changed, 15 insertions(+), 13 deletions(-)

diff --git a/java/java-impl/src/com/intellij/refactoring/changeSignature/ChangeSignatureUtil.java b/java/java-impl/src/com/intellij/refactoring/changeSignature/ChangeSignatureUtil.java
index f23cabd6143b..665d3d574c48 100644
--- a/java/java-impl/src/com/intellij/refactoring/changeSignature/ChangeSignatureUtil.java
+++ b/java/java-impl/src/com/intellij/refactoring/changeSignature/ChangeSignatureUtil.java
@@ -72,19 +72,21 @@ public class ChangeSignatureUtil {
           }
         }
       } else {
-        PsiElement anchor = null;
-        if (index == 0) {
-          anchor = list.getFirstChild();
-        } else {
-          anchor = elements.get(index - 1);
-        }
-        final PsiElement psi = Factory
-          .createSingleLeafElement(JavaTokenType.COMMA, ",", 0, 1, SharedImplUtil.findCharTableByTree(list.getNode()), list.getManager())
-          .getPsi();
-        if (anchor != null) {
-          list.addAfter(psi, anchor);
-        } else {
-          list.add(psi);
+        if (newElements.size() > 1) {
+          PsiElement anchor;
+          if (index == 0) {
+            anchor = list.getFirstChild();
+          } else {
+            anchor = elements.get(index - 1);
+          }
+          final PsiElement psi = Factory
+            .createSingleLeafElement(JavaTokenType.COMMA, ",", 0, 1, SharedImplUtil.findCharTableByTree(list.getNode()), list.getManager())
+            .getPsi();
+          if (anchor != null) {
+            list.addAfter(psi, anchor);
+          } else {
+            list.add(psi);
+          }
         }
       }
       index++;