diff options
author | Stephan Bergmann <sbergman@redhat.com> | 2015-09-08 15:34:46 +0200 |
---|---|---|
committer | Stephan Bergmann <sbergman@redhat.com> | 2015-09-08 15:34:46 +0200 |
commit | bd53697bab53d46df952eeb23b609c59adc330cc (patch) | |
tree | 8d9f32fedc60b91aa256c184366b422188844533 /connectivity | |
parent | 4e95a613dc5cc836d0c9582fd647d676107d1786 (diff) |
clang-analyzer-deadcode.DeadStores
...ever since OResultSet::fillNeededData got introduced in
8687beec5dfc08b28accc7779e0f7762cd947f12 "INTEGRATION: CWS insight01" (and the
return value of the N3SQLPutData call in the first case block had always been
ignored completely, not even unnecessarily assigned to nRet as in these later
two case blocks)
Change-Id: I1d2005e7c91749510b23f86ac792bd11ae564608
Diffstat (limited to 'connectivity')
-rw-r--r-- | connectivity/source/drivers/odbc/OResultSet.cxx | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/connectivity/source/drivers/odbc/OResultSet.cxx b/connectivity/source/drivers/odbc/OResultSet.cxx index f8136680d9cc..b6feb2f358d9 100644 --- a/connectivity/source/drivers/odbc/OResultSet.cxx +++ b/connectivity/source/drivers/odbc/OResultSet.cxx @@ -1805,7 +1805,7 @@ void OResultSet::fillNeededData(SQLRETURN _nRet) { OUString sRet; sRet = m_aRow[nColumnIndex].getString(); - nRet = N3SQLPutData (m_aStatementHandle, static_cast<SQLPOINTER>(const_cast<sal_Unicode *>(sRet.getStr())), sizeof(sal_Unicode)*sRet.getLength()); + N3SQLPutData (m_aStatementHandle, static_cast<SQLPOINTER>(const_cast<sal_Unicode *>(sRet.getStr())), sizeof(sal_Unicode)*sRet.getLength()); break; } case DataType::LONGVARCHAR: @@ -1814,7 +1814,7 @@ void OResultSet::fillNeededData(SQLRETURN _nRet) OUString sRet; sRet = m_aRow[nColumnIndex].getString(); OString aString(OUStringToOString(sRet,m_nTextEncoding)); - nRet = N3SQLPutData (m_aStatementHandle, static_cast<SQLPOINTER>(const_cast<char *>(aString.getStr())), aString.getLength()); + N3SQLPutData (m_aStatementHandle, static_cast<SQLPOINTER>(const_cast<char *>(aString.getStr())), aString.getLength()); break; } default: |