Regexp: Fix rightContext in the lastMatchInfoOverride

src/regexp.js

Index: src/regexp.js
===================================================================
--- src/regexp.js	(revision 11284)
+++ src/regexp.js	(working copy)
@@ -343,14 +343,15 @@
 
 function RegExpGetRightContext() {
   var start_index;
-  var subject;
+  var subject, pattern;

[Michael Starzinger, 2012/04/12 12:10:28]

The "pattern" variable could be moved into the else block.

   if (!lastMatchInfoOverride) {
     start_index = lastMatchInfo[CAPTURE1];
     subject = LAST_SUBJECT(lastMatchInfo);
   } else {
     var override = lastMatchInfoOverride;
     subject = override[override.length - 1];
-    start_index = override[override.length - 2] + subject.length;
+    pattern = override[override.length - 3];

[Lasse Reichstein Nielsen, 2012/04/13 06:59:52]

I don't think this is correct in all cases.

The override has the same format as the parameters to a function argument of
String.prototype.replace. I.e., it's 
[match, captures*, match-position, subject].

So 'pattern' should probably be called 'match', and it should be read as
override[0], not override[override.length - 3], in the cases where there are
captures.

You might want to consider making macros like

 macro OVERRIDE_MATCH(override) = ((override)[0]);
 macro OVERRIDE_POS(override) = ((override)[(override).length - 2]);
 macro OVERRIDE_SUBJECT(override) = ((override)[(override).length - 1]);
 macro OVERRIDE_CAPTURE(override, index) = ((override)[(index) + 1]);

(if for nothing else, it's documentation :)

+    start_index = override[override.length - 2] + pattern.length;
   }
   return SubString(subject, start_index, subject.length);
 }