Build fix.
authorap@webkit.org <ap@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 15 Aug 2008 08:32:01 +0000 (08:32 +0000)
committerap@webkit.org <ap@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 15 Aug 2008 08:32:01 +0000 (08:32 +0000)
        * JavaScriptCore.xcodeproj/project.pbxproj: Marked OpaqueJSString as private

        * kjs/identifier.cpp:
        (KJS::Identifier::checkSameIdentifierTable):
        * kjs/identifier.h:
        (KJS::Identifier::add):
        Since checkSameIdentifierTable is exported for debug build's sake, gcc wants it to be
        non-inline in release builds, too.

        * JavaScriptCore.exp: Don't export inline OpaqueJSString destructor.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@35776 268f45cc-cd09-0410-ab3c-d52691b4dbfc

JavaScriptCore/ChangeLog
JavaScriptCore/JavaScriptCore.exp
JavaScriptCore/JavaScriptCore.xcodeproj/project.pbxproj
JavaScriptCore/kjs/identifier.cpp
JavaScriptCore/kjs/identifier.h

index cd44210..7018915 100644 (file)
@@ -1,5 +1,20 @@
 2008-08-15  Alexey Proskuryakov  <ap@webkit.org>
 
+        Build fix.
+
+        * JavaScriptCore.xcodeproj/project.pbxproj: Marked OpaqueJSString as private
+
+        * kjs/identifier.cpp:
+        (KJS::Identifier::checkSameIdentifierTable):
+        * kjs/identifier.h:
+        (KJS::Identifier::add):
+        Since checkSameIdentifierTable is exported for debug build's sake, gcc wants it to be
+        non-inline in release builds, too.
+
+        * JavaScriptCore.exp: Don't export inline OpaqueJSString destructor.
+
+2008-08-15  Alexey Proskuryakov  <ap@webkit.org>
+
         Reviewed by Geoff Garen.
 
         JSStringRef is created context-free, but can get linked to one via an identifier table,
index 169b0f9..a6384e7 100644 (file)
@@ -76,7 +76,6 @@ _WTFReportFatalError
 __Z12jsRegExpFreeP8JSRegExp
 __Z15jsRegExpCompilePKti24JSRegExpIgnoreCaseOption23JSRegExpMultilineOptionPjPPKc
 __Z15jsRegExpExecutePK8JSRegExpPKtiiPii
-__ZN14OpaqueJSStringD1Ev
 __ZN3KJS10Identifier11addSlowCaseEPNS_12JSGlobalDataEPNS_7UString3RepE
 __ZN3KJS10Identifier11addSlowCaseEPNS_9ExecStateEPNS_7UString3RepE
 __ZN3KJS10Identifier24checkSameIdentifierTableEPNS_12JSGlobalDataEPNS_7UString3RepE
index 58bdf2d..b7d62e2 100644 (file)
                BCD203E80E1718F4002C7E82 /* DatePrototype.lut.h in Headers */ = {isa = PBXBuildFile; fileRef = BCD203E70E1718F4002C7E82 /* DatePrototype.lut.h */; };
                BCF605140E203EF800B9A64D /* ArgList.h in Headers */ = {isa = PBXBuildFile; fileRef = BCF605120E203EF800B9A64D /* ArgList.h */; settings = {ATTRIBUTES = (Private, ); }; };
                C0A272630E50A06300E96E15 /* NotFound.h in Headers */ = {isa = PBXBuildFile; fileRef = C0A2723F0E509F1E00E96E15 /* NotFound.h */; settings = {ATTRIBUTES = (Private, ); }; };
-               E124A8F70E555775003091F1 /* OpaqueJSString.h in Headers */ = {isa = PBXBuildFile; fileRef = E124A8F50E555775003091F1 /* OpaqueJSString.h */; };
+               E124A8F70E555775003091F1 /* OpaqueJSString.h in Headers */ = {isa = PBXBuildFile; fileRef = E124A8F50E555775003091F1 /* OpaqueJSString.h */; settings = {ATTRIBUTES = (Private, ); }; };
                E124A8F80E555775003091F1 /* OpaqueJSString.cpp in Sources */ = {isa = PBXBuildFile; fileRef = E124A8F60E555775003091F1 /* OpaqueJSString.cpp */; };
                E178636D0D9BEEC300D74E75 /* InitializeThreading.cpp in Sources */ = {isa = PBXBuildFile; fileRef = E178636C0D9BEEC300D74E75 /* InitializeThreading.cpp */; };
                E18E3A590DF9278C00D90B34 /* JSGlobalData.cpp in Sources */ = {isa = PBXBuildFile; fileRef = E18E3A570DF9278C00D90B34 /* JSGlobalData.cpp */; };
index ad29486..82e8452 100644 (file)
@@ -233,7 +233,14 @@ void Identifier::checkSameIdentifierTable(JSGlobalData* globalData, UString::Rep
 {
     ASSERT(rep->identifierTable() == globalData->identifierTable);
 }
+#else
+void Identifier::checkSameIdentifierTable(ExecState*, UString::Rep*)
+{
+}
 
+void Identifier::checkSameIdentifierTable(JSGlobalData*, UString::Rep*)
+{
+}
 #endif
 
 } // namespace KJS
index a23a5f9..ba4cfd6 100644 (file)
@@ -92,7 +92,9 @@ namespace KJS {
         static PassRefPtr<UString::Rep> add(ExecState* exec, UString::Rep* r)
         {
             if (r->identifierTable()) {
+#ifndef NDEBUG
                 checkSameIdentifierTable(exec, r);
+#endif
                 return r;
             }
             return addSlowCase(exec, r);
@@ -100,7 +102,9 @@ namespace KJS {
         static PassRefPtr<UString::Rep> add(JSGlobalData* globalData, UString::Rep* r)
         {
             if (r->identifierTable()) {
+#ifndef NDEBUG
                 checkSameIdentifierTable(globalData, r);
+#endif
                 return r;
             }
             return addSlowCase(globalData, r);
@@ -128,11 +132,6 @@ namespace KJS {
         return Identifier::equal(a, b);
     }
 
-#ifdef NDEBUG
-    void UString::checkSameIdentifierTable(ExecState*, UString::Rep*) {}
-    void UString::checkSameIdentifierTable(JSGlobalData*, UString::Rep*) {}
-#endif
-
     IdentifierTable* createIdentifierTable();
     void deleteIdentifierTable(IdentifierTable*);