Changeset 38575 in webkit for trunk/JavaScriptCore/runtime/RegExpObject.cpp

Timestamp:

Nov 18, 2008, 1:52:48 PM (17 years ago)

Author:

[email protected]

Message:

2008-11-18 Geoffrey Garen <​[email protected]>

Reviewed by Sam Weinig.

Minor RegExp cleanup.

SunSpider says no change.

• runtime/RegExpObject.cpp: (JSC::RegExpObject::match): Renamed "regExpObj" to "regExpConstructor".

• wrec/WREC.cpp: (JSC::WREC::compileRegExp): Instead of checking for a NULL output vector, ASSERT that the output vector is not NULL. (The rest of WREC is not safe to use with a NULL output vector, and we probably don't want to spend the time and/or performance to make it safe.)

File:

• trunk/JavaScriptCore/runtime/RegExpObject.cpp (modified) (3 diffs)

trunk/JavaScriptCore/runtime/RegExpObject.cpp

@@ -109 +109 @@
 bool RegExpObject::match(ExecState* exec, const ArgList& args)
 {
-    RegExpConstructor* regExpObj = exec->lexicalGlobalObject()->regExpConstructor();
+    RegExpConstructor* regExpConstructor = exec->lexicalGlobalObject()->regExpConstructor();
 
     UString input;
@@ -115 +115 @@
         input = args.at(exec, 0)->toString(exec);
     else {
-        input = regExpObj->input();
+        input = regExpConstructor->input();
         if (input.isNull()) {
             throwError(exec, GeneralError, "No input.");
@@ -134 +134 @@
     int foundIndex;
     int foundLength;
-    regExpObj->performMatch(d->regExp.get(), input, lastIndex, foundIndex, foundLength);
+    regExpConstructor->performMatch(d->regExp.get(), input, lastIndex, foundIndex, foundLength);
 
     if (global) {