From 6e0285bb2aae7c033b4f817c07b5d0caa1c2fbd4 Mon Sep 17 00:00:00 2001
From: Sergey Prigogin <eclipse.sprigogin@gmail.com>
Date: Wed, 21 Sep 2011 14:22:30 -0700
Subject: [PATCH] Revert "Bug 357300 - Function parameter line wrapping wraps
 commas separating parameters"

This reverts commit cd17d74157b9c6bb56241efb848b4238e2519e04.
---
 .../internal/formatter/CodeFormatterVisitor.java   | 14 ++++++++------
 .../cdt/ui/tests/text/CodeFormatterTest.java       |  8 --------
 2 files changed, 8 insertions(+), 14 deletions(-)

diff --git a/core/org.eclipse.cdt.core/src/org/eclipse/cdt/internal/formatter/CodeFormatterVisitor.java b/core/org.eclipse.cdt.core/src/org/eclipse/cdt/internal/formatter/CodeFormatterVisitor.java
index 553f2c6b5e3..8cc1eb756e9 100644
--- a/core/org.eclipse.cdt.core/src/org/eclipse/cdt/internal/formatter/CodeFormatterVisitor.java
+++ b/core/org.eclipse.cdt.core/src/org/eclipse/cdt/internal/formatter/CodeFormatterVisitor.java
@@ -2089,6 +2089,14 @@ public class CodeFormatterVisitor extends ASTVisitor implements ICPPASTVisitor,
 					int i;
 					for (i = 0; i < elementsLength; i++) {
 						final IASTNode node= elements.get(i);
+						if (i < elementsLength - 1) {
+							scribe.setTailFormatter(
+									new TrailingTokenFormatter(options.fSeparatorToken,
+											options.fSpaceBeforeSeparator,
+											options.fSpaceAfterSeparator));
+						} else {
+							scribe.setTailFormatter(tailFormatter);
+						}
 						scribe.alignFragment(alignment, i);
 						if (node instanceof ICPPASTConstructorChainInitializer) {
 							// Constructor chain initializer is a special case.
@@ -2097,13 +2105,7 @@ public class CodeFormatterVisitor extends ASTVisitor implements ICPPASTVisitor,
 							node.accept(this);
 						}
 						if (i < elementsLength - 1) {
-							scribe.setTailFormatter(
-									new TrailingTokenFormatter(options.fSeparatorToken,
-											options.fSpaceBeforeSeparator,
-											options.fSpaceAfterSeparator));
 							scribe.runTailFormatter();
-						} else {
-							scribe.setTailFormatter(tailFormatter);
 						}
 					}
 					if (addEllipsis) {
diff --git a/core/org.eclipse.cdt.ui.tests/ui/org/eclipse/cdt/ui/tests/text/CodeFormatterTest.java b/core/org.eclipse.cdt.ui.tests/ui/org/eclipse/cdt/ui/tests/text/CodeFormatterTest.java
index 16b944194ec..35feb483ca8 100644
--- a/core/org.eclipse.cdt.ui.tests/ui/org/eclipse/cdt/ui/tests/text/CodeFormatterTest.java
+++ b/core/org.eclipse.cdt.ui.tests/ui/org/eclipse/cdt/ui/tests/text/CodeFormatterTest.java
@@ -2688,12 +2688,4 @@ public class CodeFormatterTest extends BaseUITestCase {
 		assertFormatterResult();
 	}
 
-	//void wrap_when_necessary(more_than_one_template_argument_t<int, float> p1, no_template_arguments_t p2, no_template_arguments_t p3) {}
-
-	//void wrap_when_necessary(more_than_one_template_argument_t<int, float> p1,
-	//		no_template_arguments_t p2, no_template_arguments_t p3) {
-	//}
-	public void testTemplateIdInParameterList_Bug357300() throws Exception {
-		assertFormatterResult();
-	}
 }