Cannot use enum as int without a cast:
[881/3415] Building CXX object Source/...ore.dir/loader/HistoryController.cpp.o
In file included from ../../Source/WTF/wtf/StdLibExtras.h:34:0,
../../Source/WebCore/loader/HistoryController.cpp: In member function ‘void WebCore::HistoryController::goToItem(WebCore::HistoryItem&, WebCore::FrameLoadType)’:
../../Source/WTF/wtf/Assertions.h:407:68: warning: format ‘%d’ expects argument of type ‘int’, but argument 5 has type ‘WebCore::FrameLoadType’ [-Wformat=]
#define LOG(channel, ...) WTFLog(&LOG_CHANNEL(channel), __VA_ARGS__)
../../Source/WebCore/loader/HistoryController.cpp:295:5: note: in expansion of macro ‘LOG’
LOG(History, "HistoryController %p goToItem %p type=%d", this, &targetItem, type);
Created attachment 307539 [details]
Comment on attachment 307539 [details]
View in context: https://bugs.webkit.org/attachment.cgi?id=307539&action=review
I am assuming that -Wall does not include -Wformat and that we do not enable -Wformat on the Mac, iOS, or Windows bots as I have not seen this failure. We should look to enable -Wformat on these bots.
Please explain the change.
(In reply to Daniel Bates from comment #2)
> Comment on attachment 307539 [details]
> View in context:
> I am assuming that -Wall does not include -Wformat
No, -Wall does include -Wformat (at least in GCC).
> and that we do not enable
> -Wformat on the Mac, iOS, or Windows bots as I have not seen this failure.
> We should look to enable -Wformat on these bots.
I don't know about that. Certainly the Apple builds seem to be less-strict about such issues.
> > Source/WebCore/ChangeLog:7
> > +
> Please explain the change.
Aw man. :P
Committed r215580: <http://trac.webkit.org/changeset/215580>