[geary/mjog/db-result-timing: 4/8] Geary.Db.DatabaseConnection: Use a prepared prepared statement for exec

[Michael Gratton <mjog src gnome org>]

commit 97749c55b2ac067c52f1fa5e396dc03039f891ed
Author: Michael Gratton <mike vee net>
Date:   Tue Sep 8 08:34:23 2020 +1000

    Geary.Db.DatabaseConnection: Use a prepared prepared statement for exec
    
    Rather than just passing SQL to SQLite in `exec`, use a prepared
    statement and drain its results so we get elapsed time warnings for
    that as well.

 src/engine/db/db-database-connection.vala | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)
---
diff --git a/src/engine/db/db-database-connection.vala b/src/engine/db/db-database-connection.vala
index dd311bea6..1c5301e7b 100644
--- a/src/engine/db/db-database-connection.vala
+++ b/src/engine/db/db-database-connection.vala
@@ -137,7 +137,11 @@ public class Geary.Db.DatabaseConnection : Context, Connection {
         }
 
         check_cancelled("Connection.exec", cancellable);
-        throw_on_error("Connection.exec", db.exec(sql), sql);
+        check_cancelled("Connection.exec", cancellable);
+        var result = prepare(sql).exec(cancellable);
+        while (!result.finished) {
+            result.next(cancellable);
+        }
     }
 
     /** {@inheritDoc} */