diff options
author | jhb <jhb@FreeBSD.org> | 2004-07-16 21:00:50 +0000 |
---|---|---|
committer | jhb <jhb@FreeBSD.org> | 2004-07-16 21:00:50 +0000 |
commit | 4585fb3d5c30fd0c0e9b63baf908f0fad38791f3 (patch) | |
tree | ca4321cee7547ab71c782cced788d4dd0e5e1c65 /sys/kern | |
parent | 34ebc51c75bfcb9efa79473aae83589fa31b1d44 (diff) | |
download | FreeBSD-src-4585fb3d5c30fd0c0e9b63baf908f0fad38791f3.zip FreeBSD-src-4585fb3d5c30fd0c0e9b63baf908f0fad38791f3.tar.gz |
Improve readability a bit by changing some code at the end of a function
that did:
if (foo)
return
else
blah
to just do the simpler
if (!foo)
blah
instead.
Diffstat (limited to 'sys/kern')
-rw-r--r-- | sys/kern/kern_sig.c | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/sys/kern/kern_sig.c b/sys/kern/kern_sig.c index 371e62d..f82b838 100644 --- a/sys/kern/kern_sig.c +++ b/sys/kern/kern_sig.c @@ -219,9 +219,7 @@ signotify(struct thread *td) mtx_unlock_spin(&sched_lock); } if ((p->p_flag & P_SA) && !(p->p_flag & P_SIGEVENT)) { - if (SIGSETEQ(saved, p->p_siglist)) - return; - else { + if (!SIGSETEQ(saved, p->p_siglist)) { /* pending set changed */ p->p_flag |= P_SIGEVENT; wakeup(&p->p_siglist); @@ -1644,9 +1642,7 @@ tdsignal(struct thread *td, int sig, sigtarget_t target) saved = p->p_siglist; do_tdsignal(td, sig, target); if ((p->p_flag & P_SA) && !(p->p_flag & P_SIGEVENT)) { - if (SIGSETEQ(saved, p->p_siglist)) - return; - else { + if (!SIGSETEQ(saved, p->p_siglist)) { /* pending set changed */ p->p_flag |= P_SIGEVENT; wakeup(&p->p_siglist); |