-Cleaner pseudo selector regex

changelog

@@ -6,7 +6,7 @@
 
 # Compressor Additions
 - Configuration Modes
-- pseduo-space, add space between pseduo selectors and comman/brace for ie6
+- pseduo-space, add space between pseduo selectors and comma/brace for ie6
 - removing leading 0 of a decimal, 0.633px -> .633px
 - handling all At Rules (charsets,media,imports,font-family,etc..)
 - handling ms filter paths
@@ -17,5 +17,6 @@
 
 # Codebase Changes
 - Converted to subclass architecture, more modular.
-- Using JSON based sandbox spec for focused unit tests.
+- Using JSON based sandbox specs for focused unit tests.
 - Helper conversion definitions(long2hex, hex2short) are now in JSON based files.
+- Added benchmark/regression testing

src/lib/Selectors.php

@@ -22,7 +22,8 @@
 	 * @param (regex) rcomma: looks for an unescaped comma character
 	 * @param (regex) rspace: looks for an unescaped space character
 	 * @param (regex) rid: looks for an unescaped hash character
-	 * @param (array) pseudos: Contains pattterns and replacments to space out pseudo selectors
+	 * @param (regex) rpseudo: Add space after first-letter|line pseudo selector
+	 * --- when it occurs before comma or rule set
 	 */
 	private $Control;
 	private $token = '';
@@ -36,18 +37,7 @@
 	private $rcomma = "/(?<!\\\),/";
 	private $rspace = "/(?<!\\\)\s/";
 	private $rid = "/(?<!\\\)#/";
-	private $pseudos = array(
-		'patterns' => array(
-			"/\:first-(letter|line)[,]/i",
-			"/  /",
-			"/:first-(letter|line)$/i",
-		),
-		'replacements' => array(
-			":first-$1 ,",
-			" ",
-			":first-$1 ",
-		)
-	);
+	private $rpseudo = "/:first-(letter|line)(,|$)/i";
 
 	/**
 	 * Stash a reference to the controller on each instantiation
@@ -228,7 +218,7 @@ private function repeats( $selector ) {
 	 * @param (string) selector: CSS Selector
 	 */ 
 	private function pseudoSpace( $selector ) {
-		return preg_replace( $this->pseudos['patterns'], $this->pseudos['replacements'], $selector );
+		return preg_replace( $this->rpseudo, ":first-$1 $2", $selector );
 	}
 
 	/**

unit/sandbox/Selectors.json

@@ -135,6 +135,10 @@
 			"params": [ "a:first-letter,a,b" ],
 			"expect": "a:first-letter ,a,b"
 		},
+		"first-letter comma exists": {
+			"params": [ "a:first-letter ,a,b" ],
+			"expect": "a:first-letter ,a,b"
+		},
 		"first-line tree": {
 			"params": [ "a:first-line>a" ],
 			"expect": "a:first-line>a"