make AnsiControlSequenceFactory an interface, provide default impleme…

…ntation with DefaultControlSequenceFactory
jbosstools · Mar 1, 2014 · c6212a3 · c6212a3
1 parent 9281e4d
commit c6212a3
Show file tree

Hide file tree

Showing 3 changed files with 16 additions and 4 deletions.
diff --git a/...tools.aesh.core/src/org/jboss/tools/aesh/core/ansi/AbstractAnsiControlSequenceFilter.java b/...tools.aesh.core/src/org/jboss/tools/aesh/core/ansi/AbstractAnsiControlSequenceFilter.java
@@ -1,6 +1,6 @@
 package org.jboss.tools.aesh.core.ansi;
 
-import org.jboss.tools.aesh.core.internal.ansi.AnsiControlSequenceFactory;
+import org.jboss.tools.aesh.core.internal.ansi.DefaultControlSequenceFactory;
 import org.jboss.tools.aesh.core.io.StreamListener;
 
 public abstract class AbstractAnsiControlSequenceFilter implements AnsiControlSequenceFilter {
@@ -39,7 +39,7 @@ private void charAppended(char c) {
 			escapeSequence.append(c);
 		} else if (escapeSequence.length() > 1) {
 			escapeSequence.append(c);
-			AnsiControlSequence command = AnsiControlSequenceFactory.create(escapeSequence.toString());
+			AnsiControlSequence command = DefaultControlSequenceFactory.INSTANCE.create(escapeSequence.toString());
 			if (command != null) {
 				escapeSequence.setLength(0);
 				controlSequenceAvailable(command);

diff --git a/....jboss.tools.aesh.core/src/org/jboss/tools/aesh/core/ansi/AnsiControlSequenceFactory.java b/....jboss.tools.aesh.core/src/org/jboss/tools/aesh/core/ansi/AnsiControlSequenceFactory.java
@@ -0,0 +1,7 @@
+package org.jboss.tools.aesh.core.ansi;
+
+public interface AnsiControlSequenceFactory {
+
+	AnsiControlSequence create(String controlSequence);
+
+}
diff --git a/...rnal/ansi/AnsiControlSequenceFactory.java → ...l/ansi/DefaultControlSequenceFactory.java b/...rnal/ansi/AnsiControlSequenceFactory.java → ...l/ansi/DefaultControlSequenceFactory.java
@@ -1,12 +1,17 @@
 package org.jboss.tools.aesh.core.internal.ansi;
 
 import org.jboss.tools.aesh.core.ansi.AnsiControlSequence;
+import org.jboss.tools.aesh.core.ansi.AnsiControlSequenceFactory;
 
 
 
-public class AnsiControlSequenceFactory {
+public class DefaultControlSequenceFactory implements AnsiControlSequenceFactory {
 
-	public static AnsiControlSequence create(String controlSequence) {
+	public static final DefaultControlSequenceFactory INSTANCE = new DefaultControlSequenceFactory();
+
+	private DefaultControlSequenceFactory() {}
+
+	public AnsiControlSequence create(String controlSequence) {
 		int last = controlSequence.length() - 1;
 		char c = controlSequence.charAt(last);
 		AnsiControlSequenceType type = AnsiControlSequenceType.fromCharacter(c);