Use finalizer() on SQLStmts again but us gc() before closing dbs.

Author: Sean1708

src/SQLite.jl

@@ -60,6 +60,8 @@ end
 
 function Base.close{T}(db::SQLiteDB{T})
     db.handle == C_NULL && return
+    # ensure SQLiteStmts are finalised
+    gc()
     @CHECK db sqlite3_close(db.handle)
     db.handle = C_NULL
     return
@@ -80,6 +82,7 @@ function SQLiteStmt{T}(db::SQLiteDB{T},sql::String)
     handle = [C_NULL]
     sqliteprepare(db,sql,handle,[C_NULL])
     stmt = SQLiteStmt(db,handle[1],convert(T,sql))
+    finalizer(stmt, close)
     return stmt
 end
 
@@ -128,7 +131,6 @@ end
 function execute(db::SQLiteDB,sql::String)
     stmt = SQLiteStmt(db,sql)
     execute(stmt)
-    close(stmt)
     return changes(db)
 end
 
@@ -139,7 +141,6 @@ function query(db::SQLiteDB,sql::String)
     status = execute(stmt)
     ncols = sqlite3_column_count(stmt.handle)
     if status == SQLITE_DONE || ncols == 0
-        close(stmt)
         return changes(db)
     end
     colnames = Array(String,ncols)
@@ -171,7 +172,6 @@ function query(db::SQLiteDB,sql::String)
         end
         status = sqlite3_step(stmt.handle)
     end
-    close(stmt)
     if status == SQLITE_DONE
         return ResultSet(colnames, results)
     else
@@ -274,7 +274,6 @@ function create(db::SQLiteDB,name::String,table,
             end
             execute(stmt)
         end
-        close(stmt)
     end
     execute(db,"analyze $name")
     return changes(db)
@@ -303,7 +302,6 @@ function append(db::SQLiteDB,name::String,table)
             end
             execute(stmt)
         end
-        close(stmt)
     end
     execute(db,"analyze $name")
     return return changes(db)