WebCore/WebCore.pro defines WTF_USE_JSC=1, assigned to compiler command line. But WebCore/config.h also defines WTF_USE_JSC: #define WTF_USE_JSC !WTF_USE_V8. The macro redifinig is warning according to gcc. I think it should be fixed. One way to remove defining WTF_USE_JSC from WebCore.pro. The other way to use #undef macro in config.h, if WTF_USE_JSC defined before.
Created attachment 23290 [details]
Undef WTF_USE_JSC before redefining.
Created attachment 23292 [details]
Defining unnecessary WTF_USE_JSC macro removed.
Please pick the patch that you feel is appropriate and set it for review. The two you have provided appear to be mutually exclusive. Please also include a ChangeLog entry.
My bad, you did include ChangeLog entries. Please mark the patch that you feel is the right approach for review.
Comment on attachment 23292 [details]
WTF_USE_JSC=1 defined by http://trac.webkit.org/changeset/36268 because of Qt/Windows build.
(In reply to comment #4)
I marked one of my patyh for review. The other patch is bad as I mentioned in my previous remark.
Comment on attachment 23290 [details]
The way we would normally do this is:
#define WTF_USE_JSC !WTF_USE_V8
This bug fixed by other, landed in r36439 (http://trac.webkit.org/changeset/36439)