Move dbi_no_dbsync logic to inside dbiSync()
authorPanu Matilainen <pmatilai@redhat.com>
Thu, 1 Apr 2010 07:18:05 +0000 (10:18 +0300)
committerPanu Matilainen <pmatilai@redhat.com>
Thu, 1 Apr 2010 07:18:05 +0000 (10:18 +0300)
- avoids having to check for it in every single caller

lib/backend/db3.c
lib/rpmdb.c

index a7977de..f017833 100644 (file)
@@ -182,7 +182,7 @@ int dbiSync(dbiIndex dbi, unsigned int flags)
     DB * db = dbi->dbi_db;
     int rc = 0;
 
-    if (db != NULL) {
+    if (db != NULL && !dbi->dbi_no_dbsync) {
        rc = db->sync(db, flags);
        rc = cvtdberr(dbi, "db->sync", rc, _debug);
     }
index 3fdea75..3519667 100644 (file)
@@ -735,8 +735,6 @@ int rpmdbSync(rpmdb db)
        int xx;
        if (db->_dbi[dbix] == NULL)
            continue;
-       if (db->_dbi[dbix]->dbi_no_dbsync)
-           continue;
        xx = dbiSync(db->_dbi[dbix], 0);
        if (xx && rc == 0) rc = xx;
     }
@@ -2373,8 +2371,7 @@ int rpmdbRemove(rpmdb db, int rid, unsigned int hdrNum,
                    rc = dbiDel(dbi, dbcursor, &key, &data, 0);
                xx = dbiCclose(dbi, dbcursor, DB_WRITECURSOR);
                dbcursor = NULL;
-               if (!dbi->dbi_no_dbsync)
-                   xx = dbiSync(dbi, 0);
+               xx = dbiSync(dbi, 0);
                continue;
            }
        
@@ -2460,8 +2457,7 @@ cont:
            xx = dbiCclose(dbi, dbcursor, DB_WRITECURSOR);
            dbcursor = NULL;
 
-           if (!dbi->dbi_no_dbsync)
-               xx = dbiSync(dbi, 0);
+           xx = dbiSync(dbi, 0);
 
            rpmtdFreeData(&tagdata);
        }
@@ -2617,8 +2613,7 @@ int rpmdbAdd(rpmdb db, int iid, Header h,
                data.size = 0;
                xx = dbiCclose(dbi, dbcursor, DB_WRITECURSOR);
                dbcursor = NULL;
-               if (!dbi->dbi_no_dbsync)
-                   xx = dbiSync(dbi, 0);
+               xx = dbiSync(dbi, 0);
                continue;
                break;
            case RPMTAG_REQUIRENAME:
@@ -2731,8 +2726,7 @@ cont:
            xx = dbiCclose(dbi, dbcursor, DB_WRITECURSOR);
            dbcursor = NULL;
 
-           if (!dbi->dbi_no_dbsync)
-               xx = dbiSync(dbi, 0);
+           xx = dbiSync(dbi, 0);
 
            rpmtdFreeData(&tagdata);
        }