Regexp.escape should also escape '(' character

adambeynon · adambeynon · commit 7451fb580479 · 2014-01-08T18:05:56.000Z
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -104,6 +104,8 @@
 
 *   Implicitly require `erb` from stdlib when including erb templates.
 
+*   Fix `Regexp.escape` to also escape '(' character.
+
 ## 0.5.5 2013-11-25
 
 *   Fix regression: add `%i[foo bar]` style words back to lexer
diff --git a/opal/corelib/regexp.rb b/opal/corelib/regexp.rb
@@ -2,7 +2,7 @@ class Regexp
   `def._isRegexp = true`
 
   def self.escape(string)
-    `string.replace(/[\-\[\]\/\{\}\)\)\*\+\?\.\\\^\$| ]/g, '\\$&')`
+    `string.replace(/[\-\[\]\/\{\}\(\)\*\+\?\.\\\^\$| ]/g, '\\$&')`
   end
 
   def self.union(*parts)
