Changeset 104359 in webkit

Timestamp:

Jan 6, 2012 4:54:54 PM (12 years ago)

Author:

commit-queue@webkit.org

Message:

​https://bugs.webkit.org/show_bug.cgi?id=75296
JSString should not have JS_EXPORTCLASS annotation

Patch by Hajime Morrita <​morrita@chromium.org> on 2012-01-06
Reviewed by Kevin Ollivier.

• runtime/JSString.h: Removed JS_EXPORTCLASS annotation.
• JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCore.def:

Added missing symbols which were hidden by JS_EXPORTCLASS.

Location:

trunk/Source/JavaScriptCore

Files:

• ChangeLog (modified) (1 diff)
• JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCore.def (modified) (2 diffs)
• runtime/JSString.h (modified) (1 diff)

trunk/Source/JavaScriptCore/ChangeLog

@@ +1 @@
+2012-01-06  Hajime Morrita  <morrita@chromium.org>
+
+        https://bugs.webkit.org/show_bug.cgi?id=75296
+        JSString should not have JS_EXPORTCLASS annotation
+
+        Reviewed by Kevin Ollivier.
+
+        * runtime/JSString.h: Removed JS_EXPORTCLASS annotation.
+        * JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCore.def:
+        Added missing symbols which were hidden by JS_EXPORTCLASS.
+
 2012-01-06  Michael Saboff  <msaboff@apple.com>
 

trunk/Source/JavaScriptCore/JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCore.def

@@ -297 +297 @@
     ?s_globalObjectMethodTable@JSGlobalObject@JSC@@1UGlobalObjectMethodTable@2@B
     ?s_info@ExecutableBase@JSC@@2UClassInfo@2@B
+    ?s_info@JSString@JSC@@2UClassInfo@2@B
     ?setConfigurable@PropertyDescriptor@JSC@@QAEX_N@Z
     ?setDescriptor@PropertyDescriptor@JSC@@QAEXVJSValue@2@I@Z
@@ -337 +338 @@
     ?tlsKeyCount@WTF@@YAAAJXZ
     ?tlsKeys@WTF@@YAPAKXZ
+    ?toBoolean@JSString@JSC@@QBE_NPAVExecState@2@@Z
     ?toInt32@JSC@@YAHN@Z
     ?toInteger@JSValue@JSC@@QBENPAVExecState@2@@Z

trunk/Source/JavaScriptCore/runtime/JSString.h

@@ -60 +60 @@
     JSString* jsStringBuilder(JSGlobalData*);
 
-    class JS_EXPORTCLASS JSString : public JSCell {
+    class JSString : public JSCell {
     public:
         friend class JIT;