Browse Source

[2378] Simplify the loadIncremental code

Some of the code can be shared between the variants, saving some code.
Michal 'vorner' Vaner 12 years ago
parent
commit
27033b59dc
1 changed files with 9 additions and 17 deletions
  1. 9 17
      src/lib/datasrc/zone_loader.cc

+ 9 - 17
src/lib/datasrc/zone_loader.cc

@@ -110,26 +110,18 @@ ZoneLoader::loadIncremental(size_t limit) {
 
     if (iterator_ == ZoneIteratorPtr()) {
         assert(loader_.get() != NULL);
-        if (loader_->loadIncremental(limit)) {
-            complete_ = true;
-            if (!loaded_ok_) {
-                isc_throw(MasterFileError, "Error while loading master file");
-            } else {
-                updater_->commit();
-            }
-            return (true);
-        } else {
-            return (false);
+        complete_ = loader_->loadIncremental(limit);
+        if (complete_ && !loaded_ok_) {
+            isc_throw(MasterFileError, "Error while loading master file");
         }
     } else {
-        if (copyRRsets(updater_, iterator_, limit)) {
-            updater_->commit();
-            complete_ = true;
-            return (true);
-        } else {
-            return (false);
-        }
+        complete_ = copyRRsets(updater_, iterator_, limit);
+    }
+
+    if (complete_) {
+        updater_->commit();
     }
+    return (complete_);
 }
 
 } // end namespace datasrc