diff options
author | Omar Polo <op@omarpolo.com> | 2021-10-09 18:54:41 +0000 |
---|---|---|
committer | Omar Polo <op@omarpolo.com> | 2021-10-09 18:54:41 +0000 |
commit | 5eb3fc905f5e3bd2f2d586fb1e0ceda879500b3e (patch) | |
tree | 846526cf50242dea26016f337a717dc9ad87f9f3 | |
parent | c6bcc919c658676844c857d59388d4108fcc7b8f (diff) |
don't work around a missing -Wno-unused-parameter
It's been there for a long time, and it's frankly annoying to pretend
to use parameters. Most of the time, they're there to satisfy an
interface and nothings more.
-rw-r--r-- | ex.c | 4 | ||||
-rw-r--r-- | sandbox.c | 3 | ||||
-rw-r--r-- | server.c | 9 |
3 files changed, 0 insertions, 16 deletions
@@ -429,10 +429,6 @@ handle_imsg_quit(struct imsgbuf *ibuf, struct imsg *imsg, size_t datalen) { int i; - (void)ibuf; - (void)imsg; - (void)datalen; - for (i = 0; i < conf.prefork; ++i) { imsg_compose(&servibuf[i], IMSG_QUIT, 0, 0, -1, NULL, 0); imsg_flush(&exibuf); @@ -393,9 +393,6 @@ static struct sock_filter filter[] = { static void sandbox_seccomp_violation(int signum, siginfo_t *info, void *ctx) { - (void)signum; - (void)ctx; - fprintf(stderr, "%s: unexpected system call (arch:0x%x,syscall:%d @ %p)\n", __func__, info->si_arch, info->si_syscall, info->si_call_addr); _exit(1); @@ -1281,8 +1281,6 @@ do_accept(int sock, short et, void *d) socklen_t len; int fd; - (void)et; - saddr = (struct sockaddr*)&addr; len = sizeof(addr); if ((fd = accept(sock, saddr, &len)) == -1) { @@ -1385,9 +1383,6 @@ handle_imsg_fcgi_fd(struct imsgbuf *ibuf, struct imsg *imsg, size_t len) static void handle_imsg_quit(struct imsgbuf *ibuf, struct imsg *imsg, size_t len) { - (void)imsg; - (void)len; - /* * don't call event_loopbreak since we want to finish to * handle the ongoing connections. @@ -1414,10 +1409,6 @@ handle_dispatch_imsg(int fd, short ev, void *d) static void handle_siginfo(int fd, short ev, void *d) { - (void)fd; - (void)ev; - (void)d; - log_info(NULL, "%d connected clients", connected_clients); } |