Remove problem characters from task related branch name suggestion

In addition to standard branch name normalization, remove also specific
mylyn related problem characters when creating the branch name from a
mylyn task context.

Signed-off-by: Michael Keppler <Michael.Keppler@gmx.de>
Change-Id: I7a6a254c75f37fac12930e33771820d9b47b29e7
Bug: 531171
diff --git a/org.eclipse.egit.mylyn.ui/src/org/eclipse/egit/internal/mylyn/ui/commit/ActiveTaskBranchNameProvider.java b/org.eclipse.egit.mylyn.ui/src/org/eclipse/egit/internal/mylyn/ui/commit/ActiveTaskBranchNameProvider.java
index 8c06b6d..a15ddb0 100644
--- a/org.eclipse.egit.mylyn.ui/src/org/eclipse/egit/internal/mylyn/ui/commit/ActiveTaskBranchNameProvider.java
+++ b/org.eclipse.egit.mylyn.ui/src/org/eclipse/egit/internal/mylyn/ui/commit/ActiveTaskBranchNameProvider.java
@@ -53,8 +53,9 @@
 	}
 
 	private String normalizeBranchName(String name) {
-		String normalized = Repository.normalizeBranchName(name);
-		if (normalized != null && normalized.length() > 30) {
+		String normalized = Repository
+				.normalizeBranchName(name.replaceAll("[#$!]", " ")); //$NON-NLS-1$ //$NON-NLS-2$
+		if (normalized.length() > 30) {
 			normalized = Repository
 					.normalizeBranchName(normalized.substring(0, 30));
 		}