# HG changeset patch # User J. Ali Harlow # Date 1408807711 -3600 # Node ID c4bcba8023a9805f8723e56d72a01a713af24649 # Parent cf499fd51df7543278e7fca74f96566aeadad603 Fix compiler warnings diff -r cf499fd51df7 -r c4bcba8023a9 configure.ac --- a/configure.ac Sat Aug 23 16:07:09 2014 +0100 +++ b/configure.ac Sat Aug 23 16:28:31 2014 +0100 @@ -118,7 +118,7 @@ changequote(,)dnl case " $CFLAGS " in *[\ \ ]-Wall[\ \ ]*) ;; - *) CFLAGS="$CFLAGS -Wall" ;; + *) CFLAGS="$CFLAGS -Wall -Wno-parentheses" ;; esac case " $CFLAGS " in diff -r cf499fd51df7 -r c4bcba8023a9 librazor/error.c --- a/librazor/error.c Sat Aug 23 16:07:09 2014 +0100 +++ b/librazor/error.c Sat Aug 23 16:28:31 2014 +0100 @@ -27,7 +27,7 @@ #include "razor.h" #include "razor-internal.h" -const char *razor_error_get_path_str(struct razor_error *error) +static const char *razor_error_get_path_str(struct razor_error *error) { if (error->path_str) return error->path_str; diff -r cf499fd51df7 -r c4bcba8023a9 librazor/importer.c --- a/librazor/importer.c Sat Aug 23 16:07:09 2014 +0100 +++ b/librazor/importer.c Sat Aug 23 16:28:31 2014 +0100 @@ -434,7 +434,7 @@ struct import_entry *filenames; char *f, *end; uint32_t name, *r, s; - char rootname[256], dirname[256]; + char dirname[256]; struct import_directory *d, *last_root; struct array roots; struct razor_entry *e; diff -r cf499fd51df7 -r c4bcba8023a9 librazor/iterator.c --- a/librazor/iterator.c Sat Aug 23 16:07:09 2014 +0100 +++ b/librazor/iterator.c Sat Aug 23 16:28:31 2014 +0100 @@ -24,6 +24,7 @@ #include #include #include +#include #include "razor-internal.h" #include "razor.h" diff -r cf499fd51df7 -r c4bcba8023a9 librazor/lua.c --- a/librazor/lua.c Sat Aug 23 16:07:09 2014 +0100 +++ b/librazor/lua.c Sat Aug 23 16:28:31 2014 +0100 @@ -323,13 +323,13 @@ end = razor_preload.loaders.data + razor_preload.loaders.size; for(ploader = razor_preload.loaders.data; ploader < end; ploader++) if (ploader->name == name) { - ploader->func = loader; + ploader->func = (lua_CFunction)loader; return; } ploader = array_add(&razor_preload.loaders, sizeof(*ploader)); ploader->name = name; - ploader->func = loader; + ploader->func = (lua_CFunction)loader; } RAZOR_EXPORT void (*razor_get_lua_loader(const char *modname))() @@ -345,7 +345,7 @@ end = razor_preload.loaders.data + razor_preload.loaders.size; for(ploader = razor_preload.loaders.data; ploader < end; ploader++) if (ploader->name == name) - return ploader->func; + return (void (*)())ploader->func; return 0; } diff -r cf499fd51df7 -r c4bcba8023a9 librazor/razor.c --- a/librazor/razor.c Sat Aug 23 16:07:09 2014 +0100 +++ b/librazor/razor.c Sat Aug 23 16:28:31 2014 +0100 @@ -1225,7 +1225,7 @@ { struct razor_merger *merger; struct razor_set *set; - struct razor_package *n, *nend, *npkgs, *s, *send, *spkgs; + struct razor_package *n, *nend, *s, *send; struct deque *done; size_t pos; char *npool, *spool; @@ -1237,12 +1237,10 @@ deque_shift(done); s = ii->set->packages.data; - spkgs = ii->set->packages.data; send = ii->set->packages.data + ii->set->packages.size; spool = ii->set->string_pool.data; n = ii->next->packages.data; - npkgs = ii->next->packages.data; nend = ii->next->packages.data + ii->next->packages.size; npool = ii->next->string_pool.data; diff -r cf499fd51df7 -r c4bcba8023a9 librazor/rpm.c --- a/librazor/rpm.c Sat Aug 23 16:07:09 2014 +0100 +++ b/librazor/rpm.c Sat Aug 23 16:28:31 2014 +0100 @@ -1077,7 +1077,7 @@ err = inflateInit2(&installer->stream, -MAX_WBITS); if (err != Z_OK) { sprintf(buffer, "%d", err); - s = razor_concat("inflateEnd error: ", s, NULL); + s = razor_concat("inflateEnd error: ", buffer, NULL); razor_atomic_abort(installer->atomic, s); free(s); return -1; @@ -1096,7 +1096,7 @@ if (err != Z_OK) { sprintf(buffer, "%d", err); - s = razor_concat("inflateEnd error: ", s, NULL); + s = razor_concat("inflateEnd error: ", buffer, NULL); razor_atomic_abort(installer->atomic, s); free(s); } diff -r cf499fd51df7 -r c4bcba8023a9 librazor/transaction.c --- a/librazor/transaction.c Sat Aug 23 16:07:09 2014 +0100 +++ b/librazor/transaction.c Sat Aug 23 16:28:31 2014 +0100 @@ -293,7 +293,7 @@ const char *version) { struct razor_property *p; - struct razor_package *pkg, *pkgs; + struct razor_package *pkg; struct razor_package_iterator pkg_iter; struct razor_set *set; const char *n, *v; @@ -304,7 +304,6 @@ else set = trans->upstream.set; - pkgs = (struct razor_package *) set->packages.data; type = ppi->p->flags & RAZOR_PROPERTY_TYPE_MASK; for (p = ppi->p; p < ppi->end && @@ -425,10 +424,8 @@ remove_obsoleted_packages(struct razor_transaction *trans) { struct razor_property *up; - struct razor_package *spkgs; struct prop_iter spi, upi; - spkgs = trans->system.set->packages.data; prop_iter_init(&spi, &trans->system); prop_iter_init(&upi, &trans->upstream); diff -r cf499fd51df7 -r c4bcba8023a9 librazor/util.c --- a/librazor/util.c Sat Aug 23 16:07:09 2014 +0100 +++ b/librazor/util.c Sat Aug 23 16:28:31 2014 +0100 @@ -244,7 +244,10 @@ void environment_set(struct environment *env) { int i, count; - char *s, *t; + char *s; +#ifndef WIN32 + char *t; +#endif uint32_t *r; if (!env->is_set) { diff -r cf499fd51df7 -r c4bcba8023a9 src/import-yum.c --- a/src/import-yum.c Sat Aug 23 16:07:09 2014 +0100 +++ b/src/import-yum.c Sat Aug 23 16:28:31 2014 +0100 @@ -287,7 +287,7 @@ { struct yum_context ctx={0}; void *buf; - int len, ret; + int len; gzFile primary, filelists; XML_ParsingStatus status; @@ -325,7 +325,7 @@ XML_GetParsingStatus(ctx.current_parser, &status); switch (status.parsing) { case XML_SUSPENDED: - ret = XML_ResumeParser(ctx.current_parser); + XML_ResumeParser(ctx.current_parser); break; case XML_PARSING: case XML_INITIALIZED: diff -r cf499fd51df7 -r c4bcba8023a9 src/main.c --- a/src/main.c Sat Aug 23 16:07:09 2014 +0100 +++ b/src/main.c Sat Aug 23 16:28:31 2014 +0100 @@ -411,6 +411,7 @@ RAZOR_PROPERTY_PROVIDES); } +#ifdef HAVE_CURL static int show_progress(void *clientp, double dltotal, double dlnow, double ultotal, double ulnow) @@ -423,6 +424,7 @@ return 0; } +#endif /* HAVE_CURL */ static int download_if_missing(const char *url, const char *file)