Make dtors not throw exceptions. (src dir).

iwongu · May 3, 2016 · 9a594a6 · 9a594a6
1 parent aa23baa
commit 9a594a6
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 10 deletions.
diff --git a/src/sqlite3pp.cpp b/src/sqlite3pp.cpp
@@ -234,11 +234,11 @@ namespace sqlite3pp
     }
   }
 
-  statement::~statement() noexcept(false)
+  statement::~statement()
   {
-    auto rc = finish();
-    if (rc != SQLITE_OK)
-      throw database_error(db_);
+    // finish() can return error. If you want to check the error, call
+    // finish() explicitly before this object is destructed.
+    finish();
   }
 
   int statement::prepare(char const* stmt)
@@ -551,12 +551,13 @@ namespace sqlite3pp
       throw database_error(*db_);
   }
 
-  transaction::~transaction() noexcept(false)
+  transaction::~transaction()
   {
     if (db_) {
-      auto rc = db_->execute(fcommit_ ? "COMMIT" : "ROLLBACK");
-      if (rc != SQLITE_OK)
-	throw database_error(*db_);
+      // execute() can return error. If you want to check the error,
+      // call commit() or rollback() explicitly before this object is
+      // destructed.
+      db_->execute(fcommit_ ? "COMMIT" : "ROLLBACK");
     }
   }
 

diff --git a/src/sqlite3pp.h b/src/sqlite3pp.h
@@ -164,7 +164,7 @@ namespace sqlite3pp
 
    protected:
     explicit statement(database& db, char const* stmt = nullptr);
-    ~statement() noexcept(false);
+    ~statement();
 
     int prepare_impl(char const* stmt);
     int finish_impl(sqlite3_stmt* stmt);
@@ -312,7 +312,7 @@ namespace sqlite3pp
   {
    public:
     explicit transaction(database& db, bool fcommit = false, bool freserve = false);
-    ~transaction() noexcept(false);
+    ~transaction();
 
     int commit();
     int rollback();