ansi-codes: slight regex change

\033];3m is considered invalid so we fix the regex and add a test for it Signed-off-by: Pierre Tardy <pierre.tardy@intel.com>
buildbot · Sep 2, 2013 · 8589634 · 8589634
1 parent c451dd5
commit 8589634
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/master/buildbot/test/unit/test_util_ansi_codes.py b/master/buildbot/test/unit/test_util_ansi_codes.py
@@ -37,3 +37,6 @@ def test_ansi_notm(self):
 
     def test_ansi_invalid(self):
         self.runTest("<>foo", ("\033[<>foo", []))
+
+    def test_ansi_invalid_start_by_semicolon(self):
+        self.runTest(";3m", ("\033[;3m", []))
diff --git a/master/buildbot/util/ansicodes.py b/master/buildbot/util/ansicodes.py
@@ -15,7 +15,7 @@
 
 import re
 
-ANSI_RE = re.compile(r"^((\d*)(;\d+)*)([a-zA-Z])")
+ANSI_RE = re.compile(r"^((\d+)(;\d+)*)([a-zA-Z])")
 
 
 def parse_ansi_sgr(ansi_entry):