summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAurelien Foret <aurelien@archlinux.org>2006-01-02 22:33:58 +0100
committerAurelien Foret <aurelien@archlinux.org>2006-01-02 22:33:58 +0100
commitf48aced5b29266b260b2a94f2b28e1e2f8e1b4d0 (patch)
treed7f302478533dabdab378d9b3a6921bd27398572
parent5eb29e2127e6834a52c452a8dc07dba7f68f283e (diff)
downloadpacman-f48aced5b29266b260b2a94f2b28e1e2f8e1b4d0.tar.gz
pacman-f48aced5b29266b260b2a94f2b28e1e2f8e1b4d0.tar.xz
code cleanup
-rw-r--r--lib/libalpm/sync.c25
1 files changed, 13 insertions, 12 deletions
diff --git a/lib/libalpm/sync.c b/lib/libalpm/sync.c
index 330e754c..7b3cafae 100644
--- a/lib/libalpm/sync.c
+++ b/lib/libalpm/sync.c
@@ -382,7 +382,7 @@ int sync_prepare(pmtrans_t *trans, pmdb_t *db_local, PMList *dbs_sync, PMList **
EVENT(trans, PM_TRANS_EVT_INTERCONFLICTS_START, NULL, NULL);
deps = checkdeps(db_local, PM_TRANS_TYPE_UPGRADE, list);
if(deps) {
- int found;
+ int found = 0;
PMList *j, *k;
int errorout = 0;
@@ -418,16 +418,14 @@ int sync_prepare(pmtrans_t *trans, pmdb_t *db_local, PMList *dbs_sync, PMList **
/* check if the conflicting package is one that's about to be removed/replaced.
* if so, then just ignore it
*/
- found = 0;
for(j = trans->packages; j && !found; j = j->next) {
pmsyncpkg_t *sync = j->data;
- if(sync->type != PM_SYNC_TYPE_REPLACE) {
- continue;
- }
- for(k = sync->data; k && !found; k = k->next) {
- pmpkg_t *p = k->data;
- if(!strcmp(p->name, miss->depend.name)) {
- found = 1;
+ if(sync->type == PM_SYNC_TYPE_REPLACE || sync->type == PM_SYNC_TYPE_REMOVE) {
+ for(k = sync->data; k && !found; k = k->next) {
+ pmpkg_t *p = k->data;
+ if(!strcmp(p->name, miss->depend.name)) {
+ found = 1;
+ }
}
}
}
@@ -479,8 +477,11 @@ int sync_prepare(pmtrans_t *trans, pmdb_t *db_local, PMList *dbs_sync, PMList **
if(rmpkg) {
for(k = trans->packages; k; k = k->next) {
pmsyncpkg_t *sync = k->data;
- if(!strcmp(sync->pkg->name, rmpkg))
- trans->packages = _alpm_list_remove(trans->packages, sync, ptr_cmp, (void **)&data);
+ if(!strcmp(sync->pkg->name, rmpkg)) {
+ pmsyncpkg_t *spkg;
+ trans->packages = _alpm_list_remove(trans->packages, sync, ptr_cmp, (void **)&spkg);
+ FREESYNC(spkg);
+ }
}
solved = 1;
FREE(rmpkg);
@@ -599,7 +600,7 @@ int sync_commit(pmtrans_t *trans, pmdb_t *db_local)
goto error;
}
- trans_init(tr, PM_TRANS_TYPE_REMOVE, PM_TRANS_FLAG_NODEPS, trans->cb_event, trans->cb_conv);
+ trans_init(tr, PM_TRANS_TYPE_REMOVE, PM_TRANS_FLAG_NODEPS, NULL, NULL);
for(i = trans->packages; i; i = i->next) {
pmsyncpkg_t *sync = i->data;