Reviewers: Lasse Reichstein,

Description:
Minor regexp test cleanup.

Please review this at http://codereview.chromium.org/4371002/show

SVN Base: http://v8.googlecode.com/svn/branches/bleeding_edge/

Affected files:
  M     src/regexp.js


Index: src/regexp.js
===================================================================
--- src/regexp.js       (revision 5758)
+++ src/regexp.js       (working copy)
@@ -271,13 +271,6 @@
     if (!regexp_val.test(s)) return false;
   }

-  var length = s.length;
-
-  if (i < 0 || i > length) {
-    this.lastIndex = 0;
-    return false;
-  }
-
   %_Log('regexp', 'regexp-exec,%0r,%1S,%2i', [this, s, lastIndex]);
   // matchIndices is either null or the lastMatchInfo array.
   var matchIndices = %_RegExpExec(this, s, i, lastMatchInfo);


--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to