diff options
author | Panu Matilainen <pmatilai@redhat.com> | 2010-04-06 13:09:34 +0300 |
---|---|---|
committer | Panu Matilainen <pmatilai@redhat.com> | 2010-04-06 13:09:34 +0300 |
commit | 3437ad49b934cbf6503129dfbb6effd0c6012221 (patch) | |
tree | d581a7ffc4688c5f3b2a8c5089ed1005d088ed5d /lib/rpmdb.c | |
parent | 7b4da15c666d1d744b794a43098691900bcf0797 (diff) | |
download | rpm-3437ad49b934cbf6503129dfbb6effd0c6012221.tar.gz rpm-3437ad49b934cbf6503129dfbb6effd0c6012221.tar.bz2 rpm-3437ad49b934cbf6503129dfbb6effd0c6012221.zip |
Lose unused fluff from openDatabase()
- As RPMDBI_PACKAGES is always the first index, the for-loop never
did anything else than open the Packages db and exit with success/fail.
For the same reason RPMDB_FLAG_MINIMAL didn't do anything at all here,
lose the unnecessary flag..
Diffstat (limited to 'lib/rpmdb.c')
-rw-r--r-- | lib/rpmdb.c | 39 |
1 files changed, 4 insertions, 35 deletions
diff --git a/lib/rpmdb.c b/lib/rpmdb.c index 33fe2305a..0f7f8e283 100644 --- a/lib/rpmdb.c +++ b/lib/rpmdb.c @@ -37,8 +37,7 @@ enum rpmdbFlags { RPMDB_FLAG_JUSTCHECK = (1 << 0), - RPMDB_FLAG_MINIMAL = (1 << 1), - RPMDB_FLAG_REBUILD = (1 << 2), + RPMDB_FLAG_REBUILD = (1 << 1), }; #define _DBI_FLAGS 0 @@ -734,7 +733,6 @@ static int openDatabase(const char * prefix, rpmdb db; int rc, xx; int justCheck = flags & RPMDB_FLAG_JUSTCHECK; - int minimal = flags & RPMDB_FLAG_MINIMAL; /* Insure that _dbapi has one of -1, 1, 2, or 3 */ if (_dbapi < -1 || _dbapi > 4) @@ -759,37 +757,9 @@ static int openDatabase(const char * prefix, db->db_api = _dbapi; - { rc = 0; - for (int dbix = 0; rc == 0 && dbix < dbiTagsMax; dbix++) { - rpmTag rpmtag = dbiTags[dbix]; - dbiIndex dbi = rpmdbOpenIndex(db, rpmtag, 0); - - if (dbi == NULL) { - rc = -2; - break; - } - - switch (rpmtag) { - case RPMDBI_PACKAGES: - if (dbi == NULL) rc |= 1; -#if 0 - /* XXX open only Packages, indices created on the fly. */ - if (db->db_api == 3) -#endif - goto exit; - break; - case RPMTAG_NAME: - if (dbi == NULL) rc |= 1; - if (minimal) - goto exit; - break; - default: - break; - } - } - } + /* Just the primary Packages database opened here */ + rc = (rpmdbOpenIndex(db, RPMDBI_PACKAGES, 0) != NULL) ? 0 : -2; -exit: if (rc || justCheck || dbp == NULL) xx = rpmdbClose(db); else { @@ -2883,8 +2853,7 @@ int rpmdbRebuild(const char * prefix, rpmts ts, rpmlog(RPMLOG_DEBUG, "opening old database with dbapi %d\n", _dbapi); - if (openDatabase(prefix, dbpath, _dbapi, &olddb, O_RDONLY, 0644, - RPMDB_FLAG_MINIMAL)) { + if (openDatabase(prefix, dbpath, _dbapi, &olddb, O_RDONLY, 0644, 0)) { rc = 1; goto exit; } |