Eliminate useless dbapi version to internal openDatabase()
authorPanu Matilainen <pmatilai@redhat.com>
Thu, 4 Nov 2010 13:45:27 +0000 (15:45 +0200)
committerPanu Matilainen <pmatilai@redhat.com>
Thu, 4 Nov 2010 13:45:27 +0000 (15:45 +0200)
lib/rpmdb.c

index c1a24da..628dae9 100644 (file)
@@ -750,20 +750,13 @@ static rpmdb newRpmdb(const char * root, const char * home,
 }
 
 static int openDatabase(const char * prefix,
-               const char * dbpath,
-               int _dbapi, rpmdb *dbp,
+               const char * dbpath, rpmdb *dbp,
                int mode, int perms, int flags)
 {
     rpmdb db;
     int rc, xx;
     int justCheck = flags & RPMDB_FLAG_JUSTCHECK;
 
-    /* Insure that _dbapi has one of -1, 1, 2, or 3 */
-    if (_dbapi < -1 || _dbapi > 4)
-       _dbapi = -1;
-    if (_dbapi == 0)
-       _dbapi = 1;
-
     if (dbp)
        *dbp = NULL;
     if ((mode & O_ACCMODE) == O_WRONLY) 
@@ -773,8 +766,6 @@ static int openDatabase(const char * prefix,
     if (db == NULL)
        return 1;
 
-    db->db_api = _dbapi;
-
     /* Try to ensure db home exists, error out if we cant even create */
     rc = rpmioMkpath(rpmdbHome(db), 0755, getuid(), getgid());
     if (rc == 0) {
@@ -816,17 +807,15 @@ rpmdb rpmdbLink(rpmdb db)
 /* XXX python/rpmmodule.c */
 int rpmdbOpen (const char * prefix, rpmdb *dbp, int mode, int perms)
 {
-    int _dbapi = rpmExpandNumeric("%{_dbapi}");
-    return openDatabase(prefix, NULL, _dbapi, dbp, mode, perms, 0);
+    return openDatabase(prefix, NULL, dbp, mode, perms, 0);
 }
 
 int rpmdbInit (const char * prefix, int perms)
 {
     rpmdb db = NULL;
-    int _dbapi = rpmExpandNumeric("%{_dbapi}");
     int rc;
 
-    rc = openDatabase(prefix, NULL, _dbapi, &db, (O_CREAT | O_RDWR), perms, 0);
+    rc = openDatabase(prefix, NULL, &db, (O_CREAT | O_RDWR), perms, 0);
     if (db != NULL) {
        int xx;
        xx = rpmdbOpenAll(db);
@@ -841,11 +830,9 @@ int rpmdbInit (const char * prefix, int perms)
 int rpmdbVerify(const char * prefix)
 {
     rpmdb db = NULL;
-    int _dbapi = rpmExpandNumeric("%{_dbapi}");
     int rc = 0;
 
-    rc = openDatabase(prefix, NULL, _dbapi, &db, O_RDONLY, 0644,
-                       RPMDB_FLAG_VERIFYONLY);
+    rc = openDatabase(prefix, NULL, &db, O_RDONLY, 0644, RPMDB_FLAG_VERIFYONLY);
 
     if (db != NULL) {
        int xx;
@@ -2969,14 +2956,14 @@ 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, 0)) {
+    if (openDatabase(prefix, dbpath, &olddb, O_RDONLY, 0644, 0)) {
        rc = 1;
        goto exit;
     }
     _dbapi = olddb->db_api;
     rpmlog(RPMLOG_DEBUG, "opening new database with dbapi %d\n",
                _dbapi_rebuild);
-    if (openDatabase(prefix, newdbpath, _dbapi_rebuild, &newdb,
+    if (openDatabase(prefix, newdbpath, &newdb,
                     (O_RDWR | O_CREAT), 0644, RPMDB_FLAG_REBUILD)) {
        rc = 1;
        goto exit;