diff options
| author | Dan McGee <dan@archlinux.org> | 2011-06-27 17:10:08 +0200 |
|---|---|---|
| committer | Dan McGee <dan@archlinux.org> | 2011-06-27 17:10:08 +0200 |
| commit | f01c6f814a278a3d024d34fed0c219c8cb1e1e33 (patch) | |
| tree | 759833d3d7c46bca4f962c5fd442ab3f37d85b14 /lib/libalpm/backup.c | |
| parent | 77a09c92c66bd06440be0ccb75daf07c620acbee (diff) | |
| download | pacman-f01c6f814a278a3d024d34fed0c219c8cb1e1e33.tar.gz pacman-f01c6f814a278a3d024d34fed0c219c8cb1e1e33.tar.xz | |
Fix several -Wshadow warnings
Only one of these looked like a real red flag, in find_requiredby(), but
it doesn't hurt to fix several of them up anyway.
Unfortunately, we can't turn this on universally due to things like the
sync(), remove(), etc. builtins which we often use as variable names.
Signed-off-by: Dan McGee <dan@archlinux.org>
Diffstat (limited to 'lib/libalpm/backup.c')
| -rw-r--r-- | lib/libalpm/backup.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/lib/libalpm/backup.c b/lib/libalpm/backup.c index 20fb8a80..dc0c5674 100644 --- a/lib/libalpm/backup.c +++ b/lib/libalpm/backup.c @@ -58,16 +58,16 @@ int _alpm_split_backup(const char *string, pmbackup_t **backup) /* Look for a filename in a pmpkg_t.backup list. If we find it, * then we return the full backup entry. */ -pmbackup_t *_alpm_needbackup(const char *file, const alpm_list_t *backup) +pmbackup_t *_alpm_needbackup(const char *file, const alpm_list_t *backup_list) { const alpm_list_t *lp; - if(file == NULL || backup == NULL) { + if(file == NULL || backup_list == NULL) { return NULL; } /* run through the backup list and parse out the hash for our file */ - for(lp = backup; lp; lp = lp->next) { + for(lp = backup_list; lp; lp = lp->next) { pmbackup_t *backup = lp->data; if(strcmp(file, backup->name) == 0) { |