diff options
| author | bakkeby <[email protected]> | 2022-08-07 10:41:01 +0200 |
|---|---|---|
| committer | bakkeby <[email protected]> | 2022-08-07 10:41:01 +0200 |
| commit | 6a0e5b884ef6d27171ada6c2c279a666a7039494 (patch) | |
| tree | eb79c4f30b07d3deb8cb7455be240b181210869e /patch/layout_tile.c | |
| parent | 10aa27171f87b88bfb48747e9d57361ec7bdaacd (diff) | |
| download | dwm-flexipatch-6a0e5b884ef6d27171ada6c2c279a666a7039494.tar.gz dwm-flexipatch-6a0e5b884ef6d27171ada6c2c279a666a7039494.zip | |
Bump to 5e76e7e.
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.
Ref.
https://git.suckless.org/dwm/commit/5e76e7e21da042c493c59235ca82d7275f20a7e4.html
Diffstat (limited to 'patch/layout_tile.c')
0 files changed, 0 insertions, 0 deletions
