Bug 180579 – [WTF] Remove remaining use of Mutex

• Status: RESOLVED
• Resolution: FIXED
• Component: New Bugs

[Yusuke Suzuki]

[WTF] Remove remaining use of Mutex

[Yusuke Suzuki]

Created attachment 328803 [details]
Patch

[Yusuke Suzuki]

Created attachment 328804 [details]
Patch

[EWS Watchlist]

Attachment 328804 [details] did not pass style-queue:


ERROR: Source/WTF/wtf/threads/BinarySemaphore.cpp:41:  More than one command on the same line  [whitespace/newline] [4]
Total errors found: 1 in 8 files


If any of these errors are false positives, please file a bug against check-webkit-style.

[Yusuke Suzuki]

Created attachment 328809 [details]
Patch

[EWS Watchlist]

Attachment 328809 [details] did not pass style-queue:


ERROR: Source/WTF/wtf/threads/BinarySemaphore.cpp:41:  More than one command on the same line  [whitespace/newline] [4]
Total errors found: 1 in 8 files


If any of these errors are false positives, please file a bug against check-webkit-style.

[Yusuke Suzuki]

WinCairo failure seems related to incremental build issue.

[Alex Christensen]

Comment on attachment 328809 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=328809&action=review

> Source/WTF/wtf/win/WorkQueueWin.cpp:159
> +    HANDLE timer { 0 };

I think this could be nullptr.

[Yusuke Suzuki]

Comment on attachment 328809 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=328809&action=review

Thanks!

>> Source/WTF/wtf/win/WorkQueueWin.cpp:159
>> +    HANDLE timer { 0 };
> 
> I think this could be nullptr.

Nice, fixed.

[Yusuke Suzuki]

Committed r225681: <https://trac.webkit.org/changeset/225681>

[Radar WebKit Bug Importer]

<rdar://problem/35936516>