Decomplicate SqliteDb::exec interface.

1 files changed, 20 insertions, 9 deletions

diff --git a/source/fud_sqlite.cpp b/source/fud_sqlite.cpp
index c607b37..f7193bf 100644
--- a/source/fud_sqlite.cpp
+++ b/source/fud_sqlite.cpp
@@ -138,29 +138,40 @@ int SqliteDb::open()
     return sqlite3_open_v2(m_name.c_str(), &m_dbHandle, static_cast<int>(m_mode) | m_extraFlags, nullptr);
 }
 
+FudStatus SqliteDb::exec(const String& statement, int (*callback)(void*, int, char**, char**), void* context)
+{
+    if (!valid()) {
+        return FudStatus::ObjectInvalid;
+    }
+
+    if (!statement.utf8Valid()) {
+        return FudStatus::Utf8Invalid;
+    }
+
+    m_errorCode = sqlite3_exec(m_dbHandle, statement.c_str(), callback, context, nullptr);
+
+    return m_errorCode == SQLITE_OK ? FudStatus::Success : FudStatus::Failure;
+}
+
 FudStatus SqliteDb::exec(
     const String& statement,
     int (*callback)(void*, int, char**, char**),
     void* context,
-    // NOLINTNEXTLINE(performance-copy-param-value)
-    Option<SqliteErrorMsg&>& errorMessage)
+    SqliteErrorMsg& errorMessage)
 {
     if (!valid()) {
         return FudStatus::ObjectInvalid;
     }
 
+    char* errorMsgPtr = nullptr;
+
     if (!statement.utf8Valid()) {
         return FudStatus::Utf8Invalid;
     }
 
-    char* errorMsgPtr = nullptr;
-    char** errorMsgPtrAddress = errorMessage.isNone() ? nullptr : &errorMsgPtr;
-
-    m_errorCode = sqlite3_exec(m_dbHandle, statement.c_str(), callback, context, errorMsgPtrAddress);
+    m_errorCode = sqlite3_exec(m_dbHandle, statement.c_str(), callback, context, &errorMsgPtr);
 
-    if (errorMessage.hasValue()) {
-        errorMessage.value().setMessage(errorMsgPtr);
-    }
+    errorMessage.setMessage(errorMsgPtr);
 
     return m_errorCode == SQLITE_OK ? FudStatus::Success : FudStatus::Failure;
 }