Changeset 15584 for gsdl/trunk/src/colservr/gdbmsource.cpp

Timestamp:

2008-05-20T10:22:16+12:00 (16 years ago)

Author:

mdewsnip

Message:

(Adding new DB support) Changed the gdbmsource classes to use dbclass instead of gdbmclass, as part of reducing the dependence on gdbm.

File:

• gsdl/trunk/src/colservr/gdbmsource.cpp (modified) (15 diffs)

gsdl/trunk/src/colservr/gdbmsource.cpp

@@ -31 +31 @@
 
 gdbmsourceclass::gdbmsourceclass () {
-  gdbmptr = NULL;
+  db_ptr = NULL;
   textsearchptr = NULL;
   classname = "gdbmsource";
@@ -37 +37 @@
 
 gdbmsourceclass::~gdbmsourceclass () {
-  if (gdbmptr != NULL) delete gdbmptr;
+  if (db_ptr != NULL) delete db_ptr;
   if (textsearchptr != NULL) delete textsearchptr;
 }
@@ -117 +117 @@
     indexstem = collection;
   }
-  gdbm_filename = filename_cat(gdbmhome, "collect", collection, "index", "text", indexstem);
-  if (littleEndian()) gdbm_filename += ".ldb";
-  else gdbm_filename += ".bdb";
-
-  if (!file_exists(gdbm_filename)) {
+  db_filename = filename_cat(gdbmhome, "collect", collection, "index", "text", indexstem);
+  if (littleEndian()) db_filename += ".ldb";
+  else db_filename += ".bdb";
+
+  if (!file_exists(db_filename)) {
     logout << text_t2ascii
-       << "warning: gdbm database \"" //****
-       << gdbm_filename << "\" does not exist\n\n";
-    // return false; //****
+       << "warning: database \"" << db_filename << "\" does not exist\n\n";
+    // return false;
   }
   
@@ -137 +136 @@
 
   err = noError;
-  if (gdbmptr == NULL) {
+  if (db_ptr == NULL) {
     // most likely a configuration problem
     logout << text_t2ascii 
-       << "configuration error: " << classname << " contains a null gdbmclass\n\n";
+       << "configuration error: " << classname << " contains a null dbclass\n\n";
     err = configurationError;
     return true;
@@ -146 +145 @@
 
   // open the database
-  gdbmptr->setlogout(&logout);
-  if (!gdbmptr->opendatabase (gdbm_filename, GDBM_READER, 100, false)) {
-    // most likely a system problem (we have already checked that the
-    // gdbm database exists)
+  db_ptr->setlogout(&logout);
+  if (!db_ptr->opendatabase (db_filename, DB_READER, 100, false)) {
+    // most likely a system problem (we have already checked that the database exists)
     logout << text_t2ascii
-       << "system problem: open on gdbm database \""
-       << gdbm_filename << "\" failed\n\n";
+       << "system problem: open on database \"" << db_filename << "\" failed\n\n";
     err = systemProblem;
     return true;
@@ -158 +155 @@
 
   infodbclass info;
-  OIDout = gdbmptr->translate_OID (OIDin, info);
-  gdbmptr->closedatabase();  // Important that local library doesn't leave any files open
+  OIDout = db_ptr->translate_OID (OIDin, info);
+  db_ptr->closedatabase();  // Important that local library doesn't leave any files open
   return true;
 }
@@ -172 +169 @@
 
   err = noError;
-  if (gdbmptr == NULL) {
+  if (db_ptr == NULL) {
     // most likely a configuration problem
     logout << text_t2ascii 
-       << "configuration error: " << classname <<" contains a null gdbmclass\n\n";
+       << "configuration error: " << classname <<" contains a null dbclass\n\n";
     err = configurationError;
     return true;
@@ -181 +178 @@
 
   // open the database
-  gdbmptr->setlogout(&logout);
-  if (!gdbmptr->opendatabase (gdbm_filename, GDBM_READER, 100, false)) {
-    // most likely a system problem (we have already checked that the
-    // gdbm database exists)
+  db_ptr->setlogout(&logout);
+  if (!db_ptr->opendatabase (db_filename, DB_READER, 100, false)) {
+    // most likely a system problem (we have already checked that the database exists)
     logout << text_t2ascii
-       << "system problem: open on gdbm database \""
-       << gdbm_filename << "\" failed\n\n";
+       << "system problem: open on database \"" << db_filename << "\" failed\n\n";
     err = systemProblem;
     return true;
@@ -204 +199 @@
   while (this_OID != end_OID) {
     infodbclass info;
-    if (!gdbmptr->getinfo(*this_OID, info)) return false;
+    if (!db_ptr->getinfo(*this_OID, info)) return false;
 
     // adjust the metadata
@@ -260 +255 @@
           parentOID = thisparent;
           parentcontents.erase(parentcontents.begin(), parentcontents.end());
-          if (gdbmptr->getinfo(parentOID, parentinfo)) {
+          if (db_ptr->getinfo(parentOID, parentinfo)) {
         text_t &parentinfocontains = parentinfo["contains"];
         if (!parentinfocontains.empty())
@@ -316 +311 @@
   }
 
-  gdbmptr->closedatabase();  // Important that local library doesn't leave any files open
+  db_ptr->closedatabase();  // Important that local library doesn't leave any files open
   return true;
 }
@@ -327 +322 @@
 
   err = noError;
-  if (gdbmptr == NULL) {
+  if (db_ptr == NULL) {
     // most likely a configuration problem
     logout << text_t2ascii 
-       << "configuration error: " << classname << " contains a null gdbmclass\n\n";
+       << "configuration error: " << classname << " contains a null dbclass\n\n";
     err = configurationError;
     return true;
@@ -336 +331 @@
 
   // open the database
-  gdbmptr->setlogout(&logout);
-  if (!gdbmptr->opendatabase (gdbm_filename, GDBM_READER, 100, false)) {
-    // most likely a system problem (we have already checked that the
-    // gdbm database exists)
+  db_ptr->setlogout(&logout);
+  if (!db_ptr->opendatabase (db_filename, DB_READER, 100, false)) {
+    // most likely a system problem (we have already checked that the database exists)
     logout << text_t2ascii
-       << "system problem: open on gdbm database \""
-       << gdbm_filename << "\" failed\n\n";
+       << "system problem: open on database \"" << db_filename << "\" failed\n\n";
     err = systemProblem;
     return true;
@@ -351 +344 @@
     translate_OID (OID, tOID, err, logout);
   infodbclass info;
-  if (!gdbmptr->getinfo(tOID, info)) {
-    gdbmptr->closedatabase();  // Important that local library doesn't leave any files open
+  if (!db_ptr->getinfo(tOID, info)) {
+    db_ptr->closedatabase();  // Important that local library doesn't leave any files open
     return false;
   }
@@ -367 +360 @@
   }
 
-  gdbmptr->closedatabase();  // Important that local library doesn't leave any files open
+  db_ptr->closedatabase();  // Important that local library doesn't leave any files open
   return true;
 }