From 0fce5bca6498ce007dfcd7d537c7e7c4d54cf6f6 Mon Sep 17 00:00:00 2001 From: Johannes Löthberg Date: Sun, 18 Jan 2015 04:15:56 +0100 Subject: sds.c: Add two function prototypes is_hex_digit and hex_digit_to_int functions had no function prototypes. Silences two warnings. --- src/sds.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/sds.c b/src/sds.c index f0515f1..f3396c9 100644 --- a/src/sds.c +++ b/src/sds.c @@ -33,6 +33,9 @@ #include "sds.h" +int is_hex_digit(char c); +int hex_digit_to_int(char c); + /* Create a new sds string with the content specified by the 'init' pointer * and 'initlen'. * If NULL is used for 'init' the string is initialized with zero bytes. -- cgit v1.2.3-54-g00ecf From 2bf92c344eb50a72d730eb27aaea208ef5539181 Mon Sep 17 00:00:00 2001 From: Johannes Löthberg Date: Sun, 18 Jan 2015 15:36:39 +0100 Subject: sds.c: Disable format-nonliteral warning around sdscatvprintf --- src/sds.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/sds.c b/src/sds.c index f3396c9..0e62f31 100644 --- a/src/sds.c +++ b/src/sds.c @@ -292,6 +292,7 @@ sds sdscpy(sds s, const char *t) { } /* Like sdscatpritf() but gets va_list instead of being variadic. */ +#pragma GCC diagnostic ignored "-Wformat-nonliteral" sds sdscatvprintf(sds s, const char *fmt, va_list ap) { va_list cpy; char *buf, *t; @@ -314,6 +315,7 @@ sds sdscatvprintf(sds s, const char *fmt, va_list ap) { free(buf); return t; } +#pragma GCC diagnostic warning "-Wformat-nonliteral" /* Append to the sds string 's' a string obtained using printf-alike format * specifier. -- cgit v1.2.3-54-g00ecf From a3d3b21dbdad912fcca74ba2263d007c764b9ea3 Mon Sep 17 00:00:00 2001 From: Johannes Löthberg Date: Sun, 18 Jan 2015 15:38:11 +0100 Subject: sds.h: Change len and free to size_t --- src/sds.h | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/sds.h b/src/sds.h index 489e623..86e2a8c 100644 --- a/src/sds.h +++ b/src/sds.h @@ -31,13 +31,14 @@ #define SDS_MAX_PREALLOC (1024*1024) #include +#include #include typedef char *sds; struct sdshdr { - int len; - int free; + size_t len; + size_t free; char buf[]; }; -- cgit v1.2.3-54-g00ecf From 459d2cb217e1af053f08d7e30a92b1b6bf61a90a Mon Sep 17 00:00:00 2001 From: Johannes Löthberg Date: Sun, 18 Jan 2015 15:41:09 +0100 Subject: sdsrange: change start/end args to ptrdiff_t MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit An sds string can hold a much bigger value than an int can hold, but since size_t is unsigned it can’t be used for this function. Using ptrdiff_t limits the function to work on roughly 1 Exabyte long strings, but if you need to work with longer ones you probably will have your custom string library already. --- src/sds.c | 3 ++- src/sds.h | 2 +- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/src/sds.c b/src/sds.c index 0e62f31..96a6c4c 100644 --- a/src/sds.c +++ b/src/sds.c @@ -27,6 +27,7 @@ #include #include +#include #include #include #include @@ -388,7 +389,7 @@ void sdstrim(sds s, const char *cset) { * s = sdsnew("Hello World"); * sdsrange(s,1,-1); => "ello World" */ -void sdsrange(sds s, int start, int end) { +void sdsrange(sds s, ptrdiff_t start, ptrdiff_t end) { struct sdshdr *sh = sdsheader(s); size_t newlen, len = sdslen(s); diff --git a/src/sds.h b/src/sds.h index 86e2a8c..c0fe90e 100644 --- a/src/sds.h +++ b/src/sds.h @@ -77,7 +77,7 @@ sds sdscatprintf(sds s, const char *fmt, ...); #endif void sdstrim(sds s, const char *cset); -void sdsrange(sds s, int start, int end); +void sdsrange(sds s, ptrdiff_t start, ptrdiff_t end); void sdsupdatelen(sds s); void sdsclear(sds s); int sdscmp(const sds s1, const sds s2); -- cgit v1.2.3-54-g00ecf From 6b0750a000900d2ff775e6792797d64174e509c1 Mon Sep 17 00:00:00 2001 From: Johannes Löthberg Date: Sun, 18 Jan 2015 15:43:51 +0100 Subject: sdsIncrLen: change incr argument to size_t --- src/sds.c | 2 +- src/sds.h | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/sds.c b/src/sds.c index 96a6c4c..e68a401 100644 --- a/src/sds.c +++ b/src/sds.c @@ -200,7 +200,7 @@ size_t sdsAllocSize(sds s) { * ... check for nread <= 0 and handle it ... * sdsIncrLen(s, nread); */ -void sdsIncrLen(sds s, int incr) { +void sdsIncrLen(sds s, size_t incr) { struct sdshdr *sh = sdsheader(s); assert(sh->free >= incr); diff --git a/src/sds.h b/src/sds.h index c0fe90e..a46b06b 100644 --- a/src/sds.h +++ b/src/sds.h @@ -94,7 +94,7 @@ sds sdsjoinsds(sds *argv, int argc, const char *sep, size_t seplen); /* Low level functions exposed to the user API */ sds sdsMakeRoomFor(sds s, size_t addlen); -void sdsIncrLen(sds s, int incr); +void sdsIncrLen(sds s, size_t incr); sds sdsRemoveFreeSpace(sds s); size_t sdsAllocSize(sds s); -- cgit v1.2.3-54-g00ecf From 5079d22319930f47647d1ed9507ad07ec0b922d0 Mon Sep 17 00:00:00 2001 From: Johannes Löthberg Date: Sun, 18 Jan 2015 15:45:20 +0100 Subject: test.c: Make test_list static const, fix oldfree type --- src/test.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/test.c b/src/test.c index 3d71f6f..d2ad304 100644 --- a/src/test.c +++ b/src/test.c @@ -27,7 +27,7 @@ bool sdsIncrLen_content (void); bool sdsIncrLen_len (void); bool sdsIncrLen_free (void); -struct test test_list [] = { +static const struct test test_list [] = { { "create a string and obtain the length", check_string_length }, { "create a string with specified length", create_with_length }, { "string concatenation", string_concat }, @@ -216,7 +216,7 @@ sdsIncrLen_free (void) { _sds_cleanup_ sds x = sdsnew("0"); x = sdsMakeRoomFor(x, 1); struct sdshdr *sh = (void*) (x-(sizeof(struct sdshdr))); - int oldfree = sh->free; + size_t oldfree = sh->free; x[1] = '1'; sdsIncrLen(x, 1); return (sh->free == oldfree-1); -- cgit v1.2.3-54-g00ecf From 021b386ddac3384643d9342683cb448047f83a1a Mon Sep 17 00:00:00 2001 From: Johannes Löthberg Date: Sun, 18 Jan 2015 15:46:39 +0100 Subject: sdsheader: cast pointer through void pointer Because a char* and struct sdshdr pointer have a different alignment you get the following warning when try to cast it directly: warning: cast from 'sds' (aka 'char *') to 'struct sdshdr *' increases required alignment from 1 to 8 [-Wcast-align] To silence the warning we cast it to a void pointer first. --- src/sds.h | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/sds.h b/src/sds.h index a46b06b..d0830fe 100644 --- a/src/sds.h +++ b/src/sds.h @@ -43,7 +43,9 @@ struct sdshdr { }; static inline struct sdshdr * sdsheader(const sds s) { - return s - (sizeof (struct sdshdr)); + /* The sdshdr pointer has a different alignment than the original char + * pointer, so cast it through a void pointer to silence the warning. */ + return (void *)(s - (sizeof (struct sdshdr))); } static inline size_t sdslen(const sds s) { -- cgit v1.2.3-54-g00ecf From 6aa8c1d8976ad3a609f6600966ad48582d024d5e Mon Sep 17 00:00:00 2001 From: Johannes Löthberg Date: Sun, 18 Jan 2015 15:51:30 +0100 Subject: sdssplitlen: Change length arguments to size_t --- src/sds.c | 6 +++--- src/sds.h | 2 +- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/src/sds.c b/src/sds.c index e68a401..aee013f 100644 --- a/src/sds.c +++ b/src/sds.c @@ -472,11 +472,11 @@ int sdscmp(const sds s1, const sds s2) { * requires length arguments. sdssplit() is just the * same function but for zero-terminated strings. */ -sds *sdssplitlen(const char *s, int len, const char *sep, int seplen, int *count) { - int elements = 0, slots = 5, start = 0, j; +sds *sdssplitlen(const char *s, size_t len, const char *sep, size_t seplen, size_t *count) { + size_t elements = 0, slots = 5, start = 0, j; sds *tokens; - if (seplen < 1 || len < 0) { return NULL; } + if (seplen < 1) { return NULL; } tokens = malloc(sizeof(sds)*slots); if (tokens == NULL) { return NULL; } diff --git a/src/sds.h b/src/sds.h index d0830fe..13c94de 100644 --- a/src/sds.h +++ b/src/sds.h @@ -83,7 +83,7 @@ void sdsrange(sds s, ptrdiff_t start, ptrdiff_t end); void sdsupdatelen(sds s); void sdsclear(sds s); int sdscmp(const sds s1, const sds s2); -sds *sdssplitlen(const char *s, int len, const char *sep, int seplen, int *count); +sds *sdssplitlen(const char *s, size_t len, const char *sep, size_t seplen, size_t *count); void sdsfreesplitres(sds *tokens, int count); void sdstolower(sds s); void sdstoupper(sds s); -- cgit v1.2.3-54-g00ecf From f97d7f2eb8c948eb1334036ff0c1e9fbefbc2e06 Mon Sep 17 00:00:00 2001 From: Johannes Löthberg Date: Sun, 18 Jan 2015 15:54:33 +0100 Subject: sds.c: Change spacing around arguments and operators --- src/sds.c | 162 +++++++++++++++++++++++++++++++------------------------------- 1 file changed, 81 insertions(+), 81 deletions(-) diff --git a/src/sds.c b/src/sds.c index aee013f..6b0a633 100644 --- a/src/sds.c +++ b/src/sds.c @@ -44,7 +44,7 @@ int hex_digit_to_int(char c); * The string is always null-termined (all the sds strings are, always) so * even if you create an sds string with: * - * mystring = sdsnewlen("abc",3"); + * mystring = sdsnewlen("abc", 3"); * * You can print the string with printf() as there is an implicit \0 at the * end of the string. However the string is binary safe and can contain @@ -53,9 +53,9 @@ sds sdsnewlen(const void *init, size_t initlen) { struct sdshdr *sh; if (init) { - sh = malloc(sizeof(struct sdshdr)+initlen+1); + sh = malloc(sizeof(struct sdshdr) + initlen + 1); } else { - sh = calloc(sizeof(struct sdshdr)+initlen+1,1); + sh = calloc(sizeof(struct sdshdr) + initlen + 1, 1); } if (sh == NULL) { return NULL; } sh->len = initlen; @@ -70,7 +70,7 @@ sds sdsnewlen(const void *init, size_t initlen) { /* Create an empty (zero length) sds string. Even in this case the string * always has an implicit null term. */ sds sdsempty(void) { - return sdsnewlen("",0); + return sdsnewlen("", 0); } /* Create a new sds string starting from a null termined C string. */ @@ -107,7 +107,7 @@ void sdsfree(sds s) { void sdsupdatelen(sds s) { struct sdshdr *sh = sdsheader(s); int reallen = strlen(s); - sh->free += (sh->len-reallen); + sh->free += (sh->len - reallen); sh->len = reallen; } @@ -136,13 +136,13 @@ sds sdsMakeRoomFor(sds s, size_t addlen) { if (free >= addlen) { return s; } len = sdslen(s); sh = sdsheader(s); - newlen = (len+addlen); + newlen = (len + addlen); if (newlen < SDS_MAX_PREALLOC) { newlen *= 2; } else { newlen += SDS_MAX_PREALLOC; } - newsh = realloc(sh, sizeof(struct sdshdr)+newlen+1); + newsh = realloc(sh, sizeof(struct sdshdr) + newlen + 1); if (newsh == NULL) { return NULL; } newsh->free = newlen - len; @@ -159,7 +159,7 @@ sds sdsRemoveFreeSpace(sds s) { struct sdshdr *sh; sh = sdsheader(s); - sh = realloc(sh, sizeof(struct sdshdr)+sh->len+1); + sh = realloc(sh, sizeof(struct sdshdr) + sh->len + 1); sh->free = 0; return sh->buf; } @@ -174,7 +174,7 @@ sds sdsRemoveFreeSpace(sds s) { size_t sdsAllocSize(sds s) { struct sdshdr *sh = sdsheader(s); - return sizeof(*sh)+sh->len+sh->free+1; + return sizeof(*sh) + sh->len + sh->free + 1; } /* Increment the sds length and decrements the left free space at the @@ -196,7 +196,7 @@ size_t sdsAllocSize(sds s) { * * oldlen = sdslen(s); * s = sdsMakeRoomFor(s, BUFFER_SIZE); - * nread = read(fd, s+oldlen, BUFFER_SIZE); + * nread = read(fd, s + oldlen, BUFFER_SIZE); * ... check for nread <= 0 and handle it ... * sdsIncrLen(s, nread); */ @@ -220,15 +220,15 @@ sds sdsgrowzero(sds s, size_t len) { size_t totlen, curlen = sh->len; if (len <= curlen) { return s; } - s = sdsMakeRoomFor(s,len-curlen); + s = sdsMakeRoomFor(s, len - curlen); if (s == NULL) { return NULL; } /* Make sure added region doesn't contain garbage */ sh = sdsheader(s); - memset(s+curlen,0,(len-curlen+1)); /* also set trailing \0 byte */ - totlen = sh->len+sh->free; + memset(s + curlen, 0, (len - curlen + 1)); /* also set trailing \0 byte */ + totlen = sh->len + sh->free; sh->len = len; - sh->free = totlen-sh->len; + sh->free = totlen - sh->len; return s; } @@ -241,13 +241,13 @@ sds sdscatlen(sds s, const void *t, size_t len) { struct sdshdr *sh; size_t curlen = sdslen(s); - s = sdsMakeRoomFor(s,len); + s = sdsMakeRoomFor(s, len); if (s == NULL) { return NULL; } sh = sdsheader(s); - memcpy(s+curlen, t, len); - sh->len = curlen+len; - sh->free = sh->free-len; - s[curlen+len] = '\0'; + memcpy(s + curlen, t, len); + sh->len = curlen + len; + sh->free = sh->free - len; + s[curlen + len] = '\0'; return s; } @@ -271,18 +271,18 @@ sds sdscatsds(sds s, const sds t) { * safe string pointed by 't' of length 'len' bytes. */ sds sdscpylen(sds s, const char *t, size_t len) { struct sdshdr *sh = sdsheader(s); - size_t totlen = sh->free+sh->len; + size_t totlen = sh->free + sh->len; if (totlen < len) { - s = sdsMakeRoomFor(s,len-sh->len); + s = sdsMakeRoomFor(s, len - sh->len); if (s == NULL) { return NULL; } sh = sdsheader(s); - totlen = sh->free+sh->len; + totlen = sh->free + sh->len; } memcpy(s, t, len); s[len] = '\0'; sh->len = len; - sh->free = totlen-len; + sh->free = totlen - len; return s; } @@ -302,10 +302,10 @@ sds sdscatvprintf(sds s, const char *fmt, va_list ap) { while(1) { buf = malloc(buflen); if (buf == NULL) { return NULL; } - buf[buflen-2] = '\0'; - va_copy(cpy,ap); + buf[buflen - 2] = '\0'; + va_copy(cpy, ap); vsnprintf(buf, buflen, fmt, cpy); - if (buf[buflen-2] != '\0') { + if (buf[buflen - 2] != '\0') { free(buf); buflen *= 2; continue; @@ -327,7 +327,7 @@ sds sdscatvprintf(sds s, const char *fmt, va_list ap) { * Example: * * s = sdsempty("Sum is: "); - * s = sdscatprintf(s,"%d+%d = %d",a,b,a+b). + * s = sdscatprintf(s, "%d + %d = %d", a, b, a + b). * * Often you need to create a string from scratch with the printf-alike * format. When this is the need, just use sdsempty() as the target string: @@ -338,7 +338,7 @@ sds sdscatprintf(sds s, const char *fmt, ...) { va_list ap; char *t; va_start(ap, fmt); - t = sdscatvprintf(s,fmt,ap); + t = sdscatvprintf(s, fmt, ap); va_end(ap); return t; } @@ -352,7 +352,7 @@ sds sdscatprintf(sds s, const char *fmt, ...) { * Example: * * s = sdsnew("AA...AA.a.aa.aHelloWorld :::"); - * s = sdstrim(s,"A. :"); + * s = sdstrim(s, "A. :"); * printf("%s\n", s); * * Output will be just "Hello World". @@ -363,13 +363,13 @@ void sdstrim(sds s, const char *cset) { size_t len; sp = start = s; - ep = end = s+sdslen(s)-1; + ep = end = s + sdslen(s) - 1; while(sp <= end && strchr(cset, *sp)) { sp++; } while(ep > start && strchr(cset, *ep)) { ep--; } - len = (sp > ep) ? 0 : ((ep-sp)+1); + len = (sp > ep) ? 0 : ((ep - sp) + 1); if (sh->buf != sp) { memmove(sh->buf, sp, len); } sh->buf[len] = '\0'; - sh->free = sh->free+(sh->len-len); + sh->free = sh->free + (sh->len - len); sh->len = len; } @@ -387,7 +387,7 @@ void sdstrim(sds s, const char *cset) { * Example: * * s = sdsnew("Hello World"); - * sdsrange(s,1,-1); => "ello World" + * sdsrange(s, 1, -1); => "ello World" */ void sdsrange(sds s, ptrdiff_t start, ptrdiff_t end) { struct sdshdr *sh = sdsheader(s); @@ -395,27 +395,27 @@ void sdsrange(sds s, ptrdiff_t start, ptrdiff_t end) { if (len == 0) { return; } if (start < 0) { - start = len+start; + start = len + start; if (start < 0) { start = 0; } } if (end < 0) { - end = len+end; + end = len + end; if (end < 0) { end = 0; } } - newlen = (start > end) ? 0 : (end-start)+1; + newlen = (start > end) ? 0 : (end - start) + 1; if (newlen != 0) { if (start >= (signed)len) { newlen = 0; } else if (end >= (signed)len) { - end = len-1; - newlen = (start > end) ? 0 : (end-start)+1; + end = len - 1; + newlen = (start > end) ? 0 : (end - start) + 1; } } else { start = 0; } - if (start && newlen) { memmove(sh->buf, sh->buf+start, newlen); } + if (start && newlen) { memmove(sh->buf, sh->buf + start, newlen); } sh->buf[newlen] = 0; - sh->free = sh->free+(sh->len-newlen); + sh->free = sh->free + (sh->len - newlen); sh->len = newlen; } @@ -451,8 +451,8 @@ int sdscmp(const sds s1, const sds s2) { l1 = sdslen(s1); l2 = sdslen(s2); minlen = (l1 < l2) ? l1 : l2; - cmp = memcmp(s1,s2,minlen); - if (cmp == 0) { return l1-l2; } + cmp = memcmp(s1, s2, minlen); + if (cmp == 0) { return (l1 - l2); } return cmp; } @@ -465,7 +465,7 @@ int sdscmp(const sds s1, const sds s2) { * * Note that 'sep' is able to split a string using * a multi-character separator. For example - * sdssplit("foo_-_bar","_-_"); will return two + * sdssplit("foo_-_bar", "_-_"); will return two * elements "foo" and "bar". * * This version of the function is binary-safe but @@ -485,23 +485,23 @@ sds *sdssplitlen(const char *s, size_t len, const char *sep, size_t seplen, size *count = 0; return tokens; } - for (j = 0; j < (len-(seplen-1)); j++) { + for (j = 0; j < (len - (seplen - 1)); j++) { /* make sure there is room for the next element and the final one */ - if (slots < elements+2) { + if (slots < elements + 2) { sds *newtokens; slots *= 2; - newtokens = realloc(tokens,sizeof(sds)*slots); + newtokens = realloc(tokens, sizeof(sds) * slots); if (newtokens == NULL) { goto cleanup; } tokens = newtokens; } /* search the separator */ - if ((seplen == 1 && *(s+j) == sep[0]) || (memcmp(s+j,sep,seplen) == 0)) { - tokens[elements] = sdsnewlen(s+start,j-start); + if ((seplen == 1 && *(s + j) == sep[0]) || (memcmp(s + j, sep, seplen) == 0)) { + tokens[elements] = sdsnewlen(s + start, j - start); if (tokens[elements] == NULL) { goto cleanup; } elements++; - start = j+seplen; - j = j+seplen-1; /* skip the separator */ + start = j + seplen; + j = j + seplen - 1; /* skip the separator */ } } /* Add the final element. We are sure there is room in the tokens array. */ @@ -532,16 +532,16 @@ void sdsfreesplitres(sds *tokens, int count) { /* Create an sds string from a long long value. It is much faster than: * - * sdscatprintf(sdsempty(),"%lld\n", value); + * sdscatprintf(sdsempty(), "%lld\n", value); */ sds sdsfromlonglong(long long value) { char buf[32], *p; unsigned long long v; v = (unsigned long long)((value < 0) ? -value : value); - p = buf+31; /* point to the last character */ + p = buf + 31; /* point to the last character */ do { - *p-- = '0'+(v%10); + *p-- = '0' + (v%10); v /= 10; } while(v); if (value < 0) { *p-- = '-'; } @@ -556,29 +556,29 @@ sds sdsfromlonglong(long long value) { * After the call, the modified sds string is no longer valid and all the * references must be substituted with the new pointer returned by the call. */ sds sdscatrepr(sds s, const char *p, size_t len) { - s = sdscatlen(s,"\"",1); + s = sdscatlen(s, "\"", 1); while(len--) { switch(*p) { case '\\': case '"': - s = sdscatprintf(s,"\\%c",*p); + s = sdscatprintf(s, "\\%c", *p); break; - case '\n': s = sdscatlen(s,"\\n",2); break; - case '\r': s = sdscatlen(s,"\\r",2); break; - case '\t': s = sdscatlen(s,"\\t",2); break; - case '\a': s = sdscatlen(s,"\\a",2); break; - case '\b': s = sdscatlen(s,"\\b",2); break; + case '\n': s = sdscatlen(s, "\\n", 2); break; + case '\r': s = sdscatlen(s, "\\r", 2); break; + case '\t': s = sdscatlen(s, "\\t", 2); break; + case '\a': s = sdscatlen(s, "\\a", 2); break; + case '\b': s = sdscatlen(s, "\\b", 2); break; default: if (isprint(*p)) { - s = sdscatprintf(s,"%c",*p); + s = sdscatprintf(s, "%c", *p); } else { - s = sdscatprintf(s,"\\x%02x",(unsigned char)*p); + s = sdscatprintf(s, "\\x%02x", (unsigned char)*p); break; } } p++; } - return sdscatlen(s,"\"",1); + return sdscatlen(s, "\"", 1); } /* Helper function for sdssplitargs() that returns non zero if 'c' @@ -649,17 +649,17 @@ sds *sdssplitargs(const char *line, int *argc) { if (current == NULL) { current = sdsempty(); } while(!done) { if (inq) { - if (*p == '\\' && *(p+1) == 'x' && - is_hex_digit(*(p+2)) && - is_hex_digit(*(p+3))) + if (*p == '\\' && *(p + 1) == 'x' && + is_hex_digit(*(p + 2)) && + is_hex_digit(*(p + 3))) { unsigned char byte; - byte = (hex_digit_to_int(*(p+2))*16)+ - hex_digit_to_int(*(p+3)); - current = sdscatlen(current,(char*)&byte,1); + byte = ((hex_digit_to_int(*(p + 2))*16) + + hex_digit_to_int(*(p + 3))); + current = sdscatlen(current, (char*)&byte, 1); p += 3; - } else if (*p == '\\' && *(p+1)) { + } else if (*p == '\\' && *(p + 1)) { char c; p++; @@ -671,32 +671,32 @@ sds *sdssplitargs(const char *line, int *argc) { case 'a': c = '\a'; break; default: c = *p; break; } - current = sdscatlen(current,&c,1); + current = sdscatlen(current, &c, 1); } else if (*p == '"') { /* closing quote must be followed by a space or * nothing at all. */ - if (*(p+1) && !isspace(*(p+1))) { goto err; } + if (*(p + 1) && !isspace(*(p + 1))) { goto err; } done=1; } else if (!*p) { /* unterminated quotes */ goto err; } else { - current = sdscatlen(current,p,1); + current = sdscatlen(current, p, 1); } } else if (insq) { - if (*p == '\\' && *(p+1) == '\'') { + if (*p == '\\' && *(p + 1) == '\'') { p++; - current = sdscatlen(current,"'",1); + current = sdscatlen(current, "'", 1); } else if (*p == '\'') { /* closing quote must be followed by a space or * nothing at all. */ - if (*(p+1) && !isspace(*(p+1))) { goto err; } + if (*(p + 1) && !isspace(*(p + 1))) { goto err; } done=1; } else if (!*p) { /* unterminated quotes */ goto err; } else { - current = sdscatlen(current,p,1); + current = sdscatlen(current, p, 1); } } else { switch(*p) { @@ -714,14 +714,14 @@ sds *sdssplitargs(const char *line, int *argc) { insq=1; break; default: - current = sdscatlen(current,p,1); + current = sdscatlen(current, p, 1); break; } } if (*p) { p++; } } /* add the token to the vector */ - vector = realloc(vector,((*argc)+1)*sizeof(char*)); + vector = realloc(vector, ((*argc) + 1) * (sizeof (char *))); vector[*argc] = current; (*argc)++; current = NULL; @@ -773,7 +773,7 @@ sds sdsjoin(char **argv, int argc, char *sep, size_t seplen) { for (j = 0; j < argc; j++) { join = sdscat(join, argv[j]); - if (j != argc-1) { join = sdscatlen(join,sep,seplen); } + if (j != argc - 1) { join = sdscatlen(join, sep, seplen); } } return join; } @@ -785,7 +785,7 @@ sds sdsjoinsds(sds *argv, int argc, const char *sep, size_t seplen) { for (j = 0; j < argc; j++) { join = sdscatsds(join, argv[j]); - if (j != argc-1) { join = sdscatlen(join,sep,seplen); } + if (j != argc - 1) { join = sdscatlen(join, sep, seplen); } } return join; } -- cgit v1.2.3-54-g00ecf From dd96070610efd6fd6bf887f7680b76dda37a5c1a Mon Sep 17 00:00:00 2001 From: Johannes Löthberg Date: Sun, 18 Jan 2015 16:10:03 +0100 Subject: sds.c: Change remaining len vars to size_t --- src/sds.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/sds.c b/src/sds.c index 6b0a633..b7bf98f 100644 --- a/src/sds.c +++ b/src/sds.c @@ -106,7 +106,7 @@ void sdsfree(sds s) { * remains 6 bytes. */ void sdsupdatelen(sds s) { struct sdshdr *sh = sdsheader(s); - int reallen = strlen(s); + size_t reallen = strlen(s); sh->free += (sh->len - reallen); sh->len = reallen; } @@ -421,14 +421,14 @@ void sdsrange(sds s, ptrdiff_t start, ptrdiff_t end) { /* Apply tolower() to every character of the sds string 's'. */ void sdstolower(sds s) { - int len = sdslen(s), j; + size_t len = sdslen(s), j; for (j = 0; j < len; j++) s[j] = tolower(s[j]); } /* Apply toupper() to every character of the sds string 's'. */ void sdstoupper(sds s) { - int len = sdslen(s), j; + size_t len = sdslen(s), j; for (j = 0; j < len; j++) s[j] = toupper(s[j]); } @@ -513,7 +513,7 @@ sds *sdssplitlen(const char *s, size_t len, const char *sep, size_t seplen, size cleanup: { - int i; + size_t i; for (i = 0; i < elements; i++) sdsfree(tokens[i]); free(tokens); *count = 0; -- cgit v1.2.3-54-g00ecf From d2a12b1e97815ffebcfd1bc5355e8c374ab32adb Mon Sep 17 00:00:00 2001 From: Johannes Löthberg Date: Sun, 18 Jan 2015 16:10:55 +0100 Subject: sds.c: Add explicit casts everywhere Silences a /lot/ of warnings. --- src/sds.c | 34 +++++++++++++++++----------------- 1 file changed, 17 insertions(+), 17 deletions(-) diff --git a/src/sds.c b/src/sds.c index b7bf98f..13e4634 100644 --- a/src/sds.c +++ b/src/sds.c @@ -366,7 +366,7 @@ void sdstrim(sds s, const char *cset) { ep = end = s + sdslen(s) - 1; while(sp <= end && strchr(cset, *sp)) { sp++; } while(ep > start && strchr(cset, *ep)) { ep--; } - len = (sp > ep) ? 0 : ((ep - sp) + 1); + len = (size_t)((sp > ep) ? 0 : ((ep - sp) + 1)); if (sh->buf != sp) { memmove(sh->buf, sp, len); } sh->buf[len] = '\0'; sh->free = sh->free + (sh->len - len); @@ -395,20 +395,20 @@ void sdsrange(sds s, ptrdiff_t start, ptrdiff_t end) { if (len == 0) { return; } if (start < 0) { - start = len + start; + start = (ptrdiff_t)len + start; if (start < 0) { start = 0; } } if (end < 0) { - end = len + end; + end = (ptrdiff_t)len + end; if (end < 0) { end = 0; } } - newlen = (start > end) ? 0 : (end - start) + 1; + newlen = (size_t)((start > end) ? 0 : (end - start) + 1); if (newlen != 0) { - if (start >= (signed)len) { + if ((size_t)start >= len) { newlen = 0; - } else if (end >= (signed)len) { - end = len - 1; - newlen = (start > end) ? 0 : (end - start) + 1; + } else if ((size_t)end >= len) { + end = (ptrdiff_t)len - 1; + newlen = (size_t)((start > end) ? 0 : (end - start) + 1); } } else { start = 0; @@ -423,14 +423,14 @@ void sdsrange(sds s, ptrdiff_t start, ptrdiff_t end) { void sdstolower(sds s) { size_t len = sdslen(s), j; - for (j = 0; j < len; j++) s[j] = tolower(s[j]); + for (j = 0; j < len; j++) s[j] = (char)tolower(s[j]); } /* Apply toupper() to every character of the sds string 's'. */ void sdstoupper(sds s) { size_t len = sdslen(s), j; - for (j = 0; j < len; j++) s[j] = toupper(s[j]); + for (j = 0; j < len; j++) s[j] = (char)toupper(s[j]); } /* Compare two sds strings s1 and s2 with memcmp(). @@ -452,7 +452,7 @@ int sdscmp(const sds s1, const sds s2) { l2 = sdslen(s2); minlen = (l1 < l2) ? l1 : l2; cmp = memcmp(s1, s2, minlen); - if (cmp == 0) { return (l1 - l2); } + if (cmp == 0) { return (int)(l1 - l2); } return cmp; } @@ -497,7 +497,7 @@ sds *sdssplitlen(const char *s, size_t len, const char *sep, size_t seplen, size } /* search the separator */ if ((seplen == 1 && *(s + j) == sep[0]) || (memcmp(s + j, sep, seplen) == 0)) { - tokens[elements] = sdsnewlen(s + start, j - start); + tokens[elements] = sdsnewlen(s + start, (size_t)(j - start)); if (tokens[elements] == NULL) { goto cleanup; } elements++; start = j + seplen; @@ -505,7 +505,7 @@ sds *sdssplitlen(const char *s, size_t len, const char *sep, size_t seplen, size } } /* Add the final element. We are sure there is room in the tokens array. */ - tokens[elements] = sdsnewlen(s+start,len-start); + tokens[elements] = sdsnewlen(s + start, (size_t)(len - start)); if (tokens[elements] == NULL) { goto cleanup; } elements++; *count = elements; @@ -546,7 +546,7 @@ sds sdsfromlonglong(long long value) { } while(v); if (value < 0) { *p-- = '-'; } p++; - return sdsnewlen(p,32-(p-buf)); + return sdsnewlen(p, (size_t)(32 - (p - buf))); } /* Append to the sds string "s" an escaped string representation where @@ -655,8 +655,8 @@ sds *sdssplitargs(const char *line, int *argc) { { unsigned char byte; - byte = ((hex_digit_to_int(*(p + 2))*16) + - hex_digit_to_int(*(p + 3))); + byte = (unsigned char)((hex_digit_to_int(*(p + 2)) * 16) + + hex_digit_to_int(*(p + 3))); current = sdscatlen(current, (char*)&byte, 1); p += 3; } else if (*p == '\\' && *(p + 1)) { @@ -721,7 +721,7 @@ sds *sdssplitargs(const char *line, int *argc) { if (*p) { p++; } } /* add the token to the vector */ - vector = realloc(vector, ((*argc) + 1) * (sizeof (char *))); + vector = realloc(vector, (unsigned long)((*argc) + 1) * (sizeof (char *))); vector[*argc] = current; (*argc)++; current = NULL; -- cgit v1.2.3-54-g00ecf From d7d703e33aa65a4818a05bc678fe62750390ed67 Mon Sep 17 00:00:00 2001 From: Johannes Löthberg Date: Sun, 18 Jan 2015 17:36:39 +0100 Subject: Move variables into for loop, remove unnevessary lens --- src/sds.c | 31 +++++++++++++------------------ 1 file changed, 13 insertions(+), 18 deletions(-) diff --git a/src/sds.c b/src/sds.c index 13e4634..0179e60 100644 --- a/src/sds.c +++ b/src/sds.c @@ -421,16 +421,16 @@ void sdsrange(sds s, ptrdiff_t start, ptrdiff_t end) { /* Apply tolower() to every character of the sds string 's'. */ void sdstolower(sds s) { - size_t len = sdslen(s), j; - - for (j = 0; j < len; j++) s[j] = (char)tolower(s[j]); + for (size_t j = 0; j < sdslen(s); j++) { + s[j] = (char)tolower(s[j]); + } } /* Apply toupper() to every character of the sds string 's'. */ void sdstoupper(sds s) { - size_t len = sdslen(s), j; - - for (j = 0; j < len; j++) s[j] = (char)toupper(s[j]); + for (size_t j = 0; j < sdslen(s); j++) { + s[j] = (char)toupper(s[j]); + } } /* Compare two sds strings s1 and s2 with memcmp(). @@ -473,7 +473,7 @@ int sdscmp(const sds s1, const sds s2) { * same function but for zero-terminated strings. */ sds *sdssplitlen(const char *s, size_t len, const char *sep, size_t seplen, size_t *count) { - size_t elements = 0, slots = 5, start = 0, j; + size_t elements = 0, slots = 5, start = 0; sds *tokens; if (seplen < 1) { return NULL; } @@ -485,7 +485,7 @@ sds *sdssplitlen(const char *s, size_t len, const char *sep, size_t seplen, size *count = 0; return tokens; } - for (j = 0; j < (len - (seplen - 1)); j++) { + for (size_t j = 0; j < (len - (seplen - 1)); j++) { /* make sure there is room for the next element and the final one */ if (slots < elements + 2) { sds *newtokens; @@ -513,8 +513,7 @@ sds *sdssplitlen(const char *s, size_t len, const char *sep, size_t seplen, size cleanup: { - size_t i; - for (i = 0; i < elements; i++) sdsfree(tokens[i]); + for (size_t i = 0; i < elements; i++) sdsfree(tokens[i]); free(tokens); *count = 0; return NULL; @@ -752,10 +751,8 @@ err: * The function returns the sds string pointer, that is always the same * as the input pointer since no resize is needed. */ sds sdsmapchars(sds s, const char *from, const char *to, size_t setlen) { - size_t j, i, l = sdslen(s); - - for (j = 0; j < l; j++) { - for (i = 0; i < setlen; i++) { + for (size_t j = 0; j < sdslen(s); j++) { + for (size_t i = 0; i < setlen; i++) { if (s[j] == from[i]) { s[j] = to[i]; break; @@ -769,9 +766,8 @@ sds sdsmapchars(sds s, const char *from, const char *to, size_t setlen) { * Returns the result as an sds string. */ sds sdsjoin(char **argv, int argc, char *sep, size_t seplen) { sds join = sdsempty(); - int j; - for (j = 0; j < argc; j++) { + for (int j = 0; j < argc; j++) { join = sdscat(join, argv[j]); if (j != argc - 1) { join = sdscatlen(join, sep, seplen); } } @@ -781,9 +777,8 @@ sds sdsjoin(char **argv, int argc, char *sep, size_t seplen) { /* Like sdsjoin, but joins an array of SDS strings. */ sds sdsjoinsds(sds *argv, int argc, const char *sep, size_t seplen) { sds join = sdsempty(); - int j; - for (j = 0; j < argc; j++) { + for (int j = 0; j < argc; j++) { join = sdscatsds(join, argv[j]); if (j != argc - 1) { join = sdscatlen(join, sep, seplen); } } -- cgit v1.2.3-54-g00ecf From 5a0205be790dbe6402c36ae36151b8688522e2f0 Mon Sep 17 00:00:00 2001 From: Johannes Löthberg Date: Sun, 18 Jan 2015 18:42:56 +0100 Subject: sdsfreesplitres: Change count arg to size_t This makes it match sdssplitlen --- src/sds.c | 2 +- src/sds.h | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/sds.c b/src/sds.c index 0179e60..b0e596e 100644 --- a/src/sds.c +++ b/src/sds.c @@ -521,7 +521,7 @@ cleanup: } /* Free the result returned by sdssplitlen(), or do nothing if 'tokens' is NULL. */ -void sdsfreesplitres(sds *tokens, int count) { +void sdsfreesplitres(sds *tokens, size_t count) { if (!tokens) { return; } while(count--) { sdsfree(tokens[count]); diff --git a/src/sds.h b/src/sds.h index 13c94de..3354096 100644 --- a/src/sds.h +++ b/src/sds.h @@ -84,7 +84,7 @@ void sdsupdatelen(sds s); void sdsclear(sds s); int sdscmp(const sds s1, const sds s2); sds *sdssplitlen(const char *s, size_t len, const char *sep, size_t seplen, size_t *count); -void sdsfreesplitres(sds *tokens, int count); +void sdsfreesplitres(sds *tokens, size_t count); void sdstolower(sds s); void sdstoupper(sds s); sds sdsfromlonglong(long long value); -- cgit v1.2.3-54-g00ecf From 8d3aabccc040414cfddd0bdfcc1fa3cbe741c784 Mon Sep 17 00:00:00 2001 From: Johannes Löthberg Date: Sun, 18 Jan 2015 18:43:38 +0100 Subject: README: Bring up to date with type changes Also change the printf calls to use the proper `%zu` type for size_t instead of casting to int, and add missing spaces after commas. --- README.md | 93 +++++++++++++++++++++++++++++++-------------------------------- 1 file changed, 46 insertions(+), 47 deletions(-) diff --git a/README.md b/README.md index a99c5ea..8d2db6b 100644 --- a/README.md +++ b/README.md @@ -67,7 +67,7 @@ one: API calls look something like this: ```c - s = sdscat(s,"Some more data"); + s = sdscat(s, "Some more data"); ``` As you can see `s` is used as input for `sdscat` but is also set to the @@ -177,20 +177,17 @@ There are many ways to create SDS strings: buf[0] = 'A'; buf[1] = 'B'; buf[2] = 'C'; - mystring = sdsnewlen(buf,3); - printf("%s of len %d\n", mystring, (int) sdslen(mystring)); + mystring = sdsnewlen(buf, 3); + printf("%s of len %zu\n", mystring, sdslen(mystring)); output> ABC of len 3 ``` - Note: `sdslen` return value is casted to `int` because it returns a `size_t` -type. You can use the right `printf` specifier instead of casting. - * The `sdsempty()` function creates an empty zero-length string: ```c sds mystring = sdsempty(); - printf("%d\n", (int) sdslen(mystring)); + printf("%zu\n", sdslen(mystring)); output> 0 ``` @@ -225,8 +222,8 @@ As an example of the binary safeness of SDS strings, we can run the following code: ```c -sds s = sdsnewlen("A\0\0B",4); -printf("%d\n", (int) sdslen(s)); +sds s = sdsnewlen("A\0\0B", 4); +printf("%zu\n", sdslen(s)); output> 4 ``` @@ -294,7 +291,7 @@ Usage is straightforward: ```c sds s1 = sdsnew("aaa"); sds s2 = sdsnew("bbb"); -s1 = sdscatsds(s1,s2); +s1 = sdscatsds(s1, s2); sdsfree(s2); printf("%s\n", s1); @@ -315,7 +312,7 @@ it with zero bytes. ```c sds s = sdsnew("Hello"); -s = sdsgrowzero(s,6); +s = sdsgrowzero(s, 6); s[5] = '!'; /* We are sure this is safe because of sdsgrowzero() */ printf("%s\n", s); @@ -338,7 +335,7 @@ Example: sds s; int a = 10, b = 20; s = sdsnew("The sum is: "); -s = sdscatprintf(s,"%d+%d = %d",a,b,a+b); +s = sdscatprintf(s, "%d+%d = %d", a, b, a + b); ``` Often you need to create SDS string directly from `printf` format specifiers. @@ -357,7 +354,7 @@ You can use `sdscatprintf` in order to convert numbers into SDS strings: ```c int some_integer = 100; -sds num = sdscatprintf(sdsempty(),"%d\n", some_integer); +sds num = sdscatprintf(sdsempty(), "%d\n", some_integer); ``` However this is slow and we have a special function to make it efficient. @@ -377,7 +374,7 @@ Use it like this: ```c sds s = sdsfromlonglong(10000); -printf("%d\n", (int) sdslen(s)); +printf("%zu\n", sdslen(s)); output> 5 ``` @@ -391,7 +388,7 @@ on strings is to just take a range out of a larger string. ```c void sdstrim(sds s, const char *cset); -void sdsrange(sds s, int start, int end); +void sdsrange(sds s, ptrdiff_t start, ptrdiff_t end); ``` SDS provides both the operations with the `sdstrim` and `sdsrange` functions. @@ -408,8 +405,8 @@ from an SDS strings: ```c sds s = sdsnew(" my string\n\n "); -sdstrim(s," \n"); -printf("-%s-\n",s); +sdstrim(s, " \n"); +printf("-%s-\n", s); output> -my string- ``` @@ -426,8 +423,8 @@ indexes inside the string, to obtain the range that will be retained. ```c sds s = sdsnew("Hello World!"); -sdsrange(s,1,4); -printf("-%s-\n",s); +sdsrange(s, 1, 4); +printf("-%s-\n", s); output> -ello- ``` @@ -437,10 +434,10 @@ string, so that `-1` means the last character, `-2` the penultimate, and so fort ```c sds s = sdsnew("Hello World!"); -sdsrange(s,6,-1); -printf("-%s-\n",s); -sdsrange(s,0,-2); -printf("-%s-\n",s); +sdsrange(s, 6, -1); +printf("-%s-\n", s); +sdsrange(s, 0, -2); +printf("-%s-\n", s); output> -World!- output> -World- @@ -458,7 +455,7 @@ void clusterWriteHandler(..., int fd, void *privdata, ...) { if (nwritten <= 0) { /* Error handling... */ } - sdsrange(link->sndbuf,nwritten,-1); + sdsrange(link->sndbuf, nwritten, -1); ... more code here ... } ``` @@ -498,7 +495,7 @@ The string copy function of SDS is called `sdscpylen` and works like this: ```c s = sdsnew("Hello World!"); -s = sdscpylen(s,"Hello Superman!",15); +s = sdscpylen(s, "Hello Superman!", 15); ``` As you can see the function receives as input the SDS string `s`, but also @@ -558,7 +555,7 @@ sds s2 = sdsempty(); s1[1] = 1; s1[2] = 2; s1[3] = '\n'; -s2 = sdscatrepr(s2,s1,sdslen(s1)); +s2 = sdscatrepr(s2, s1, sdslen(s1)); printf("%s\n", s2); output> "a\x01\x02\n" @@ -597,8 +594,8 @@ strings it is composed of, so SDS provides a function that returns an array of SDS strings given a string and a separator. ```c -sds *sdssplitlen(const char *s, int len, const char *sep, int seplen, int *count); -void sdsfreesplitres(sds *tokens, int count); +sds *sdssplitlen(const char *s, size_t len, const char *sep, size_t seplen, size_t *count); +void sdsfreesplitres(sds *tokens, size_t count); ``` As usual the function can work with either SDS strings or normal C strings. @@ -611,14 +608,14 @@ The return value is a heap allocated array of SDS strings. ```c sds *tokens; -int count, j; +size_t count; sds line = sdsnew("Hello World!"); -tokens = sdssplitlen(line,sdslen(line)," ",1,&count); +tokens = sdssplitlen(line, sdslen(line), " ", 1, &count); -for (j = 0; j < count; j++) +for (size_t j = 0; j < count; j++) printf("%s\n", tokens[j]); -sdsfreesplitres(tokens,count); +sdsfreesplitres(tokens, count); output> Hello output> World! @@ -690,8 +687,8 @@ in the array to be SDS strings. Because of this only `sdsjoinsds` is able to deal with binary data. ```c -char *tokens[3] = {"foo","bar","zap"}; -sds s = sdsjoin(tokens,3,"|",1); +char *tokens[3] = {"foo", "bar", "zap"}; +sds s = sdsjoin(tokens, 3, "|", 1); printf("%s\n", s); output> foo|bar|zap @@ -718,8 +715,8 @@ structure implementing it: ```c struct sdshdr { - int len; - int free; + size_t len; + size_t free; char buf[]; }; ``` @@ -727,7 +724,7 @@ struct sdshdr { As you can see, the structure may resemble the one of a conventional string library, however the `buf` field of the structure is different since it is not a pointer but an array without any length declared, so `buf` actually -points at the first byte just after the `free` integer. So in order to create +points at the first byte just after the `free` size_t. So in order to create an SDS string we just allocate a piece of memory that is as large as the `sdshdr` structure plus the length of our string, plus an additional byte for the mandatory null term that every SDS string has. @@ -753,9 +750,9 @@ will create extra free space at the end, like in the following program: ```c s = sdsempty(); -s = sdscat(s,"foo"); -s = sdscat(s,"bar"); -s = sdscat(s,"123"); +s = sdscat(s, "foo"); +s = sdscat(s, "bar"); +s = sdscat(s, "123"); ``` Since SDS tries to be efficient it can't afford to reallocate the string every @@ -796,10 +793,10 @@ total allocation for a given string, and is called `sdsAllocSize`. ```c sds s = sdsnew("Ladies and gentlemen"); -s = sdscat(s,"... welcome to the C language."); -printf("%d\n", (int) sdsAllocSize(s)); +s = sdscat(s, "... welcome to the C language."); +printf("%zu\n", sdsAllocSize(s)); s = sdsRemoveFreeSpace(s); -printf("%d\n", (int) sdsAllocSize(s)); +printf("%zu\n", sdsAllocSize(s)); output> 109 output> 59 @@ -823,9 +820,9 @@ information for the specified string to the length obtained via `strlen`. ```c sds s = sdsnew("foobar"); s[2] = '\0'; -printf("%d\n", sdslen(s)); +printf("%zu\n", sdslen(s)); sdsupdatelen(s); -printf("%d\n", sdslen(s)); +printf("%zu\n", sdslen(s)); output> 6 output> 2 @@ -886,7 +883,7 @@ sds string without copying into an intermediate buffer: ```c oldlen = sdslen(s); s = sdsMakeRoomFor(s, BUFFER_SIZE); -nread = read(fd, s+oldlen, BUFFER_SIZE); +nread = read(fd, s + oldlen, BUFFER_SIZE); ... check for nread <= 0 and handle it ... sdsIncrLen(s, nread); ``` @@ -903,4 +900,6 @@ it without issues. ## Credits and license -SDS was created by Salvatore Sanfilippo and is released under the BSD two clause license. See the LICENSE file in this source distribution for more information. +SDS was created by Salvatore Sanfilippo and is released under the BSD two +clause license. See the LICENSE file in this source distribution for more +information. -- cgit v1.2.3-54-g00ecf