diff options
| -rw-r--r-- | Makefile | 4 | ||||
| -rw-r--r-- | README | 8 | ||||
| -rw-r--r-- | communication.c | 6 | ||||
| -rw-r--r-- | communication.h | 12 | ||||
| -rw-r--r-- | config.mk | 12 | ||||
| -rw-r--r-- | coopgammad.1 | 10 | ||||
| -rw-r--r-- | coopgammad.c | 6 | ||||
| -rw-r--r-- | servers-coopgamma.c | 4 | ||||
| -rw-r--r-- | servers-crtc.c | 2 | ||||
| -rw-r--r-- | servers-kernel.c | 11 | ||||
| -rw-r--r-- | state.h | 4 | ||||
| -rw-r--r-- | types-ramps.c | 4 | ||||
| -rw-r--r-- | types-ramps.h | 2 | ||||
| -rw-r--r-- | types-ring.c | 2 | ||||
| -rw-r--r-- | types-ring.h | 2 | ||||
| -rw-r--r-- | util.c | 8 | ||||
| -rw-r--r-- | util.h | 4 |
17 files changed, 52 insertions, 49 deletions
@@ -20,12 +20,12 @@ PARTS =\ types-message\ types-ring -OBJ = $(PARTS:=.o) coopgammad.c +OBJ = $(PARTS:=.o) coopgammad.o HDR = $(PARTS:=.h) arg.h all: coopgammad -$(OBJ): $(@:.o=.c) $(HDR) +$(OBJ): $(HDR) .c.o: $(CC) -c -o $@ $< $(XCPPFLAGS) $(CPPFLAGS) $(CFLAGS) @@ -23,7 +23,7 @@ OPTIONS -f Don't fork the process to the background. If used, you can still detect when the - process has been initialised be waiting + process has been initialised by waiting for its stdout to close. -k @@ -43,7 +43,7 @@ OPTIONS The adjustment methods are supported via libgamma(7). Only methods that were enabled - when libgamma as compiled will be supported. + when libgamma was compiled will be supported. -p Add the current gamma adjustments to the @@ -59,10 +59,10 @@ OPTIONS the name of the selected site on the second line to stdout. The second line can be omitted if -s has not been used and the default site - cannot be find. + cannot be found. If used at least twice, print the pathname - of the socket for the select method and site + of the socket for the selected method and site combination to stdout. Under unusual circumstances, the path may contain LF characters, but it will always be terminated diff --git a/communication.c b/communication.c index 13d5046..75a6429 100644 --- a/communication.c +++ b/communication.c @@ -14,7 +14,7 @@ * @param conn The index of the connection * @param buf The data to send * @param n The size of `buf` - * @return Zero on success, -1 on error, 1 if disconncted + * @return Zero on success, -1 on error, 1 if disconnected * EINTR, EAGAIN, EWOULDBLOCK, and ECONNRESET count * as success (ECONNRESET cause 1 to be returned), * and are handled appropriately. @@ -110,7 +110,7 @@ proper_fail: * @param message_id The ID of the message to which this message is a response * @param desc The error description to send * @return 1: Client disconnected - * 0: Success (possibily delayed) + * 0: Success (possibly delayed) * -1: An error occurred */ int @@ -139,7 +139,7 @@ int * @param message_id The ID of the message to which this message is a response * @param number The value of `errno`, 0 to indicate success * @return 1: Client disconnected - * 0: Success (possibily delayed) + * 0: Success (possibly delayed) * -1: An error occurred */ int diff --git a/communication.h b/communication.h index b40022b..e2407b7 100644 --- a/communication.h +++ b/communication.h @@ -38,7 +38,7 @@ * * @param ... The error description to send * @return 1: Client disconnected - * 0: Success (possibily delayed) + * 0: Success (possibly delayed) * -1: An error occurred */ #define send_error(...) ((send_error)(conn, message_id, __VA_ARGS__)) @@ -48,7 +48,7 @@ * * @param ... The value of `errno`, 0 to indicate success * @return 1: Client disconnected - * 0: Success (possibily delayed) + * 0: Success (possibly delayed) * -1: An error occurred */ #define send_errno(...) ((send_errno)(conn, message_id, __VA_ARGS__)) @@ -59,7 +59,7 @@ * @param conn The index of the connection * @param buf The data to send * @param n The size of `buf` - * @return Zero on success, -1 on error, 1 if disconncted + * @return Zero on success, -1 on error, 1 if disconnected * EINTR, EAGAIN, EWOULDBLOCK, and ECONNRESET count * as success (ECONNRESET cause 1 to be returned), * and are handled appropriately. @@ -73,7 +73,7 @@ int send_message(size_t conn, char *restrict buf, size_t n); * @param message_id The ID of the message to which this message is a response * @param desc The error description to send * @return 1: Client disconnected - * 0: Success (possibily delayed) + * 0: Success (possibly delayed) * -1: An error occurred */ GCC_ONLY(__attribute__((__nonnull__))) @@ -86,7 +86,7 @@ int (send_error)(size_t conn, const char *restrict message_id, const char *restr * @param message_id The ID of the message to which this message is a response * @param number The value of `errno`, 0 to indicate success * @return 1: Client disconnected - * 0: Success (possibily delayed) + * 0: Success (possibly delayed) * -1: An error occurred */ GCC_ONLY(__attribute__((__nonnull__))) @@ -96,7 +96,7 @@ int (send_errno)(size_t conn, const char *restrict message_id, int number); * Continue sending the queued messages * * @param conn The index of the connection - * @return Zero on success, -1 on error, 1 if disconncted + * @return Zero on success, -1 on error, 1 if disconnected * EINTR, EAGAIN, EWOULDBLOCK, and ECONNRESET count * as success (ECONNRESET cause 1 to be returned), * and are handled appropriately. @@ -1,12 +1,14 @@ PREFIX = /usr MANPREFIX = $(PREFIX)/share/man -# For class of preserved clut +# For class of preserved CLUT PKGNAME = coopgammad COMMAND = coopgammad -CC=cc +# The makefile uses $(CC) and therefore expects it to be a single command name. +# If you need a specific language mode, set it via CFLAGS. +CC = cc -std=c11 -CPPFLAGS = -D_XOPEN_SOURCE=700 -D_GNU_SOURCE -DUSE_VALGRIND -CFLAGS = -std=c11 -Wall -Og -LDFLAGS = -lgamma -s +CPPFLAGS = -D_XOPEN_SOURCE=700 -D_DEFAULT_SOURCE -D_BSD_SOURCE -D_GNU_SOURCE -DUSE_VALGRIND +CFLAGS = -Wall +LDFLAGS = -lgamma diff --git a/coopgammad.1 b/coopgammad.1 index b8166b5..b20c6f6 100644 --- a/coopgammad.1 +++ b/coopgammad.1 @@ -1,7 +1,7 @@ .TH COOPGAMMAD 1 COOPGAMMAD .SH "NAME" coopgammad - Cooperative gamma server -.SH "SYNPOSIS" +.SH "SYNOPSIS" .B coopgammad .RB [ -m .IR method ] @@ -30,7 +30,7 @@ modified or removed later by another process. .B -f Don't fork the process to the background. If used, you can still detect when the -process has been initialised be waiting +process has been initialised by waiting for its stdout to close. .TP .B -k @@ -55,7 +55,7 @@ The adjustment methods are supported via .BR libgamma (7). Only methods that were enabled when .B libgamma -as compiled will be supported. +was compiled will be supported. .TP .B -p Add the current gamma adjustments to the @@ -73,10 +73,10 @@ line to stdout. The second line can be omitted if .B -s has not been used and the default site cannot -be find. +be found. If used at least twice, print the pathname -of the socket for the select method and site +of the socket for the selected method and site combination to stdout. Under unusual circumstances, the path may contain LF characters, but it will always be terminated diff --git a/coopgammad.c b/coopgammad.c index ecf4619..3f4eb78 100644 --- a/coopgammad.c +++ b/coopgammad.c @@ -37,7 +37,7 @@ /** * Lists all function recognised adjustment methods, - * will call macro X with the code for the each + * will call macro X with the code for each * adjustment method as the first argument and * corresponding name as the second argument */ @@ -166,7 +166,7 @@ GCC_ONLY(__attribute__((__nonnull__))) static int get_method(const char *restrict arg) { -#if LIBGAMMA_METHOD_MAX > 5 +#if LIBGAMMA_METHOD_COUNT > 6 # warning libgamma has added more adjustment methods #endif @@ -278,7 +278,7 @@ daemonise(int keep_stderr) fd = open(pidpath, O_WRONLY); if (fd < 0) goto fail; - if (dprintf(fd, "%llu\n", (unsigned long long)getpid()) < 0) + if (dprintf(fd, "%llu\n", (unsigned long long int)getpid()) < 0) goto fail; close(fd); fd = -1; diff --git a/servers-coopgamma.c b/servers-coopgamma.c index f14c0c4..42cd783 100644 --- a/servers-coopgamma.c +++ b/servers-coopgamma.c @@ -6,13 +6,13 @@ #include "util.h" #include "types-output.h" -#include <libclut.h> - #include <errno.h> #include <stdio.h> #include <stdlib.h> #include <string.h> +#include <libclut.h> + /** * Apply a filter on top of another filter diff --git a/servers-crtc.c b/servers-crtc.c index 24018e1..9171586 100644 --- a/servers-crtc.c +++ b/servers-crtc.c @@ -53,7 +53,7 @@ handle_enumerate_crtcs(size_t conn, const char *restrict message_id) * @return The name of the CRTC, `NULL` on error */ char * -get_crtc_name(const libgamma_crtc_information_t *restrict info, const libgamma_crtc_state_t *restrict crtc) +get_crtc_name(const struct libgamma_crtc_information *restrict info, const struct libgamma_crtc_state *restrict crtc) { char *name; if (!info->edid_error && info->edid) { diff --git a/servers-kernel.c b/servers-kernel.c index fd60aef..a5e3871 100644 --- a/servers-kernel.c +++ b/servers-kernel.c @@ -3,8 +3,6 @@ #include "state.h" #include "util.h" -#include <libgamma.h> - #include <sys/socket.h> #include <sys/stat.h> #include <sys/un.h> @@ -16,6 +14,8 @@ #include <string.h> #include <unistd.h> +#include <libgamma.h> + /** * Get the pathname of the runtime file @@ -29,6 +29,7 @@ get_pathname(const char *restrict suffix) { const char *restrict rundir = getenv("XDG_RUNTIME_DIR"); const char *restrict username = ""; + const char *const_name; char *name = NULL; char *p; char *restrict rc; @@ -39,8 +40,8 @@ get_pathname(const char *restrict suffix) name = memdup(sitename, strlen(sitename) + 1); if (!name) goto fail; - } else if ((name = libgamma_method_default_site(method))) { - name = memdup(name, strlen(name) + 1); + } else if ((const_name = libgamma_method_default_site(method))) { + name = memdup(const_name, strlen(const_name) + 1); if (!name) goto fail; } @@ -176,7 +177,7 @@ retry: goto bad; if ((size_t)(p - content) != n) goto bad; - sprintf(temp, "%llu\n", (unsigned long long)pid); + sprintf(temp, "%llu\n", (unsigned long long int)pid); if (strcmp(content, temp)) goto bad; free(content); @@ -6,11 +6,11 @@ #include "types-ring.h" #include "types-output.h" -#include <libgamma.h> - #include <stddef.h> #include <signal.h> +#include <libgamma.h> + #ifndef GCC_ONLY # if defined(__GNUC__) && !defined(__clang__) # define GCC_ONLY(...) __VA_ARGS__ diff --git a/types-ramps.c b/types-ramps.c index a80f66b..87a3012 100644 --- a/types-ramps.c +++ b/types-ramps.c @@ -1,12 +1,12 @@ /* See LICENSE file for copyright and license details. */ #include "types-ramps.h" -#include <libclut.h> - #include <errno.h> #include <stdlib.h> #include <string.h> +#include <libclut.h> + /** * The name of the process diff --git a/types-ramps.h b/types-ramps.h index 6e64efc..b0b1992 100644 --- a/types-ramps.h +++ b/types-ramps.h @@ -14,7 +14,7 @@ /** * Gamma ramps union for all - * lbigamma gamma ramps types + * libgamma gamma ramps types */ union gamma_ramps { /** diff --git a/types-ring.c b/types-ring.c index 6cceea1..15e52d6 100644 --- a/types-ring.c +++ b/types-ring.c @@ -173,7 +173,7 @@ ring_peek(struct ring *restrict this, size_t *restrict n) /** - * Dequeue data from a ring bubber + * Dequeue data from a ring buffer * * @param this The ring buffer * @param n The number of bytes to dequeue diff --git a/types-ring.h b/types-ring.h index c8b5f77..0a022ee 100644 --- a/types-ring.h +++ b/types-ring.h @@ -94,7 +94,7 @@ GCC_ONLY(__attribute__((__nonnull__))) void *ring_peek(struct ring *restrict this, size_t *restrict n); /** - * Dequeue data from a ring bubber + * Dequeue data from a ring buffer * * @param this The ring buffer * @param n The number of bytes to dequeue @@ -1,8 +1,6 @@ /* See LICENSE file for copyright and license details. */ #include "util.h" -#include <libclut.h> - #include <sys/stat.h> #include <errno.h> #include <fcntl.h> @@ -11,6 +9,8 @@ #include <time.h> #include <unistd.h> +#include <libclut.h> + /** * Duplicate a memory segment @@ -92,7 +92,7 @@ fail: * Not cancelled by `EINTR` * * @param fd The file descriptor - * @param buf The buffer which shall be written to the fail + * @param buf The buffer which shall be written to the file * @param n The size of the buffer * @return The number of written bytes, less than `n` * on error, cannot exceed `n` @@ -119,7 +119,7 @@ nwrite(int fd, const void *restrict buf, size_t n) /** - * Perform a timed suspention of the process. + * Perform a timed suspension of the process. * The process resumes when the timer expires, * or when it is interrupted. * @@ -42,7 +42,7 @@ void *nread(int fd, size_t *restrict n); * Not cancelled by `EINTR` * * @param fd The file descriptor - * @param buf The buffer which shall be written to the fail + * @param buf The buffer which shall be written to the file * @param n The size of the buffer * @return The number of written bytes, less than `n` * on error, cannot exceed `n` @@ -50,7 +50,7 @@ void *nread(int fd, size_t *restrict n); size_t nwrite(int fd, const void *restrict buf, size_t n); /** - * Perform a timed suspention of the process. + * Perform a timed suspension of the process. * The process resumes when the timer expires, * or when it is interrupted. * |