1
Fork 0
mirror of https://github.com/RGBCube/serenity synced 2025-07-25 14:47:44 +00:00

Kernel: Avoid redundant bool comparisons in Kernel::Thread

Two instances of comparing a bool with == true or == false, and one
instance where we can just return an expression instead of checking it
to return true on succeess and false on failure.
This commit is contained in:
Andrew Kaster 2021-10-31 16:52:43 -06:00 committed by Andreas Kling
parent c9d1c12efa
commit b1d5d3cc34
2 changed files with 4 additions and 6 deletions

View file

@ -792,9 +792,7 @@ bool Thread::should_ignore_signal(u8 signal) const
auto const& action = m_signal_action_data[signal]; auto const& action = m_signal_action_data[signal];
if (action.handler_or_sigaction.is_null()) if (action.handler_or_sigaction.is_null())
return default_signal_action(signal) == DefaultSignalAction::Ignore; return default_signal_action(signal) == DefaultSignalAction::Ignore;
if ((sighandler_t)action.handler_or_sigaction.get() == SIG_IGN) return ((sighandler_t)action.handler_or_sigaction.get() == SIG_IGN);
return true;
return false;
} }
bool Thread::has_signal_handler(u8 signal) const bool Thread::has_signal_handler(u8 signal) const
@ -1068,7 +1066,7 @@ void Thread::set_state(State new_state, u8 stop_signal)
} else if (previous_state == Stopped) { } else if (previous_state == Stopped) {
m_stop_state = State::Invalid; m_stop_state = State::Invalid;
auto& process = this->process(); auto& process = this->process();
if (process.set_stopped(false) == true) { if (process.set_stopped(false)) {
process.for_each_thread([&](auto& thread) { process.for_each_thread([&](auto& thread) {
if (&thread == this) if (&thread == this)
return; return;
@ -1092,7 +1090,7 @@ void Thread::set_state(State new_state, u8 stop_signal)
// We don't want to restore to Running state, only Runnable! // We don't want to restore to Running state, only Runnable!
m_stop_state = previous_state != Running ? previous_state : Runnable; m_stop_state = previous_state != Running ? previous_state : Runnable;
auto& process = this->process(); auto& process = this->process();
if (process.set_stopped(true) == false) { if (!process.set_stopped(true)) {
process.for_each_thread([&](auto& thread) { process.for_each_thread([&](auto& thread) {
if (&thread == this) if (&thread == this)
return; return;

View file

@ -883,7 +883,7 @@ public:
// NOTE: this may execute on the same or any other processor! // NOTE: this may execute on the same or any other processor!
SpinlockLocker scheduler_lock(g_scheduler_lock); SpinlockLocker scheduler_lock(g_scheduler_lock);
SpinlockLocker block_lock(m_block_lock); SpinlockLocker block_lock(m_block_lock);
if (m_blocker && timeout_unblocked.exchange(true) == false) if (m_blocker && !timeout_unblocked.exchange(true))
unblock(); unblock();
}); });
if (!timer_was_added) { if (!timer_was_added) {