Simplify criteria for doing an optimized test call

src/regexp.js

 // Copyright 2006-2009 the V8 project authors. All rights reserved.
 // Redistribution and use in source and binary forms, with or without
 // modification, are permitted provided that the following conditions are
 // met:
 //
 //     * Redistributions of source code must retain the above copyright
 //       notice, this list of conditions and the following disclaimer.
 //     * Redistributions in binary form must reproduce the above
 //       copyright notice, this list of conditions and the following
 //       disclaimer in the documentation and/or other materials provided
@@ skipping 275 matching lines @@
 
   var lastIndex = this.lastIndex;
   var cache = regExpCache;
   if (%_ObjectEquals(cache.type, 'test') &&
       %_ObjectEquals(cache.regExp, this) &&
       %_ObjectEquals(cache.subject, string) &&
       %_ObjectEquals(cache.lastIndex, lastIndex)) {
     return cache.answer;
   }
 
-  // Remove irrelevant '.*' around a test regexp. The expression  
-  // checks whether this.source starts and ends with '.*' and that the third 
-  // char is not a '?' and that the third to last char is not a '\'. 
+  // Remove irrelevant preceeding '.*' in a test regexp. The expression  
+  // checks whether this.source starts with '.*' and that the third 
+  // char is not a '?'
   if (%_StringCharCodeAt(this.source,0) == 46 && // '.'
       %_StringCharCodeAt(this.source,1) == 42 && // '*' 
-      %_StringCharCodeAt(this.source,2) != 63 && // '?'
-      %_StringCharCodeAt(this.source,this.source.length - 3) != 28 && // '\'
-      %_StringCharCodeAt(this.source,this.source.length - 2) == 46 && // '.'   
-      %_StringCharCodeAt(this.source,this.source.length - 1) == 42) { // '*'
+      %_StringCharCodeAt(this.source,2) != 63) { // '?' 
     if (!%_ObjectEquals(regexp_key, this)) {
       regexp_key = this; 
       regexp_val = new $RegExp(this.source.substring(2,
                                                      this.source.length - 2),
                                (this.global ? 'g' : '')
                                + (this.ignoreCase ? 'i' : '')
                                + (this.multiline ? 'm' : ''));
     }
     if (!regexp_val.test(s)) return false;
   }
@@ skipping 226 matching lines @@
   %DefineAccessor($RegExp, "$'", SETTER, NoOpSetter, DONT_ENUM | DONT_DELETE);
 
   for (var i = 1; i < 10; ++i) {
     %DefineAccessor($RegExp, '$' + i, GETTER, RegExpMakeCaptureGetter(i), DONT_DELETE);
     %DefineAccessor($RegExp, '$' + i, SETTER, NoOpSetter, DONT_DELETE);
   }
 }
 
 
 SetupRegExp();