kimhemsoe
diff --git a/‎src/Symfony/Component/Routing/Matcher/Dumper/ApacheMatcherDumper.php
Lines changed: 1 addition & 1 deletion b/‎src/Symfony/Component/Routing/Matcher/Dumper/ApacheMatcherDumper.php
Lines changed: 1 addition & 1 deletion
diff --git a/‎src/Symfony/Component/Routing/RouteCompiler.php
Lines changed: 13 additions & 9 deletions b/‎src/Symfony/Component/Routing/RouteCompiler.php
Lines changed: 13 additions & 9 deletions
diff --git a/‎src/Symfony/Component/Routing/RouteCompilerInterface.php
Lines changed: 1 addition & 1 deletion b/‎src/Symfony/Component/Routing/RouteCompilerInterface.php
Lines changed: 1 addition & 1 deletion
@@ -68,7 +68,7 @@ public function dump(array $options = array())
                 }
             }
 
-            $hasTrailingSlash = (!$methods || in_array('HEAD', $methods)) && '/$' == substr($regex, -2) && '^/$' != $regex;
+            $hasTrailingSlash = (!$methods || in_array('HEAD', $methods)) && '/$' === substr($regex, -2) && '^/$' !== $regex;
 
             $variables = array('E=_ROUTING__route:'.$name);
             foreach ($compiledRoute->getVariables() as $i => $variable) {
 
@@ -18,6 +18,8 @@
  */
 class RouteCompiler implements RouteCompilerInterface
 {
+    const REGEX_DELIMITER = '#';
+
     /**
      * Compiles the current route instance.
      *
@@ -47,7 +49,7 @@ public function compile(Route $route)
                 if ($pos !== $len) {
                     $seps[] = $pattern[$pos];
                 }
-                $regexp = sprintf('[^%s]+?', preg_quote(implode('', array_unique($seps)), '#'));
+                $regexp = sprintf('[^%s]+?', preg_quote(implode('', array_unique($seps)), self::REGEX_DELIMITER));
             }
 
             $tokens[] = array('variable', $match[0][0][0], $regexp, $var);
@@ -83,38 +85,40 @@ public function compile(Route $route)
         return new CompiledRoute(
             $route,
             'text' === $tokens[0][0] ? $tokens[0][1] : '',
-            sprintf("#^%s$#s", $regexp),
+            self::REGEX_DELIMITER.'^'.$regexp.'$'.self::REGEX_DELIMITER.'s',
             array_reverse($tokens),
             $variables
         );
     }
 
     /**
-     * Computes the regexp used to match the token.
+     * Computes the regexp used to match a specific token. It can be static text or a subpattern.
      *
      * @param array   $tokens        The route tokens
      * @param integer $index         The index of the current token
      * @param integer $firstOptional The index of the first optional token
      *
-     * @return string The regexp
+     * @return string The regexp pattern for a single token
      */
     private function computeRegexp(array $tokens, $index, $firstOptional)
     {
         $token = $tokens[$index];
         if('text' === $token[0]) {
             // Text tokens
-            return preg_quote($token[1], '#');
+            return preg_quote($token[1], self::REGEX_DELIMITER);
         } else {
             // Variable tokens
             if (0 === $index && 0 === $firstOptional && 1 == count($tokens)) {
                 // When the only token is an optional variable token, the separator is required
-                return sprintf('%s(?<%s>%s)?', preg_quote($token[1], '#'), $token[3], $token[2]);
+                return sprintf('%s(?<%s>%s)?', preg_quote($token[1], self::REGEX_DELIMITER), $token[3], $token[2]);
             } else {
-                $nbTokens = count($tokens);
-                $regexp = sprintf('%s(?<%s>%s)', preg_quote($token[1], '#'), $token[3], $token[2]);
+                $regexp = sprintf('%s(?<%s>%s)', preg_quote($token[1], self::REGEX_DELIMITER), $token[3], $token[2]);
                 if ($index >= $firstOptional) {
-                    // Enclose each optional tokens in a subpattern to make it optional
+                    // Enclose each optional token in a subpattern to make it optional.
+                    // "?:" means it is non-capturing, i.e. the portion of the subject string that
+                    // matched the optional subpattern is not passed back.
                     $regexp = "(?:$regexp";
+                    $nbTokens = count($tokens);
                     if ($nbTokens - 1 == $index) {
                         // Close the optional subpatterns
                         $regexp .= str_repeat(")?", $nbTokens - $firstOptional);
 
@@ -12,7 +12,7 @@
 namespace Symfony\Component\Routing;
 
 /**
- * RouteCompilerInterface is the interface that all RouteCompiler classes must implements.
+ * RouteCompilerInterface is the interface that all RouteCompiler classes must implement.
  *
  * @author Fabien Potencier <fabien@symfony.com>
  */
Original file line number	Diff line number	Diff line change
`@@ -68,7 +68,7 @@ public function dump(array $options = array())`
`68`	`68`	`}`
`69`	`69`	`}`
`70`	`70`
`71`		`- $hasTrailingSlash = (!$methods \|\| in_array('HEAD', $methods)) && '/$' == substr($regex, -2) && '^/$' != $regex;`
	`71`	`+ $hasTrailingSlash = (!$methods \|\| in_array('HEAD', $methods)) && '/$' === substr($regex, -2) && '^/$' !== $regex;`
`72`	`72`
`73`	`73`	`$variables = array('E=_ROUTING__route:'.$name);`
`74`	`74`	`foreach ($compiledRoute->getVariables() as $i => $variable) {`
Original file line number	Diff line number	Diff line change
`@@ -12,7 +12,7 @@`
`12`	`12`	`namespace Symfony\Component\Routing;`
`13`	`13`
`14`	`14`	`/**`
`15`		`- * RouteCompilerInterface is the interface that all RouteCompiler classes must implements.`
	`15`	`+ * RouteCompilerInterface is the interface that all RouteCompiler classes must implement.`
`16`	`16`	`*`
`17`	`17`	`* @author Fabien Potencier <fabien@symfony.com>`
`18`	`18`	`*/`