diff options
author | Dan McGee <dan@archlinux.org> | 2007-09-10 19:13:38 -0500 |
---|---|---|
committer | Dan McGee <dan@archlinux.org> | 2007-09-11 20:41:42 -0500 |
commit | 2f0de317b862e154f0b172da0668b887413cc55f (patch) | |
tree | 97435c2dd2608c793f5ccd144faef0d3aa78e0a9 /src | |
parent | b2501950c7fca0b771fc79054d9592ea79753749 (diff) |
Make some small changes recommended by splint
Signed-off-by: Dan McGee <dan@archlinux.org>
Diffstat (limited to 'src')
-rw-r--r-- | src/pacman/conf.c | 22 | ||||
-rw-r--r-- | src/pacman/conf.h | 2 | ||||
-rw-r--r-- | src/pacman/query.c | 20 | ||||
-rw-r--r-- | src/util/testpkg.c | 2 |
4 files changed, 24 insertions, 22 deletions
diff --git a/src/pacman/conf.c b/src/pacman/conf.c index a4fd1318..2bc69820 100644 --- a/src/pacman/conf.c +++ b/src/pacman/conf.c @@ -30,29 +30,29 @@ config_t *config_new(void) { - config_t *config = calloc(1, sizeof(config_t)); - if(!config) { + config_t *newconfig = calloc(1, sizeof(config_t)); + if(!newconfig) { fprintf(stderr, "malloc failure: could not allocate %d bytes\n", sizeof(config_t)); } /* defaults which may get overridden later */ - config->op = PM_OP_MAIN; - config->logmask = PM_LOG_ERROR | PM_LOG_WARNING; + newconfig->op = PM_OP_MAIN; + newconfig->logmask = PM_LOG_ERROR | PM_LOG_WARNING; /* CONFFILE is defined at compile-time */ - config->configfile = strdup(CONFFILE); + newconfig->configfile = strdup(CONFFILE); - return(config); + return(newconfig); } -int config_free(config_t *config) +int config_free(config_t *oldconfig) { - if(config == NULL) { + if(oldconfig == NULL) { return(-1); } - free(config->configfile); - free(config); - config = NULL; + free(oldconfig->configfile); + free(oldconfig); + oldconfig = NULL; return(0); } diff --git a/src/pacman/conf.h b/src/pacman/conf.h index 6e6bdb81..2f4b0bc4 100644 --- a/src/pacman/conf.h +++ b/src/pacman/conf.h @@ -82,7 +82,7 @@ enum { }; config_t *config_new(void); -int config_free(config_t *config); +int config_free(config_t *oldconfig); #endif /* _PM_CONF_H */ diff --git a/src/pacman/query.c b/src/pacman/query.c index 8ef47e6b..476ca152 100644 --- a/src/pacman/query.c +++ b/src/pacman/query.c @@ -153,7 +153,7 @@ static int query_search(alpm_list_t *targets) /* print the package size with the output if ShowSize option set */ if(config->showsize) { /* Convert byte size to MB */ - double mbsize = alpm_pkg_get_size(pkg) / (1024.0 * 1024.0); + double mbsize = (double)alpm_pkg_get_size(pkg) / (1024.0 * 1024.0); printf(" [%.2f MB]", mbsize); } @@ -168,10 +168,11 @@ static int query_search(alpm_list_t *targets) indentprint(alpm_pkg_get_desc(pkg), 4); printf("\n"); } - /* we only want to free if the list was a search list */ - if(freelist) { - alpm_list_free(searchlist); - } + + /* we only want to free if the list was a search list */ + if(freelist) { + alpm_list_free(searchlist); + } return(0); } @@ -195,8 +196,9 @@ static int query_group(alpm_list_t *targets) } } else { for(i = targets; i; i = alpm_list_next(i)) { + pmgrp_t *grp; package = alpm_list_getdata(i); - pmgrp_t *grp = alpm_db_readgrp(db_local, package); + grp = alpm_db_readgrp(db_local, package); if(grp) { const alpm_list_t *p, *pkgnames = alpm_grp_get_pkgs(grp); for(p = pkgnames; p; p = alpm_list_next(p)) { @@ -236,8 +238,8 @@ static int query_test(void) static int query_upgrades(void) { - printf(_("Checking for package upgrades... \n")); alpm_list_t *syncpkgs; + printf(_("Checking for package upgrades... \n")); if((syncpkgs = alpm_db_get_upgrades()) != NULL) { display_targets(syncpkgs); @@ -257,8 +259,8 @@ static int is_foreign(pmpkg_t *pkg) int match = 0; for(j = sync_dbs; j; j = alpm_list_next(j)) { pmdb_t *db = alpm_list_getdata(j); - pmpkg_t *pkg = alpm_db_get_pkg(db, pkgname); - if(pkg) { + pmpkg_t *findpkg = alpm_db_get_pkg(db, pkgname); + if(findpkg) { match = 1; break; } diff --git a/src/util/testpkg.c b/src/util/testpkg.c index 53ae8322..71993bdc 100644 --- a/src/util/testpkg.c +++ b/src/util/testpkg.c @@ -27,7 +27,7 @@ #include <alpm.h> -void output_cb(pmloglevel_t level, char *fmt, va_list args) +static void output_cb(pmloglevel_t level, char *fmt, va_list args) { if(strlen(fmt)) { switch(level) { |