summaryrefslogtreecommitdiff
path: root/libodbcpp/odbc-connection.cpp
diff options
context:
space:
mode:
authorDan Goodliffe <dan@randomdan.homeip.net>2019-02-24 15:29:12 +0000
committerDan Goodliffe <dan@randomdan.homeip.net>2019-02-24 15:29:12 +0000
commit881033464c13398f78be306d8d785616f0947896 (patch)
tree38a3a320b5dd90a0b377a1b69735546c3ed84057 /libodbcpp/odbc-connection.cpp
parentUpdate to new libdbpp interface (diff)
downloadlibdbpp-odbc-881033464c13398f78be306d8d785616f0947896.tar.bz2
libdbpp-odbc-881033464c13398f78be306d8d785616f0947896.tar.xz
libdbpp-odbc-881033464c13398f78be306d8d785616f0947896.zip
Bring inline with clang-tidy checks
Diffstat (limited to 'libodbcpp/odbc-connection.cpp')
-rw-r--r--libodbcpp/odbc-connection.cpp27
1 files changed, 14 insertions, 13 deletions
diff --git a/libodbcpp/odbc-connection.cpp b/libodbcpp/odbc-connection.cpp
index 9a4a82d..4e4a0a1 100644
--- a/libodbcpp/odbc-connection.cpp
+++ b/libodbcpp/odbc-connection.cpp
@@ -1,7 +1,7 @@
#include <sqlext.h>
#include <stdexcept>
-#include <stdio.h>
-#include <string.h>
+#include <cstdio>
+#include <cstring>
#include "odbc-connection.h"
#include "odbc-selectcommand.h"
#include "odbc-modifycommand.h"
@@ -10,8 +10,8 @@
NAMEDFACTORY("odbc", ODBC::Connection, DB::ConnectionFactory);
ODBC::Connection::Connection(const DSN& d) :
- env(0),
- conn(0),
+ env(nullptr),
+ conn(nullptr),
thinkDelStyle(DB::BulkDeleteUsingUsing),
thinkUpdStyle(DB::BulkUpdateUsingFromSrc)
{
@@ -56,25 +56,26 @@ ODBC::Connection::connectPost()
throw ConnectionError(dberr, SQL_HANDLE_DBC, conn);
}
char info[1024];
- dberr = SQLGetInfo(conn, SQL_DRIVER_NAME, (SQLCHAR*)info, sizeof(info), NULL);
+ dberr = SQLGetInfo(conn, SQL_DRIVER_NAME, (SQLCHAR*)info, sizeof(info), nullptr);
if (!SQL_SUCCEEDED(dberr)) {
throw ConnectionError(dberr, SQL_HANDLE_DBC, conn);
}
// Apply known DB specific tweaks
- if (strstr(info, "myodbc") != NULL) {
+ // NOLINTNEXTLINE(hicpp-no-array-decay)
+ if (strstr(info, "myodbc")) {
thinkDelStyle = DB::BulkDeleteUsingUsingAlias;
thinkUpdStyle = DB::BulkUpdateUsingJoin;
}
}
ODBC::Connection::Connection(const std::string & s) :
- env(0),
- conn(0),
+ env(nullptr),
+ conn(nullptr),
thinkDelStyle(DB::BulkDeleteUsingUsing),
thinkUpdStyle(DB::BulkUpdateUsingFromSrc)
{
connectPre();
- RETCODE dberr = SQLDriverConnect(conn, NULL, (SQLCHAR*)s.c_str(), s.length(), NULL, 0, NULL, SQL_DRIVER_NOPROMPT);
+ RETCODE dberr = SQLDriverConnect(conn, nullptr, (SQLCHAR*)s.c_str(), s.length(), nullptr, 0, nullptr, SQL_DRIVER_NOPROMPT);
if (!SQL_SUCCEEDED(dberr)) {
throw ConnectionError(dberr, SQL_HANDLE_DBC, conn);
}
@@ -84,11 +85,11 @@ ODBC::Connection::Connection(const std::string & s) :
ODBC::Connection::~Connection()
{
if (conn) {
- SQL_SUCCEEDED(SQLDisconnect(conn));
- SQL_SUCCEEDED(SQLFreeHandle(SQL_HANDLE_DBC, conn));
+ SQLDisconnect(conn);
+ SQLFreeHandle(SQL_HANDLE_DBC, conn);
}
if (env) {
- SQL_SUCCEEDED(SQLFreeHandle(SQL_HANDLE_ENV, env));
+ SQLFreeHandle(SQL_HANDLE_ENV, env);
}
}
@@ -169,7 +170,7 @@ SQLINTEGER
ODBC::Connection::getAttrInt(SQLINTEGER attr) const
{
SQLINTEGER result;
- SQLINTEGER dberr = SQLGetConnectAttr(conn, attr, &result, sizeof(result), 0);
+ SQLINTEGER dberr = SQLGetConnectAttr(conn, attr, &result, sizeof(result), nullptr);
if (!SQL_SUCCEEDED(dberr)) {
throw ODBC::Error(dberr, SQL_HANDLE_DBC, conn);
}