summaryrefslogtreecommitdiff
path: root/%252525253fid%252525253d5e76e7e21da042c493c59235ca82d7275f20a7e4%2525253fid%2...
diff options
context:
space:
mode:
authorNRK <nrk@disroot.org>2022-08-06 04:27:13 +0600
committerHiltjo Posthuma <hiltjo@codemadness.org>2022-08-06 16:09:01 +0200
commit5e76e7e21da042c493c59235ca82d7275f20a7e4 (patch)
treeec24d53824887428d186c7d38776f155ebc9b892 /%252525253fid%252525253d5e76e7e21da042c493c59235ca82d7275f20a7e4%2525253fid%2525253d5e76e7e21da042c493c59235ca82d7275f20a7e4%25253fid%25253d5e76e7e21da042c493c59235ca82d7275f20a7e4%253fid%253d5e76e7e21da042c493c59235ca82d7275f20a7e4&id2%253d5b2e5e7a4001479e4dc3e245f96e49f7ea0da658%3fid%3d5e76e7e21da042c493c59235ca82d7275f20a7e4?id=5e76e7e21da042c493c59235ca82d7275f20a7e4
parent5b2e5e7a4001479e4dc3e245f96e49f7ea0da658 (diff)
code-style: simplify some checks
main change here is making the `zoom()` logic saner. the rest of the changes are just small stuff which accumulated on my local branch. pop() must not be called with NULL. and `zoom()` achieves this, but in a very (unnecessarily) complicated way: if c == NULL then nexttiled() will return NULL as well, so we enter this branch: if (c == nexttiled(selmon->clients)) in here the !c check fails and the function returns before calling pop() if (!c || !(c = nexttiled(c->next))) return; however, none of this was needed. we can simply return early if c was NULL. Also `c` is set to `selmon->sel` so we can use `c` in the first check instead which makes things shorter.
Diffstat (limited to '%252525253fid%252525253d5e76e7e21da042c493c59235ca82d7275f20a7e4%2525253fid%2525253d5e76e7e21da042c493c59235ca82d7275f20a7e4%25253fid%25253d5e76e7e21da042c493c59235ca82d7275f20a7e4%253fid%253d5e76e7e21da042c493c59235ca82d7275f20a7e4&id2%253d5b2e5e7a4001479e4dc3e245f96e49f7ea0da658%3fid%3d5e76e7e21da042c493c59235ca82d7275f20a7e4?id=5e76e7e21da042c493c59235ca82d7275f20a7e4')
0 files changed, 0 insertions, 0 deletions