mirrors/ladybird
0
0
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2025-08-24 19:28:48 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 10

Thread: Fix a regression introduced in 80a6df9022

Browse source 
Accidentally forgot to check the state parameter, which made this rather useless.

Bug found, and cause identified by Andreas
This commit is contained in:
Robin Burchell 2019-07-19 16:02:37 +02:00 • committed by Andreas Kling
commit 4547a301c4
Notes: sideshowbarker 2024-07-19 13:08:21 +09:00
1 changed files with 7 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

9
Kernel/Thread.h
View file

@ -312,9 +312,14 @@ template<typename Callback>
inline IterationDecision Thread::for_each_in_state(State state, Callback callback) inline IterationDecision Thread::for_each_in_state(State state, Callback callback)
{ {
ASSERT_INTERRUPTS_DISABLED(); ASSERT_INTERRUPTS_DISABLED();
auto new_callback = [=](Thread& thread) -> IterationDecision {
if (thread.state() == state)
return callback(thread);
return IterationDecision::Continue;
};
if (is_runnable_state(state)) if (is_runnable_state(state))
return for_each_runnable(callback); return for_each_runnable(new_callback);
return for_each_nonrunnable(callback); return for_each_nonrunnable(new_callback);
} }
template<typename Callback> template<typename Callback>