diff options
author | François Revel <mmu_man@netsurf-browser.org> | 2008-07-29 09:05:36 +0000 |
---|---|---|
committer | François Revel <mmu_man@netsurf-browser.org> | 2008-07-29 09:05:36 +0000 |
commit | f28f4cc76085b4c5cf589b3ca771fa7379d90eac (patch) | |
tree | 53ba572fbdb0e4bda5ace3901e15b55ffb8b4337 | |
parent | 00687fcddac4f145fbe143426dbb923fca532a8b (diff) | |
download | netsurf-f28f4cc76085b4c5cf589b3ca771fa7379d90eac.tar.gz netsurf-f28f4cc76085b4c5cf589b3ca771fa7379d90eac.tar.bz2 |
First round of C89 fixes. Those should be obvious, hope I didn't introduce any side effect.
svn path=/trunk/netsurf/; revision=4788
-rw-r--r-- | content/fetch.c | 6 | ||||
-rw-r--r-- | content/fetchcache.c | 3 | ||||
-rw-r--r-- | content/fetchers/fetch_curl.c | 2 | ||||
-rw-r--r-- | content/urldb.c | 28 |
4 files changed, 25 insertions, 14 deletions
diff --git a/content/fetch.c b/content/fetch.c index f3433d040..779e1f12d 100644 --- a/content/fetch.c +++ b/content/fetch.c @@ -352,6 +352,8 @@ failed: void fetch_dispatch_jobs(void) { int all_active, all_queued; + struct fetch *q; + struct fetch *f; if (!queue_ring) return; /* Nothing to do, the queue is empty */ @@ -362,7 +364,7 @@ void fetch_dispatch_jobs(void) LOG(("queue_ring %i, fetch_ring %i", all_queued, all_active)); #endif - struct fetch *q = queue_ring; + q = queue_ring; if (q) { do { #ifdef DEBUG_FETCH_VERBOSE @@ -371,7 +373,7 @@ void fetch_dispatch_jobs(void) q = q->r_next; } while (q != queue_ring); } - struct fetch *f = fetch_ring; + f = fetch_ring; if (f) { do { #ifdef DEBUG_FETCH_VERBOSE diff --git a/content/fetchcache.c b/content/fetchcache.c index fa6938a32..e20785e2b 100644 --- a/content/fetchcache.c +++ b/content/fetchcache.c @@ -574,6 +574,7 @@ char *fetchcache_parse_type(const char *s, char **params[]) r = regexec(&re_content_type, s, 2 + MAX_ATTRS * 3, pmatch, 0); if (r) { + char *semi; LOG(("failed to parse content-type '%s'", s)); /* The mime type must be first, so only copy up to the * first semicolon in the string. This allows us to have @@ -581,7 +582,7 @@ char *fetchcache_parse_type(const char *s, char **params[]) * Content-Type headers. Obviously, any truly broken * Content-Type headers will be unaffected by this heuristic */ - char *semi = strchr(s, ';'); + semi = strchr(s, ';'); if (semi) type = strndup(s, semi - s); else diff --git a/content/fetchers/fetch_curl.c b/content/fetchers/fetch_curl.c index a88038205..4e9912277 100644 --- a/content/fetchers/fetch_curl.c +++ b/content/fetchers/fetch_curl.c @@ -252,9 +252,9 @@ void fetch_curl_finalise(const char *scheme) curl_fetchers_registered--; LOG(("Finalise cURL fetcher %s", scheme)); if (curl_fetchers_registered == 0) { + CURLMcode codem; /* All the fetchers have been finalised. */ LOG(("All cURL fetchers finalised, closing down cURL")); - CURLMcode codem; curl_easy_cleanup(fetch_blank_curl); diff --git a/content/urldb.c b/content/urldb.c index e4a81fc26..16f71eed9 100644 --- a/content/urldb.c +++ b/content/urldb.c @@ -2340,6 +2340,7 @@ char *urldb_get_cookie(const char *url) char *scheme; time_t now; url_func_result res; + int i; assert(url); @@ -2552,7 +2553,7 @@ char *urldb_get_cookie(const char *url) ret_used = strlen(ret) + 1; } - for (int i = 0; i < count; i++) { + for (i = 0; i < count; i++) { if (!urldb_concat_cookie(matched_cookies[i], version, &ret_used, &ret_alloc, &ret)) { free(path); @@ -3490,6 +3491,7 @@ void urldb_load_cookies(const char *filename) int version, domain_specified, path_specified, secure, no_destroy, value_quoted; time_t expires, last_used; + struct cookie_internal_data *c; if(s[0] == 0 || s[0] == '#') /* Skip blank lines or comments */ @@ -3548,8 +3550,7 @@ void urldb_load_cookies(const char *filename) assert(p <= end); /* Now create cookie */ - struct cookie_internal_data *c = - malloc(sizeof(struct cookie_internal_data)); + c = malloc(sizeof(struct cookie_internal_data)); if (!c) break; @@ -3602,11 +3603,12 @@ void urldb_delete_cookie(const char *domain, const char *path, void urldb_delete_cookie_hosts(const char *domain, const char *path, const char *name, struct host_part *parent) { + struct host_part *h; assert(parent); urldb_delete_cookie_paths(domain, path, name, &parent->paths); - for (struct host_part *h = parent->children; h; h = h->next) + for (h = parent->children; h; h = h->next) urldb_delete_cookie_hosts(domain, path, name, h); } @@ -3614,6 +3616,7 @@ void urldb_delete_cookie_paths(const char *domain, const char *path, const char *name, struct path_data *parent) { struct cookie_internal_data *c; + struct path_data *p; assert(parent); @@ -3635,7 +3638,7 @@ void urldb_delete_cookie_paths(const char *domain, const char *path, } } - for (struct path_data *p = parent->children; p; p = p->next) + for (p = parent->children; p; p = p->next) urldb_delete_cookie_paths(domain, path, name, p); } @@ -3684,11 +3687,12 @@ void urldb_save_cookies(const char *filename) */ void urldb_save_cookie_hosts(FILE *fp, struct host_part *parent) { + struct host_part *h; assert(fp && parent); urldb_save_cookie_paths(fp, &parent->paths); - for (struct host_part *h = parent->children; h; h = h->next) + for (h = parent->children; h; h = h->next) urldb_save_cookie_hosts(fp, h); } @@ -3700,12 +3704,14 @@ void urldb_save_cookie_hosts(FILE *fp, struct host_part *parent) */ void urldb_save_cookie_paths(FILE *fp, struct path_data *parent) { + struct path_data *p; time_t now = time(NULL); assert(fp && parent); if (parent->cookies) { - for (struct cookie_internal_data *c = parent->cookies; c; + struct cookie_internal_data *c; + for (c = parent->cookies; c; c = c->next) { if (c->expires < now) /* Skip expired cookies */ @@ -3726,7 +3732,7 @@ void urldb_save_cookie_paths(FILE *fp, struct path_data *parent) } } - for (struct path_data *p = parent->children; p; p = p->next) + for (p = parent->children; p; p = p->next) urldb_save_cookie_paths(fp, p); } @@ -3737,9 +3743,10 @@ void urldb_save_cookie_paths(FILE *fp, struct path_data *parent) void urldb_destroy(void) { struct host_part *a, *b; + int i; /* Clean up search trees */ - for (int i = 0; i < NUM_SEARCH_TREES; i++) { + for (i = 0; i < NUM_SEARCH_TREES; i++) { if (search_trees[i] != &empty) urldb_destroy_search_tree(search_trees[i]); } @@ -3809,11 +3816,12 @@ void urldb_destroy_path_tree(struct path_data *root) void urldb_destroy_path_node_content(struct path_data *node) { struct cookie_internal_data *a, *b; + unsigned int i; free(node->url); free(node->scheme); free(node->segment); - for (unsigned int i = 0; i < node->frag_cnt; i++) + for (i = 0; i < node->frag_cnt; i++) free(node->fragment[i]); free(node->fragment); |