Return change count for Statement:executeUpdate too. (firebird-sdbc)

This also simplifies Statement, although not using execute_immediate
is somewhat less efficient.

Change-Id: I4bebe13e5a63cb02e6b2659dec534f0c6feeba52

1 files changed, 2 insertions, 22 deletions

diff --git a/connectivity/source/drivers/firebird/Statement.cxx b/connectivity/source/drivers/firebird/Statement.cxx
index 59f89de4a2df..00978b708d62 100644
--- a/connectivity/source/drivers/firebird/Statement.cxx
+++ b/connectivity/source/drivers/firebird/Statement.cxx
@@ -97,28 +97,8 @@ void OStatement::disposeResultSet()
 sal_Int32 SAL_CALL OStatement::executeUpdate(const OUString& sql)
     throw(SQLException, RuntimeException)
 {
-    MutexGuard aGuard(m_pConnection->getMutex());
-    checkDisposed(OStatementCommonBase_Base::rBHelper.bDisposed);
-
-    SAL_INFO("connectivity.firebird", "executeUpdate(" << sql << ")");
-
-    int aErr = isc_dsql_execute_immediate(m_statusVector,
-                                          &m_pConnection->getDBHandle(),
-                                          &m_pConnection->getTransaction(),
-                                          0,
-                                          OUStringToOString(sql, RTL_TEXTENCODING_UTF8).getStr(),
-                                          FIREBIRD_SQL_DIALECT,
-                                          NULL);
-
-    if (aErr)
-        SAL_WARN("connectivity.firebird", "isc_dsql_execute_immediate failed" );
-
-    evaluateStatusVector(m_statusVector, sql, *this);
-    // TODO: get number of changed rows with SELECT ROW_COUNT (use executeQuery)
-    //     return getUpdateCount();
-    // We can't use     return getStatementChangeCount(); since that depends
-    // on having the statement handle, so maybe just use executeQuery instead?
-    return 0;
+    execute(sql);
+    return getStatementChangeCount();
 }