diff options
author | Dan McGee <dan@archlinux.org> | 2011-01-29 11:30:01 -0600 |
---|---|---|
committer | Dan McGee <dan@archlinux.org> | 2011-01-31 09:37:20 -0600 |
commit | 986edb8bd4308b73ef225a554f13a07c232ead91 (patch) | |
tree | 6604877bd39925d443613e03696cb5a9087056b7 /src/pacman | |
parent | d6a9436143920f794d785423ac35eb351d791776 (diff) |
Style cleanups in clean cache code
Signed-off-by: Dan McGee <dan@archlinux.org>
Diffstat (limited to 'src/pacman')
-rw-r--r-- | src/pacman/sync.c | 16 |
1 files changed, 9 insertions, 7 deletions
diff --git a/src/pacman/sync.c b/src/pacman/sync.c index 1e622d37..2e68c8da 100644 --- a/src/pacman/sync.c +++ b/src/pacman/sync.c @@ -42,6 +42,7 @@ static int sync_cleandb(const char *dbpath, int keep_used) { DIR *dir; struct dirent *ent; + alpm_list_t *syncdbs; dir = opendir(dbpath); if(dir == NULL) { @@ -49,12 +50,13 @@ static int sync_cleandb(const char *dbpath, int keep_used) { return(1); } + syncdbs = alpm_option_get_syncdbs(); + rewinddir(dir); /* step through the directory one file at a time */ while((ent = readdir(dir)) != NULL) { char path[PATH_MAX]; struct stat buf; - alpm_list_t *syncdbs = NULL, *i; int found = 0; const char *dname = ent->d_name; size_t len; @@ -77,7 +79,7 @@ static int sync_cleandb(const char *dbpath, int keep_used) { /* remove all non-skipped directories and non-database files */ stat(path, &buf); len = strlen(path); - if(S_ISDIR(buf.st_mode) || strcmp(path+(len-3),".db") != 0) { + if(S_ISDIR(buf.st_mode) || strcmp(path + len - 3, ".db") != 0) { if(rmrf(path)) { pm_fprintf(stderr, PM_LOG_ERROR, _("could not remove %s\n"), path); @@ -88,9 +90,9 @@ static int sync_cleandb(const char *dbpath, int keep_used) { } if(keep_used) { + alpm_list_t *i; len = strlen(dname); - char *dbname = strndup(dname, len-3); - syncdbs = alpm_option_get_syncdbs(); + char *dbname = strndup(dname, len - 3); for(i = syncdbs; i && !found; i = alpm_list_next(i)) { pmdb_t *db = alpm_list_getdata(i); found = !strcmp(dbname, alpm_db_get_name(db)); @@ -126,9 +128,9 @@ static int sync_cleandb_all(void) { if(!yesno(_("Do you want to remove unused repositories?"))) { return(0); } - /* The sync dbs were previously put in dbpath/, but are now in dbpath/sync/, - * so we will clean everything in dbpath/ (except dbpath/local/ and dbpath/sync/ - * and db.lck) and only the unused sync dbs in dbpath/sync/ */ + /* The sync dbs were previously put in dbpath/ but are now in dbpath/sync/. + * We will clean everything in dbpath/ except local/, sync/ and db.lck, and + * only the unused sync dbs in dbpath/sync/ */ ret += sync_cleandb(dbpath, 0); sprintf(newdbpath, "%s%s", dbpath, "sync/"); |