Use export macros instead. To do this, we need to turn WTF into a dll.
Created attachment 203395 [details] patch
Attachment 203395 [details] did not pass style-queue: Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/JavaScriptCore/ChangeLog', u'Source/JavaScriptCore/JavaScriptCore.vcxproj/JavaScriptCore.submit.sln', u'Source/JavaScriptCore/JavaScriptCore.vcxproj/JavaScriptCore.vcxproj.filters', u'Source/JavaScriptCore/JavaScriptCore.vcxproj/JavaScriptCoreCommon.props', u'Source/JavaScriptCore/JavaScriptCore.vcxproj/JavaScriptCoreExportGenerator/JavaScriptCoreExportGenerator.vcxproj', u'Source/JavaScriptCore/JavaScriptCore.vcxproj/JavaScriptCoreExportGenerator/JavaScriptCoreExportGenerator.vcxproj.filters', u'Source/JavaScriptCore/JavaScriptCore.vcxproj/JavaScriptCoreExportGenerator/JavaScriptCoreExportGeneratorBuildCmd.cmd', u'Source/JavaScriptCore/JavaScriptCore.vcxproj/JavaScriptCoreExportGenerator/JavaScriptCoreExportGeneratorCommon.props', u'Source/JavaScriptCore/JavaScriptCore.vcxproj/JavaScriptCoreExportGenerator/JavaScriptCoreExportGeneratorDebug.props', u'Source/JavaScriptCore/JavaScriptCore.vcxproj/JavaScriptCoreExportGenerator/JavaScriptCoreExportGeneratorPostBuild.cmd', u'Source/JavaScriptCore/JavaScriptCore.vcxproj/JavaScriptCoreExportGenerator/JavaScriptCoreExportGeneratorPreBuild.cmd', u'Source/JavaScriptCore/JavaScriptCore.vcxproj/JavaScriptCoreExportGenerator/JavaScriptCoreExportGeneratorProduction.props', u'Source/JavaScriptCore/JavaScriptCore.vcxproj/JavaScriptCoreExportGenerator/JavaScriptCoreExportGeneratorRelease.props', u'Source/JavaScriptCore/JavaScriptCore.vcxproj/JavaScriptCoreExportGenerator/JavaScriptCoreExports.def.in', u'Source/JavaScriptCore/JavaScriptCore.vcxproj/JavaScriptCoreExportGenerator/make-export-file-generator', u'Source/JavaScriptCore/JavaScriptCore.vcxproj/jsc/jscCommon.props', u'Source/JavaScriptCore/JavaScriptCore.vcxproj/testRegExp/testRegExp.vcxproj', u'Source/JavaScriptCore/JavaScriptCore.vcxproj/testRegExp/testRegExp.vcxproj.filters', u'Source/JavaScriptCore/JavaScriptCore.vcxproj/testRegExp/testRegExpCommon.props', u'Source/JavaScriptCore/JavaScriptCore.vcxproj/testapi/testapiCommon.props', u'Source/WTF/ChangeLog', u'Source/WTF/WTF.vcxproj/WTF.vcxproj', u'Source/WTF/WTF.vcxproj/WTFCommon.props', u'Source/WTF/wtf/DateMath.h', u'Source/WTF/wtf/ExportMacros.h', u'Source/WTF/wtf/FilePrintStream.h', u'Source/WTF/wtf/OSAllocator.h', u'Source/WTF/wtf/PageAllocationAligned.h', u'Source/WTF/wtf/Platform.h', u'Source/WTF/wtf/PrintStream.h', u'Source/WTF/wtf/StackBounds.h', u'Source/WTF/wtf/StringPrintStream.h', u'Source/WTF/wtf/ThreadSpecific.h', u'Source/WTF/wtf/WTFThreadData.h', u'Source/WTF/wtf/dtoa/cached-powers.h', u'Source/WTF/wtf/dtoa/double-conversion.h', u'Source/WTF/wtf/text/WTFString.h', u'Source/WTF/wtf/unicode/Collator.h', u'Source/WTF/wtf/unicode/UTF8.h', u'Source/WebKit/ChangeLog', u'Source/WebKit/WebKit.vcxproj/WebKit.sln', u'Source/WebKit/WebKit.vcxproj/WebKit/WebKitCommon.props', u'Tools/ChangeLog', u'Tools/DumpRenderTree/DumpRenderTree.vcxproj/DumpRenderTree/DumpRenderTreeCommon.props', u'Tools/DumpRenderTree/DumpRenderTree.vcxproj/ImageDiff/ImageDiffCommon.props', u'Tools/TestWebKitAPI/TestWebKitAPI.vcxproj/TestWebKitAPICommon.props']" exit_code: 1 Source/WTF/wtf/OSAllocator.h:60: The parameter name "size" adds no information, so it should be removed. [readability/parameter_name] [5] Source/WTF/wtf/unicode/UTF8.h:37: Code inside a namespace should not be indented. [whitespace/indent] [4] Source/WTF/wtf/PageAllocationAligned.h:44: The parameter name "size" adds no information, so it should be removed. [readability/parameter_name] [5] Source/WTF/wtf/PageAllocationAligned.h:44: The parameter name "usage" adds no information, so it should be removed. [readability/parameter_name] [5] Total errors found: 4 in 33 files If any of these errors are false positives, please file a bug against check-webkit-style.
Wow! That is totally awesome!
(In reply to comment #3) > Wow! That is totally awesome! Only for VS2010 tho :/ Hopefully when we get rid of VS2005 project files from the tree (soon!) then we actually won't have to worry about the exports file. :)
committed: http://trac.webkit.org/changeset/150995
<rdar://problem/13685861>
(In reply to comment #5) > committed: http://trac.webkit.org/changeset/150995 FYI: It broke the build on the Apple Windows buildbots.
(In reply to comment #7) > (In reply to comment #5) > > committed: http://trac.webkit.org/changeset/150995 > > FYI: It broke the build on the Apple Windows buildbots. Yup because those are 2005, but not for long.
*** Bug 76257 has been marked as a duplicate of this bug. ***