BIND 10 trac2821, updated. b11b84e52dd4670259d7c0a59d2e7cee4550b16a [2821] Updated documentation related to MySqlHolder class.

[BIND 10 source code commits]

The branch, trac2821 has been updated
       via  b11b84e52dd4670259d7c0a59d2e7cee4550b16a (commit)
      from  e58c2fd32cb85d37cd96ff8bb6a7cdf0f653a5a1 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit b11b84e52dd4670259d7c0a59d2e7cee4550b16a
Author: Stephen Morris <stephen at isc.org>
Date:   Thu Mar 7 23:00:23 2013 +0000

    [2821] Updated documentation related to MySqlHolder class.

-----------------------------------------------------------------------

Summary of changes:
 src/lib/dhcpsrv/mysql_lease_mgr.cc |    3 +++
 src/lib/dhcpsrv/mysql_lease_mgr.h  |   31 +++++++++++++++++++++++++++----
 2 files changed, 30 insertions(+), 4 deletions(-)

-----------------------------------------------------------------------
diff --git a/src/lib/dhcpsrv/mysql_lease_mgr.cc b/src/lib/dhcpsrv/mysql_lease_mgr.cc
index f029189..1264ff1 100644
--- a/src/lib/dhcpsrv/mysql_lease_mgr.cc
+++ b/src/lib/dhcpsrv/mysql_lease_mgr.cc
@@ -908,6 +908,9 @@ MySqlLeaseMgr::~MySqlLeaseMgr() {
             statements_[i] = NULL;
         }
     }
+
+    // There is no need to close the database in this destructor: it is
+    // closed in the destructor of the mysql_ member variable.
 }
 
 
diff --git a/src/lib/dhcpsrv/mysql_lease_mgr.h b/src/lib/dhcpsrv/mysql_lease_mgr.h
index 253f30f..e3bc040 100644
--- a/src/lib/dhcpsrv/mysql_lease_mgr.h
+++ b/src/lib/dhcpsrv/mysql_lease_mgr.h
@@ -19,6 +19,7 @@
 #include <dhcpsrv/lease_mgr.h>
 
 #include <boost/scoped_ptr.hpp>
+#include <boost/utility.hpp>
 #include <mysql/mysql.h>
 
 #include <time.h>
@@ -26,28 +27,50 @@
 namespace isc {
 namespace dhcp {
 
+/// @brief MySQL Handle Holder
+///
 /// Small RAII object for safer initialization, will close the database
-/// connection upon destruction
-class MySqlHolder {
+/// connection upon destruction.  This means that if an exception is thrown
+/// during database initialization, resources allocated to the database are
+/// guaranteed to be freed.
+///
+/// It makes no sense to copy an object of this class.  After the copy, both
+/// objects would contain pointers to the same MySql context object.  The
+/// destruction of one would invalid the context in the remaining object.
+/// For this reason, the class is declared noncopyable.
+class MySqlHolder : public boost::noncopyable {
 public:
+
+    /// @brief Constructor
+    ///
+    /// Initialize MySql and store the associated context object.
+    ///
+    /// @throw DbOpenError Unable to initialize MySql handle.
     MySqlHolder() : mysql_(mysql_init(NULL)) {
         if (mysql_ == NULL) {
             isc_throw(DbOpenError, "unable to initialize MySQL");
         }
     }
 
+    /// @brief Destructor
+    ///
+    /// Frees up resources allocated by the initialization of MySql.
     ~MySqlHolder() {
-        if (mysql_) {
+        if (mysql_ != NULL) {
             mysql_close(mysql_);
         }
     }
 
+    /// @brief Conversion Operator
+    ///
+    /// Allows the MySqlHolder object to be passed as the context argument to
+    /// mysql_xxx functions.
     operator MYSQL*() const {
         return (mysql_);
     }
 
 private:
-    MYSQL* mysql_;
+    MYSQL* mysql_;      ///< Initialization context
 };
 
 // Define the current database schema values