aboutsummaryrefslogtreecommitdiffstats
path: root/src/client.c
diff options
context:
space:
mode:
authoruser <user@clank>2026-08-17 14:22:38 +0200
committeruser <user@clank>2026-08-17 14:22:38 +0200
commitec91ccffe631ae34b111bf6161fd0e3c487f684c (patch)
tree4a9fdbb720d9bf03edd5f538d1d87afd2bf51bfb /src/client.c
parentsl0ppty: clang-format, and a hook that runs it on what you staged (diff)
downloadslosh-ec91ccffe631ae34b111bf6161fd0e3c487f684c.tar.gz
slosh-ec91ccffe631ae34b111bf6161fd0e3c487f684c.tar.xz
slosh-ec91ccffe631ae34b111bf6161fd0e3c487f684c.zip
sl0ppty: clang-format the tree
The bumpy one. `make fmt` over every `.c` and `.h` outside `vendor/`: 32 files, 1,843 lines rewritten, no behaviour changed. Nothing here is worth reading -- the next commit adds it to `.git-blame-ignore-revs` so `git blame` walks past it. Two thirds of it is alignment the formatter now owns: trailing comments in a struct, continuation lines under a signature. The rest is the hand-packed tables opening up to one entry per line -- `ACTIONS`, `NAMED_KEYS`, `ACTION_HELP`. That is a real loss of density and a real gain in diffs: adding an action was a reflow of the block and is now one line. Anything worth keeping packed can be fenced with `// clang-format off`; nothing was, on the grounds that a table is the thing a formatter is for. **One change is not formatting.** `src/config.h` had three trailing comments glued onto one field: uint16_t hover_delay_ms; /* how long the pointer must rest to arm a guide */ /* how long an announcement stays up */ /* rows per wheel notch */ -- `toast_ms`'s and `scroll_lines`' comments, landed on their neighbour by an earlier edit, documenting the wrong things ever since. It is in this commit because it is what caught it: clang-format could not settle on that line, wrapping it one way and then re-indenting the wrap on a second pass, so `make fmt` was not idempotent. The comments are back where they belong and it is. Verified after the reformat, not before: `make clean && make all` is warning-free, `make retest` is 1801 checks green, and `make fmt` twice in a row changes nothing the second time. A formatter that alters behaviour would be worse than none, so the gate was the whole suite rather than a reading of the diff.
Diffstat (limited to 'src/client.c')
-rw-r--r--src/client.c26
1 files changed, 15 insertions, 11 deletions
diff --git a/src/client.c b/src/client.c
index 2c22af9..b1155b8 100644
--- a/src/client.c
+++ b/src/client.c
@@ -16,14 +16,14 @@
#include "proto.h"
-#define TERM_ENTER \
- "\x1b[?1049h\x1b[0m\x1b[2J" \
- "\x1b[?1002h\x1b[?1003h\x1b[?1006h" \
- "\x1b[?1004h" \
- "\x1b[?2004h" \
+#define TERM_ENTER \
+ "\x1b[?1049h\x1b[0m\x1b[2J" \
+ "\x1b[?1002h\x1b[?1003h\x1b[?1006h" \
+ "\x1b[?1004h" \
+ "\x1b[?2004h" \
"\x1b[>1u"
-#define TERM_LEAVE \
- "\x1b[<u\x1b[?2004l\x1b[?1004l\x1b[?1006l\x1b[?1003l\x1b[?1002l" \
+#define TERM_LEAVE \
+ "\x1b[<u\x1b[?2004l\x1b[?1004l\x1b[?1006l\x1b[?1003l\x1b[?1002l" \
"\x1b[?25h\x1b[0m\x1b[?1049l"
static volatile sig_atomic_t g_winch = 0, g_quit = 0;
@@ -31,8 +31,10 @@ static struct termios g_saved;
static bool g_saved_ok = false;
static void on_sig(int s) {
- if (s == SIGWINCH) g_winch = 1;
- else g_quit = 1;
+ if (s == SIGWINCH)
+ g_winch = 1;
+ else
+ g_quit = 1;
}
static void restore(void) {
@@ -160,8 +162,10 @@ int client_run(int fd) {
restore();
g_saved_ok = false;
- if (exit_reason == EXIT_DETACHED) fprintf(stderr, "[detached]\n");
- else if (exit_reason == EXIT_REPLACED) fprintf(stderr, "[replaced by another client]\n");
+ if (exit_reason == EXIT_DETACHED)
+ fprintf(stderr, "[detached]\n");
+ else if (exit_reason == EXIT_REPLACED)
+ fprintf(stderr, "[replaced by another client]\n");
return 0;
}