Changeset 52984 in webkit for trunk/JavaScriptCore/runtime

Timestamp:

Jan 8, 2010, 3:44:19 AM (15 years ago)

Author:

[email protected]

Message:

2010-01-08 Eric Seidel <​[email protected]>

No review, rolling out r52981.
​http://trac.webkit.org/changeset/52981
​https://bugs.webkit.org/show_bug.cgi?id=33319

Caused two JS tests to start failing:
ecma_2/RegExp/properties-001.js and js1_2/regexp/toString.js

• runtime/RegExpPrototype.cpp: (JSC::regExpProtoFuncToString):

2010-01-08 Eric Seidel <​[email protected]>

No review, rolling out r52981.
​http://trac.webkit.org/changeset/52981
​https://bugs.webkit.org/show_bug.cgi?id=33319

Caused two JS tests to start failing:
ecma_2/RegExp/properties-001.js and js1_2/regexp/toString.js

• fast/js/kde/RegExp-expected.txt:
• fast/js/kde/script-tests/RegExp.js:
• fast/js/regexp-compile-expected.txt:
• fast/js/script-tests/regexp-compile.js:
• fast/regex/non-pattern-characters-expected.txt:

File:

• trunk/JavaScriptCore/runtime/RegExpPrototype.cpp (modified) (1 diff)

trunk/JavaScriptCore/runtime/RegExpPrototype.cpp

@@ -115 +115 @@
     if (asRegExpObject(thisValue)->get(exec, exec->propertyNames().multiline).toBoolean(exec))
         postfix[index] = 'm';
-    UString source = asRegExpObject(thisValue)->get(exec, exec->propertyNames().source).toString(exec);
-    return jsNontrivialString(exec, makeString("/", source.size() ? source : UString("(?:)"), postfix));
+    
+    return jsNontrivialString(exec, makeString("/", asRegExpObject(thisValue)->get(exec, exec->propertyNames().source).toString(exec), postfix));
 }