Bug 60705 - 2011-05-12 David Kilzer <ddkilzer@apple.com>
Summary: 2011-05-12 David Kilzer <ddkilzer@apple.com>
Status: RESOLVED DUPLICATE of bug 60710
Alias: None
Product: WebKit
Classification: Unclassified
Component: JavaScriptCore (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: David Kilzer (:ddkilzer)
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-05-12 10:30 PDT by David Kilzer (:ddkilzer)
Modified: 2014-04-24 16:45 PDT (History)
0 users

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description David Kilzer (:ddkilzer) 2011-05-12 10:30:52 PDT
<http://webkit.org/b/00000> JavaScriptGlue should compile with -Wmissing-prototypes

Reviewed by NOBODY (OOPS!).

Enabling -Wmissing-prototypes fixes an Xcode 4 project warning.

This also fixes the following compiler warning:

JavaScriptGlue/JSUtils.cpp:407:15: error: no previous prototype for function 'getThreadGlobalData' [-Werror,-Wmissing-prototypes,3]
JSGlobalData* getThreadGlobalData()
^

* Configurations/Base.xcconfig: Set
GCC_WARN_ABOUT_MISSING_PROTOTYPES = YES.
* JSUtils.cpp:
(getThreadGlobalData): Made static.
Comment 1 David Kilzer (:ddkilzer) 2011-05-12 12:06:02 PDT
*sigh* "webkit-patch upload" fail.  I was trying to make it smarter about reusing a local commit log.  It also likes to create bugs at the drop of a hat.

*** This bug has been marked as a duplicate of bug 60710 ***
Comment 2 Darin Adler 2014-04-24 16:45:22 PDT
Moving all JavaScriptGlue bugs to JavaScriptCore. The JavaScriptGlue framework itself is long gone. And most of the more recent bugs put in this component were put there by people who thought this was for some other aspect of “JavaScript glue” and have nothing to do with the actual original reason for the existence of this component, which was an OS-X-only framework named JavaScriptGlue.