fix(windows): fix MultiRemove by using CHECK_SQL_OK macro (#772)

vmoroz · web-flow · commit 6f3f3ba2d319 · 2022-03-15T08:32:21.000+01:00
diff --git a/windows/code/DBStorage.cpp b/windows/code/DBStorage.cpp
@@ -521,7 +521,7 @@ std::optional<bool> DBStorage::DBTask::MultiRemove(sqlite3 *db,
     auto statement = StatementPtr{nullptr, &sqlite3_finalize};
     CHECK_SQL_OK(PrepareStatement(db, sql, &statement));
     for (int i = 0; i < argCount; i++) {
-        CHECK(BindString(statement, i + 1, keys[i]));
+        CHECK_SQL_OK(BindString(statement, i + 1, keys[i]));
     }
     for (;;) {
         auto stepResult = sqlite3_step(statement.get());

Original file line number	Diff line number	Diff line change
`@@ -521,7 +521,7 @@ std::optional<bool> DBStorage::DBTask::MultiRemove(sqlite3 *db,`
`521`	`521`	`auto statement = StatementPtr{nullptr, &sqlite3_finalize};`
`522`	`522`	`CHECK_SQL_OK(PrepareStatement(db, sql, &statement));`
`523`	`523`	`for (int i = 0; i < argCount; i++) {`
`524`		`- CHECK(BindString(statement, i + 1, keys[i]));`
	`524`	`+ CHECK_SQL_OK(BindString(statement, i + 1, keys[i]));`
`525`	`525`	`}`
`526`	`526`	`for (;;) {`
`527`	`527`	`auto stepResult = sqlite3_step(statement.get());`