Changeset 52983 in webkit for trunk/JavaScriptCore/runtime

Timestamp:

Jan 8, 2010, 3:32:55 AM (15 years ago)

Author:

[email protected]

Message:

2010-01-08 Kent Hansen <​[email protected]>

Reviewed by Darin Adler.

Don't store RegExp flags string representation
​https://bugs.webkit.org/show_bug.cgi?id=33321

It's unused; the string representation is reconstructed from flags.

• runtime/RegExp.cpp: (JSC::RegExp::RegExp):
• runtime/RegExp.h:

Location:

trunk/JavaScriptCore/runtime

Files:

• RegExp.cpp (modified) (1 diff)
• RegExp.h (modified) (2 diffs)

trunk/JavaScriptCore/runtime/RegExp.cpp

@@ -66 +66 @@
 inline RegExp::RegExp(JSGlobalData* globalData, const UString& pattern, const UString& flags)
     : m_pattern(pattern)
-    , m_flags(flags)
     , m_flagBits(0)
     , m_constructionError(0)

trunk/JavaScriptCore/runtime/RegExp.h

@@ -50 +50 @@
 
         const UString& pattern() const { return m_pattern; }
-        const UString& flags() const { return m_flags; }
 
         bool isValid() const { return !m_constructionError; }
@@ -67 +66 @@
 
         UString m_pattern; // FIXME: Just decompile m_regExp instead of storing this.
-        UString m_flags; // FIXME: Just decompile m_regExp instead of storing this.
         int m_flagBits;
         const char* m_constructionError;