From 82b9f0cd2f981fabd1cfc0782ad607fc7af679d9 Mon Sep 17 00:00:00 2001
From: Anton Leherbauer <anton.leherbauer@windriver.com>
Date: Fri, 6 Apr 2007 11:54:18 +0000
Subject: [PATCH] Fix auto-insert of closing brace with semicolon for anonymous
 declarations

---
 .../cdt/ui/tests/text/CAutoIndentTest.java    | 11 ++++++++--
 .../internal/ui/text/CHeuristicScanner.java   | 21 +++++++++++++------
 2 files changed, 24 insertions(+), 8 deletions(-)

diff --git a/core/org.eclipse.cdt.ui.tests/ui/org/eclipse/cdt/ui/tests/text/CAutoIndentTest.java b/core/org.eclipse.cdt.ui.tests/ui/org/eclipse/cdt/ui/tests/text/CAutoIndentTest.java
index 5a7f8865c06..a090f207c61 100644
--- a/core/org.eclipse.cdt.ui.tests/ui/org/eclipse/cdt/ui/tests/text/CAutoIndentTest.java
+++ b/core/org.eclipse.cdt.ui.tests/ui/org/eclipse/cdt/ui/tests/text/CAutoIndentTest.java
@@ -426,7 +426,7 @@ public class CAutoIndentTest extends TestCase {
 	public void testBracketWithSemiColonInsertion() throws BadLocationException {
 		AutoEditTester tester = createAutoEditTester(); 
 		String[] kw= new String[] {"class", "union", "struct", "enum"};
-		String[] kw_inh= new String[] {"class", "union", "struct", "enum"};
+		String[] kw_anon= new String[] {"union", "struct", "enum"};
 
 		for(int i=0; i<kw.length; i++) {
 			tester.reset();
@@ -476,7 +476,14 @@ public class CAutoIndentTest extends TestCase {
 			tester.type(kw[i]+" /* for(int i=0; i<100; i++) {} */\nA \n{\n"); //$NON-NLS-1$
 			assertEquals("\n// foo\n"+kw[i]+" /* for(int i=0; i<100; i++) {} */\nA \n{\n\t\n};\n\n//bar\n\n", tester.fDoc.get()); //$NON-NLS-1$
 		}		
-	}
+
+		for(int i=0; i<kw_anon.length; i++) {		
+			tester.reset();
+
+			tester.type("\n\n\n"+kw_anon[i]+" {\n"); //$NON-NLS-1$
+			assertEquals("\n\n\n"+kw_anon[i]+" {\n\t\n};", tester.fDoc.get()); //$NON-NLS-1$
+		}
+}
 	
 	/**
 	 * Tests that brackets are inserted (without semi-colons) in appropriate
diff --git a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/CHeuristicScanner.java b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/CHeuristicScanner.java
index 4f1bfb5e7ea..f6320116bff 100644
--- a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/CHeuristicScanner.java
+++ b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/CHeuristicScanner.java
@@ -963,7 +963,12 @@ public final class CHeuristicScanner implements Symbols {
 	 */
 	public boolean looksLikeCompositeTypeDefinitionBackward(int start, int bound) {
 		int token= previousToken(start - 1, bound);
-		if (token == Symbols.TokenIDENT) {
+		switch (token) {
+		case Symbols.TokenSTRUCT:
+		case Symbols.TokenUNION:
+		case Symbols.TokenENUM:
+			return true; // anonymous
+		case Symbols.TokenIDENT:
 			token= previousToken(getPosition(), bound);
 			switch (token) {
 			case Symbols.TokenCLASS:
@@ -974,8 +979,12 @@ public final class CHeuristicScanner implements Symbols {
 			default:
 				// backtrack
 				token= previousToken(start - 1, bound);
-				break;
 			}
+			break;
+		default:
+			// backtrack
+			token= previousToken(start - 1, bound);
+			break;
 		}
 		// match base-clause
 		if (token == Symbols.TokenGREATERTHAN) {
@@ -986,13 +995,13 @@ public final class CHeuristicScanner implements Symbols {
 			}
 			token= previousToken(getPosition(), bound);
 		}
-		outer: while (token == Symbols.TokenIDENT) {// type name or base type
+		outerWhile: while (token == Symbols.TokenIDENT) {// type name or base type
 			token= previousToken(getPosition(), bound);
 			// match nested-name-specifier
 			while (token == Symbols.TokenCOLON) { // colon of qualification
 				token= previousToken(getPosition(), bound);
 				if (token != Symbols.TokenCOLON) { // second colon of qualification
-					break outer;
+					break outerWhile;
 				}
 				token= previousToken(getPosition(), bound);
 				if (token != Symbols.TokenIDENT) // qualification name?
@@ -1027,7 +1036,7 @@ public final class CHeuristicScanner implements Symbols {
 				/* fallthrough */
 			case Symbols.TokenCOLON:
 				token= previousToken(getPosition(), bound);
-				break outer;
+				break outerWhile;
 			case Symbols.TokenCOMMA:
 				token= previousToken(getPosition(), bound);
 				if (token == Symbols.TokenGREATERTHAN) {
@@ -1040,7 +1049,7 @@ public final class CHeuristicScanner implements Symbols {
 				}
 				continue; // another base type
 			case Symbols.TokenIDENT:
-				break outer;
+				break outerWhile;
 			default:
 				return false;
 			}