RESOLVED FIXED 174705
[WTF] Extend ThreadGroup::add results from bool to ThreadGroupAddResult 
https://bugs.webkit.org/show_bug.cgi?id=174705
Summary [WTF] Extend ThreadGroup::add results from bool to ThreadGroupAddResult
Yusuke Suzuki
Reported 2017-07-20 20:56:15 PDT
[WTF] Extend ThreadGroup::add results from bool to ThreadGroupAddResult
Attachments
Patch (8.48 KB, patch)
2017-07-20 21:00 PDT, Yusuke Suzuki 		mark.lam: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Yusuke Suzuki
Comment 1 2017-07-20 21:00:24 PDT
Created attachment 316066 [details] Patch
Mark Lam
Comment 2 2017-07-21 12:21:16 PDT
Comment on attachment 316066 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=316066&action=review r=me > Source/WTF/wtf/ThreadGroup.cpp:53 > + ThreadGroupAddResult result = add(Thread::current()); auto result?
Yusuke Suzuki
Comment 3 2017-07-21 22:13:59 PDT
Comment on attachment 316066 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=316066&action=review Thank you! >> Source/WTF/wtf/ThreadGroup.cpp:53 >> + ThreadGroupAddResult result = add(Thread::current()); > > auto result? Sounds fine. Fixed.
Yusuke Suzuki
Comment 4 2017-07-22 00:27:11 PDT
Committed r219760: <http://trac.webkit.org/changeset/219760>
Note You need to log in before you can comment on or make changes to this bug.