From 6104ed8199ae5d06cfaf0a9a68f6628be4cd386e Mon Sep 17 00:00:00 2001 From: krizej <60076189+krizej@users.noreply.github.com> Date: Fri, 9 Feb 2024 22:17:20 +0100 Subject: [PATCH] code style --- modules/sep.c | 22 +++++++++++----------- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git a/modules/sep.c b/modules/sep.c index 29df414..b4284d7 100644 --- a/modules/sep.c +++ b/modules/sep.c @@ -18,24 +18,24 @@ uint8_t version_patch(void) { return 0; } const char *module_name(void) { return "sep"; } static size_t length = 8; -static char *sep_char = "-"; +static char *sep_char = (char[]){'-', '\0'}; static size_t sep_char_length = 1; void init(char **config) { - for(char *opt = *config; opt != NULL; opt = *++config) { - if(!strncmp(opt, "sep", 3)) { + for (char *opt = *config; opt != NULL; opt = *++config) { + if (strncmp(opt, "sep", 3) == 0) { char *valstr = opt + 4; // skip 'sep=' long value = strtol(valstr, NULL, 10); - if(value >= 0) { + if (value >= 0) { length = (size_t) value; } - } else if(!strncmp(opt, "char", 4)) { + } else if (strncmp(opt, "char", 4) == 0) { char *value = opt + 5; // skip 'char=' - if(*value != 0) { // todo: see if there are any more characters that should be rejected + if (*value != 0) { // todo: see if there are any more characters that should be rejected sep_char_length = strlen(value); - sep_char = malloc(sep_char_length + 1); + sep_char = malloc((sep_char_length + 1) * sizeof(*sep_char)); strncpy(sep_char, value, sep_char_length); sep_char[sep_char_length] = '\0'; } else { @@ -48,16 +48,16 @@ void init(char **config) { const char *get(void) { char *ret; - if(sep_char == NULL) { - ret = malloc(1); + if (sep_char == NULL) { + ret = malloc(1 * sizeof(*ret)); *ret = 0; return ret; } - ret = malloc(length * sep_char_length + 1); + ret = malloc((length * sep_char_length + 1) * sizeof(*ret)); memset(ret, 0, length * sep_char_length + 1); - for(size_t i = 0; i < length; i++) + for (size_t i = 0; i < length; i++) strncat(ret, sep_char, sep_char_length); ret[length * sep_char_length] = '\0'; // just to be sure :-D