Changeset 149563 in webkit

Timestamp:

May 4, 2013 12:36:34 PM (11 years ago)

Author:

commit-queue@webkit.org

Message:

No need to declare JSC::LLInt::Data as friend class of WTF::StringImpl
​https://bugs.webkit.org/show_bug.cgi?id=115587

Patch by Xan Lopez <​xlopez@igalia.com> on 2013-05-04
Reviewed by Andreas Kling.

No need to declare LLInt::Data as friend class of WTF::StringImpl,
it does not use any private method/data member of the latter.

• wtf/text/StringImpl.h:

(StringImpl):

Location:

trunk/Source/WTF

Files:

• ChangeLog (modified) (1 diff)
• wtf/text/StringImpl.h (modified) (2 diffs)

trunk/Source/WTF/ChangeLog

@@ +1 @@
+2013-05-04  Xan Lopez  <xlopez@igalia.com>
+
+        No need to declare JSC::LLInt::Data as friend class of WTF::StringImpl
+        https://bugs.webkit.org/show_bug.cgi?id=115587
+
+        Reviewed by Andreas Kling.
+
+        No need to declare LLInt::Data as friend class of WTF::StringImpl,
+        it does not use any private method/data member of the latter.
+
+        * wtf/text/StringImpl.h:
+        (StringImpl):
+
 2013-05-03  Brent Fulgham  <bfulgham@webkit.org>
 

trunk/Source/WTF/wtf/text/StringImpl.h

@@ -52 +52 @@
 namespace JSC {
 struct IdentifierASCIIStringTranslator;
-namespace LLInt { class Data; }
 class LLIntOffsetsExtractor;
 template <typename T> struct IdentifierCharBufferTranslator;
@@ -145 +144 @@
     friend struct WTF::UCharBufferTranslator;
     friend class AtomicStringImpl;
-    friend class JSC::LLInt::Data;
     friend class JSC::LLIntOffsetsExtractor;