[Libreoffice-commits] core.git: 2 commits - include/vcl mysqlc/source vcl/source
Noel Grandin
noel.grandin at collabora.co.uk
Mon Apr 23 06:14:46 UTC 2018
include/vcl/dockwin.hxx | 2 -
mysqlc/source/mysqlc_preparedstatement.cxx | 38 ++++++++++++++---------------
mysqlc/source/mysqlc_statement.cxx | 5 +--
mysqlc/source/mysqlc_statement.hxx | 2 -
vcl/source/window/dockwin.cxx | 5 +--
5 files changed, 25 insertions(+), 27 deletions(-)
New commits:
commit 18c2742946a745d5311a1c26bda352cefc6c1ac0
Author: Noel Grandin <noel.grandin at collabora.co.uk>
Date: Mon Apr 16 08:45:51 2018 +0200
loplugin:useuniqueptr in mysqlc
Change-Id: Ife3a044beae03c7623c1bd3c2ef6ce5595f97cca
Reviewed-on: https://gerrit.libreoffice.org/53223
Tested-by: Jenkins <ci at libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin at collabora.co.uk>
diff --git a/mysqlc/source/mysqlc_preparedstatement.cxx b/mysqlc/source/mysqlc_preparedstatement.cxx
index 4be52eb018d1..fae2a6642724 100644
--- a/mysqlc/source/mysqlc_preparedstatement.cxx
+++ b/mysqlc/source/mysqlc_preparedstatement.cxx
@@ -78,7 +78,7 @@ OPreparedStatement::OPreparedStatement(OConnection* _pConnection, sql::PreparedS
m_xConnection = _pConnection;
try {
- m_paramCount = static_cast<sql::PreparedStatement *>(cppStatement)->getParameterMetaData()->getParameterCount();
+ m_paramCount = static_cast<sql::PreparedStatement *>(cppStatement.get())->getParameterMetaData()->getParameterCount();
} catch (const sql::SQLException &e) {
mysqlc_sdbc_driver::translateAndThrow(e, *this, m_xConnection->getConnectionEncoding());
}
@@ -122,7 +122,7 @@ Reference< XResultSetMetaData > SAL_CALL OPreparedStatement::getMetaData()
try {
if (!m_xMetaData.is()) {
m_xMetaData = new OResultSetMetaData(
- static_cast<sql::PreparedStatement *>(cppStatement)->getMetaData(),
+ static_cast<sql::PreparedStatement *>(cppStatement.get())->getMetaData(),
getOwnConnection()->getConnectionEncoding()
);
}
@@ -158,7 +158,7 @@ sal_Bool SAL_CALL OPreparedStatement::execute()
bool success = false;
try {
- success = static_cast<sql::PreparedStatement *>(cppStatement)->execute();
+ success = static_cast<sql::PreparedStatement *>(cppStatement.get())->execute();
} catch (const sql::SQLException &e) {
mysqlc_sdbc_driver::translateAndThrow(e, *this, m_xConnection->getConnectionEncoding());
}
@@ -172,7 +172,7 @@ sal_Int32 SAL_CALL OPreparedStatement::executeUpdate()
sal_Int32 affectedRows = 0;
try {
- affectedRows = static_cast<sql::PreparedStatement *>(cppStatement)->executeUpdate();
+ affectedRows = static_cast<sql::PreparedStatement *>(cppStatement.get())->executeUpdate();
} catch (const sql::SQLException &e) {
mysqlc_sdbc_driver::translateAndThrow(e, *this, m_xConnection->getConnectionEncoding());
}
@@ -187,7 +187,7 @@ void SAL_CALL OPreparedStatement::setString(sal_Int32 parameter, const rtl::OUSt
try {
std::string stringie(rtl::OUStringToOString(x, m_xConnection->getConnectionEncoding()).getStr());
- static_cast<sql::PreparedStatement *>(cppStatement)->setString(parameter, stringie);
+ static_cast<sql::PreparedStatement *>(cppStatement.get())->setString(parameter, stringie);
} catch (const sql::MethodNotImplementedException &) {
mysqlc_sdbc_driver::throwFeatureNotImplementedException("OPreparedStatement::clearParameters", *this);
} catch (const sql::SQLException &e) {
@@ -210,7 +210,7 @@ Reference< XResultSet > SAL_CALL OPreparedStatement::executeQuery()
Reference< XResultSet > xResultSet;
try {
- sql::ResultSet * res = static_cast<sql::PreparedStatement *>(cppStatement)->executeQuery();
+ sql::ResultSet * res = static_cast<sql::PreparedStatement *>(cppStatement.get())->executeQuery();
xResultSet = new OResultSet(this, res, getOwnConnection()->getConnectionEncoding());
} catch (const sql::SQLException &e) {
mysqlc_sdbc_driver::translateAndThrow(e, *this, m_xConnection->getConnectionEncoding());
@@ -225,7 +225,7 @@ void SAL_CALL OPreparedStatement::setBoolean(sal_Int32 parameter, sal_Bool x)
checkParameterIndex(parameter);
try {
- static_cast<sql::PreparedStatement *>(cppStatement)->setBoolean(parameter, x);
+ static_cast<sql::PreparedStatement *>(cppStatement.get())->setBoolean(parameter, x);
} catch (const sql::MethodNotImplementedException &) {
mysqlc_sdbc_driver::throwFeatureNotImplementedException("OPreparedStatement::setBoolean", *this);
} catch (const sql::SQLException &e) {
@@ -240,7 +240,7 @@ void SAL_CALL OPreparedStatement::setByte(sal_Int32 parameter, sal_Int8 x)
checkParameterIndex(parameter);
try {
- static_cast<sql::PreparedStatement *>(cppStatement)->setInt(parameter, x);
+ static_cast<sql::PreparedStatement *>(cppStatement.get())->setInt(parameter, x);
} catch (const sql::MethodNotImplementedException &) {
mysqlc_sdbc_driver::throwFeatureNotImplementedException("OPreparedStatement::setByte", *this);
} catch (const sql::SQLException &e) {
@@ -263,7 +263,7 @@ void SAL_CALL OPreparedStatement::setDate(sal_Int32 parameter, const Date& aData
dateStr.append(my_i_to_a(buf, sizeof(buf)-1, aData.Day));
try {
- static_cast<sql::PreparedStatement *>(cppStatement)->setDateTime(parameter, dateStr);
+ static_cast<sql::PreparedStatement *>(cppStatement.get())->setDateTime(parameter, dateStr);
} catch (const sql::MethodNotImplementedException &) {
mysqlc_sdbc_driver::throwFeatureNotImplementedException("OPreparedStatement::setDate", *this);
} catch (const sql::SQLException &e) {
@@ -286,7 +286,7 @@ void SAL_CALL OPreparedStatement::setTime(sal_Int32 parameter, const Time& aVal)
timeStr.append(my_i_to_a(buf, sizeof(buf)-1, aVal.Seconds));
try {
- static_cast<sql::PreparedStatement *>(cppStatement)->setDateTime(parameter, timeStr);
+ static_cast<sql::PreparedStatement *>(cppStatement.get())->setDateTime(parameter, timeStr);
} catch (const sql::MethodNotImplementedException &) {
mysqlc_sdbc_driver::throwFeatureNotImplementedException("OPreparedStatement::setTime", *this);
} catch (const sql::SQLException &e) {
@@ -317,7 +317,7 @@ void SAL_CALL OPreparedStatement::setTimestamp(sal_Int32 parameter, const DateTi
timeStr.append(my_i_to_a(buf, sizeof(buf)-1, aVal.Seconds));
try {
- static_cast<sql::PreparedStatement *>(cppStatement)->setDateTime(parameter, timeStr);
+ static_cast<sql::PreparedStatement *>(cppStatement.get())->setDateTime(parameter, timeStr);
} catch (const sql::MethodNotImplementedException &) {
mysqlc_sdbc_driver::throwFeatureNotImplementedException("OPreparedStatement::setTimestamp", *this);
} catch (const sql::SQLException &e) {
@@ -332,7 +332,7 @@ void SAL_CALL OPreparedStatement::setDouble(sal_Int32 parameter, double x)
checkParameterIndex(parameter);
try {
- static_cast<sql::PreparedStatement *>(cppStatement)->setDouble(parameter, x);
+ static_cast<sql::PreparedStatement *>(cppStatement.get())->setDouble(parameter, x);
} catch (const sql::MethodNotImplementedException &) {
mysqlc_sdbc_driver::throwFeatureNotImplementedException("OPreparedStatement::setDouble", *this);
} catch (const sql::SQLException &e) {
@@ -347,7 +347,7 @@ void SAL_CALL OPreparedStatement::setFloat(sal_Int32 parameter, float x)
checkParameterIndex(parameter);
try {
- static_cast<sql::PreparedStatement *>(cppStatement)->setDouble(parameter, x);
+ static_cast<sql::PreparedStatement *>(cppStatement.get())->setDouble(parameter, x);
} catch (const sql::MethodNotImplementedException &) {
mysqlc_sdbc_driver::throwFeatureNotImplementedException("OPreparedStatement::setFloat", *this);
} catch (const sql::SQLException &e) {
@@ -362,7 +362,7 @@ void SAL_CALL OPreparedStatement::setInt(sal_Int32 parameter, sal_Int32 x)
checkParameterIndex(parameter);
try {
- static_cast<sql::PreparedStatement *>(cppStatement)->setInt(parameter, x);
+ static_cast<sql::PreparedStatement *>(cppStatement.get())->setInt(parameter, x);
} catch (const sql::MethodNotImplementedException &) {
mysqlc_sdbc_driver::throwFeatureNotImplementedException("OPreparedStatement::setInt", *this);
} catch (const sql::SQLException &e) {
@@ -377,7 +377,7 @@ void SAL_CALL OPreparedStatement::setLong(sal_Int32 parameter, sal_Int64 aVal)
checkParameterIndex(parameter);
try {
- static_cast<sql::PreparedStatement *>(cppStatement)->setInt64(parameter, aVal);
+ static_cast<sql::PreparedStatement *>(cppStatement.get())->setInt64(parameter, aVal);
} catch (const sql::MethodNotImplementedException &) {
mysqlc_sdbc_driver::throwFeatureNotImplementedException("OPreparedStatement::setLong", *this);
} catch (const sql::SQLException &e) {
@@ -392,7 +392,7 @@ void SAL_CALL OPreparedStatement::setNull(sal_Int32 parameter, sal_Int32 sqlType
checkParameterIndex(parameter);
try {
- static_cast<sql::PreparedStatement *>(cppStatement)->setNull(parameter, sqlType);
+ static_cast<sql::PreparedStatement *>(cppStatement.get())->setNull(parameter, sqlType);
} catch (const sql::MethodNotImplementedException &) {
mysqlc_sdbc_driver::throwFeatureNotImplementedException("OPreparedStatement::setNull", *this);
} catch (const sql::SQLException &e) {
@@ -630,7 +630,7 @@ void SAL_CALL OPreparedStatement::setShort(sal_Int32 parameter, sal_Int16 x)
checkParameterIndex(parameter);
try {
- static_cast<sql::PreparedStatement *>(cppStatement)->setInt(parameter, x);
+ static_cast<sql::PreparedStatement *>(cppStatement.get())->setInt(parameter, x);
} catch (const sql::MethodNotImplementedException &) {
mysqlc_sdbc_driver::throwFeatureNotImplementedException("OPreparedStatement::setShort", *this);
} catch (const sql::SQLException &e) {
@@ -646,7 +646,7 @@ void SAL_CALL OPreparedStatement::setBytes(sal_Int32 parameter, const Sequence<
std::string blobby(reinterpret_cast<char const *>(x.getConstArray()), x.getLength());
try {
- static_cast<sql::PreparedStatement *>(cppStatement)->setString(parameter, blobby);
+ static_cast<sql::PreparedStatement *>(cppStatement.get())->setString(parameter, blobby);
} catch (const sql::MethodNotImplementedException &) {
mysqlc_sdbc_driver::throwFeatureNotImplementedException("OPreparedStatement::setBytes", *this);
} catch (const sql::SQLException &e) {
@@ -682,7 +682,7 @@ void SAL_CALL OPreparedStatement::clearParameters()
checkDisposed(OPreparedStatement::rBHelper.bDisposed);
try {
- static_cast<sql::PreparedStatement *>(cppStatement)->clearParameters();
+ static_cast<sql::PreparedStatement *>(cppStatement.get())->clearParameters();
} catch (const sql::MethodNotImplementedException &) {
mysqlc_sdbc_driver::throwFeatureNotImplementedException("OPreparedStatement::clearParameters", *this);
} catch (const sql::SQLException &e) {
diff --git a/mysqlc/source/mysqlc_statement.cxx b/mysqlc/source/mysqlc_statement.cxx
index 1d86127bd879..dc032fdc238d 100644
--- a/mysqlc/source/mysqlc_statement.cxx
+++ b/mysqlc/source/mysqlc_statement.cxx
@@ -67,8 +67,7 @@ OCommonStatement::~OCommonStatement()
void OCommonStatement::disposeResultSet()
{
// free the cursor if alive
- delete cppStatement;
- cppStatement = nullptr;
+ cppStatement.reset();
}
void OCommonStatement::disposing()
@@ -78,7 +77,7 @@ void OCommonStatement::disposing()
disposeResultSet();
m_xConnection.clear();
- delete cppStatement;
+ cppStatement.reset();
OCommonStatement_IBase::disposing();
}
diff --git a/mysqlc/source/mysqlc_statement.hxx b/mysqlc/source/mysqlc_statement.hxx
index 1bb4d2a8d7be..6ac561f1e447 100644
--- a/mysqlc/source/mysqlc_statement.hxx
+++ b/mysqlc/source/mysqlc_statement.hxx
@@ -66,7 +66,7 @@ namespace connectivity
protected:
rtl::Reference<OConnection> m_xConnection; // The owning Connection object
- sql::Statement *cppStatement;
+ std::unique_ptr<sql::Statement> cppStatement;
protected:
void disposeResultSet();
commit cd0a8777019391f515f4bfe6b77ad70fdfeca664
Author: Noel Grandin <noel.grandin at collabora.co.uk>
Date: Mon Apr 16 08:28:52 2018 +0200
loplugin:useuniqueptr in DockingWindow
Change-Id: I7a0fec27f50c58f02378a69db5d17c6f5eb62ee2
Reviewed-on: https://gerrit.libreoffice.org/53222
Tested-by: Jenkins <ci at libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin at collabora.co.uk>
diff --git a/include/vcl/dockwin.hxx b/include/vcl/dockwin.hxx
index d3d792d476a3..905500e8e908 100644
--- a/include/vcl/dockwin.hxx
+++ b/include/vcl/dockwin.hxx
@@ -214,7 +214,7 @@ class VCL_DLLPUBLIC DockingWindow
private:
VclPtr<FloatingWindow> mpFloatWin;
VclPtr<vcl::Window> mpOldBorderWin;
- ImplData* mpImplData;
+ std::unique_ptr<ImplData> mpImplData;
Point maFloatPos;
Point maDockPos;
Point maMouseOff;
diff --git a/vcl/source/window/dockwin.cxx b/vcl/source/window/dockwin.cxx
index d6892cc42d0f..f76a27bd6a1f 100644
--- a/vcl/source/window/dockwin.cxx
+++ b/vcl/source/window/dockwin.cxx
@@ -277,7 +277,7 @@ void DockingWindow::ImplInitDockingWindowData()
mpWindowImpl->mbDockWin = true;
mpFloatWin = nullptr;
mpOldBorderWin = nullptr;
- mpImplData = new ImplData;
+ mpImplData.reset(new ImplData);
mnTrackX = 0;
mnTrackY = 0;
mnTrackWidth = 0;
@@ -391,8 +391,7 @@ void DockingWindow::dispose()
Show( false, ShowFlags::NoFocusChange );
SetFloatingMode(false);
}
- delete mpImplData;
- mpImplData = nullptr;
+ mpImplData.reset();
mpFloatWin.clear();
mpOldBorderWin.clear();
mpDialogParent.clear();
More information about the Libreoffice-commits
mailing list