Bug 89491

Summary: ThreadingWin: Silence GCC compiler warnings
Product: WebKit Reporter: Kalev Lember <kalevlember>
Component: Web Template FrameworkAssignee: Nobody <webkit-unassigned>
Severity: Normal CC: aroben, eric, hausmann, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Description Flags
Patch none

Description Kalev Lember 2012-06-19 11:41:02 PDT
ThreadingWin.cpp: In member function 'bool WTF::PlatformCondition::timedWait(WTF::PlatformMutex&, DWORD)':
ThreadingWin.cpp:346:11: warning: variable 'res' set but not used [-Wunused-but-set-variable]
ThreadingWin.cpp: In member function 'void WTF::PlatformCondition::signal(bool)':
ThreadingWin.cpp:395:11: warning: variable 'res' set but not used [-Wunused-but-set-variable]

ThreadingWin.cpp: In function 'WTF::ThreadIdentifier WTF::createThreadInternal(WTF::ThreadFunction, void*, const char*)':
ThreadingWin.cpp:247:25: warning: ignoring return value of 'WTF::OwnPtr<T>::ValueType* WTF::OwnPtr<T>::leakPtr() [with T = WTF::ThreadFunctionInvocation; WTF::OwnPtr<T>::PtrType = WTF::ThreadFunctionInvocation*; WTF::OwnPtr<T>::ValueType = WTF::ThreadFunctionInvocation]', declared with attribute warn_unused_result [-Wunused-result]
Comment 1 Kalev Lember 2012-06-19 11:43:44 PDT
Created attachment 148378 [details]
Comment 2 Kalev Lember 2012-06-28 11:00:56 PDT
Adding a few people to CC who have reviewed Windows threading patches before.

The patch should be just a trivial warning cleanup.
Comment 3 WebKit Review Bot 2012-06-28 12:15:46 PDT
Comment on attachment 148378 [details]

Clearing flags on attachment: 148378

Committed r121448: <http://trac.webkit.org/changeset/121448>
Comment 4 WebKit Review Bot 2012-06-28 12:15:51 PDT
All reviewed patches have been landed.  Closing bug.