Addition to fix #31.

dlang-community · Nov 2, 2016 · 9963732 · 9963732
1 parent 66f1810
commit 9963732
Showing 1 changed file with 16 additions and 2 deletions.
diff --git a/source/d2sqlite3.d b/source/d2sqlite3.d
@@ -1371,8 +1371,22 @@ private:
         ~this()
         {
             auto result = sqlite3_finalize(handle);
-            enforce(result == SQLITE_OK, new SqliteException(errmsg(handle), result));
-            handle = null;
+
+			// Check that destructor was not call by the GC
+			// See https://p0nce.github.io/d-idioms/#GC-proof-resource-class
+			try
+			{
+                enforce(result == SQLITE_OK, new SqliteException(errmsg(handle), result));
+			}
+			catch (InvalidMemoryOperationError e)
+		    {
+		        import core.stdc.stdio;
+		        fprintf(stderr, "Error: release of Statement resource incorrectly"
+		                        ~ " depends on destructors called by the GC.\n");
+		        assert(false); // crash
+		    }
+
+			handle = null;
         }
 
         @disable this(this);