diff options
author | Hiltjo Posthuma <hiltjo@codemadness.org> | 2014-04-02 16:00:52 +0200 |
---|---|---|
committer | Hiltjo Posthuma <hiltjo@codemadness.org> | 2014-04-02 16:00:52 +0200 |
commit | 0c509625066b59346b6a0be6bfaf96bfeb0fa0db (patch) | |
tree | 73b5387a8de44f7f2dbb07f2c2026f5b652eb359 /util.c | |
parent | 0761823f244682e43f3058cede70f4cc239cb452 (diff) |
sfeed: improve parsetime(), cleanup
Signed-off-by: Hiltjo Posthuma <hiltjo@codemadness.org>
Diffstat (limited to 'util.c')
-rw-r--r-- | util.c | 49 |
1 files changed, 3 insertions, 46 deletions
@@ -7,47 +7,6 @@ #include "util.h" -#if 0 -/* TODO: optimize */ -char * -afgets(char **p, size_t *size, FILE *fp) { - char buf[BUFSIZ], *alloc = NULL; - size_t n, len = 0, allocsiz; - int end = 0; - - while(fgets(buf, sizeof(buf), fp)) { - n = strlen(buf); - if(buf[n - 1] == '\n') { /* dont store newlines. */ - buf[n - 1] = '\0'; - n--; - end = 1; /* newline found, end */ - } - len += n; - allocsiz = len + 1; - if(allocsiz > *size) { - if((alloc = realloc(*p, allocsiz))) { - *p = alloc; - *size = allocsiz; - } else { - free(*p); - *p = NULL; - fputs("error: could not realloc\n", stderr); - exit(EXIT_FAILURE); - return NULL; - } - } - strncpy((*p + (len - n)), buf, n); - if(end || feof(fp)) - break; - } - if(*p && len > 0) { - (*p)[len] = '\0'; - return *p; - } - return NULL; -} -#endif - /* * Taken from OpenBSD. * Copy src to string dst of size siz. At most siz-1 characters @@ -77,7 +36,6 @@ strlcpy(char *dst, const char *src, size_t siz) { return(s - src - 1); /* count does not include NUL */ } -/* TODO: optimize */ char * afgets(char **p, size_t *size, FILE *fp) { char buf[BUFSIZ], *alloc = NULL; @@ -105,8 +63,7 @@ afgets(char **p, size_t *size, FILE *fp) { return NULL; } } - strlcpy((*p + (len - n)), buf, n + 1); /* TODO: dont depend on strlcpy */ -/* strncpy((*p + (len - n)), buf, n);*/ + strlcpy((*p + (len - n)), buf, n + 1); } if(*p && len > 0) { (*p)[len] = '\0'; @@ -194,9 +151,9 @@ feedsfree(struct feed *f) { for(; f; f = next) { next = f->next; - /*f->next = NULL;*/ + f->next = NULL; free(f->name); - /*f->name = NULL;*/ + f->name = NULL; free(f); } } |