Bug 149868

Summary: Format string issue in WebResourceLoadScheduler.cpp
Product: WebKit Reporter: Michael Catanzaro <mcatanzaro>
Component: WebKit Misc.Assignee: Michael Catanzaro <mcatanzaro>
Status: RESOLVED FIXED    
Severity: Minor CC: commit-queue, mcatanzaro, ossy
Priority: P2    
Version: Other   
Hardware: PC   
OS: Linux   
Attachments:
Description Flags
Patch none

Description Michael Catanzaro 2015-10-06 20:17:33 PDT
[978/1606] Building CXX object Source/...Network/WebResourceLoadScheduler.cpp.o
In file included from ../../Source/WTF/wtf/StdLibExtras.h:32:0,
                 from ../../Source/WTF/wtf/FastMalloc.h:26,
                 from ../../Source/WebKit2/config.h:44,
                 from ../../Source/WebKit2/WebProcess/Network/WebResourceLoadScheduler.cpp:26:
../../Source/WebKit2/WebProcess/Network/WebResourceLoadScheduler.cpp: In member function ‘void WebKit::WebResourceLoadScheduler::scheduleLoad(WebCore::ResourceLoader*, WebCore::CachedResource*, bool)’:
../../Source/WTF/wtf/Assertions.h:349:105: warning: format ‘%u’ expects argument of type ‘unsigned int’, but argument 4 has type ‘WebCore::ResourceLoadPriority’ [-Wformat=]
 #define LOG(channel, ...) WTFLog(&JOIN_LOG_CHANNEL_WITH_PREFIX(LOG_CHANNEL_PREFIX, channel), __VA_ARGS__)
                                                                                                         ^
../../Source/WebKit2/WebProcess/Network/WebResourceLoadScheduler.cpp:158:5: note: in expansion of macro ‘LOG’
     LOG(NetworkScheduling, "(WebProcess) WebResourceLoadScheduler::scheduleLoad, url '%s' will be scheduled with the NetworkProcess with priority %u", resourceLoader->url().string().utf8().data(), resourceLoader->request().priority());
     ^
Comment 1 Michael Catanzaro 2015-10-06 21:25:39 PDT
Created attachment 262577 [details]
Patch
Comment 2 WebKit Commit Bot 2015-10-08 08:27:38 PDT
Comment on attachment 262577 [details]
Patch

Clearing flags on attachment: 262577

Committed r190724: <http://trac.webkit.org/changeset/190724>
Comment 3 WebKit Commit Bot 2015-10-08 08:27:41 PDT
All reviewed patches have been landed.  Closing bug.