Some cppcheck cleaning

Change-Id: I5e8c768981dc6b8029bdc7eb8887562ccc18a5e2

5 files changed, 4 insertions, 12 deletions

diff --git a/connectivity/source/drivers/ado/AColumn.cxx b/connectivity/source/drivers/ado/AColumn.cxx
index 4c4e4f23b546..ab50f2f5b3b9 100644
--- a/connectivity/source/drivers/ado/AColumn.cxx
+++ b/connectivity/source/drivers/ado/AColumn.cxx
@@ -39,10 +39,8 @@ using namespace com::sun::star::sdbc;
 
 void WpADOColumn::Create()
 {
-    HRESULT         hr = -1;
-
     _ADOColumn* pColumn = NULL;
-    hr = CoCreateInstance(ADOS::CLSID_ADOCOLUMN_25,
+    HRESULT hr = CoCreateInstance(ADOS::CLSID_ADOCOLUMN_25,
                           NULL,
                           CLSCTX_INPROC_SERVER,
                           ADOS::IID_ADOCOLUMN_25,
diff --git a/connectivity/source/drivers/ado/ADatabaseMetaDataResultSet.cxx b/connectivity/source/drivers/ado/ADatabaseMetaDataResultSet.cxx
index 61ec5d5f6e43..ed48af3963c2 100644
--- a/connectivity/source/drivers/ado/ADatabaseMetaDataResultSet.cxx
+++ b/connectivity/source/drivers/ado/ADatabaseMetaDataResultSet.cxx
@@ -728,7 +728,6 @@ sal_Bool ODatabaseMetaDataResultSet::convertFastPropertyValue(
         case PROPERTY_ID_RESULTSETCONCURRENCY:
         case PROPERTY_ID_RESULTSETTYPE:
             throw ::com::sun::star::lang::IllegalArgumentException();
-            break;
         case PROPERTY_ID_FETCHDIRECTION:
             return ::comphelper::tryPropertyValue(rConvertedValue, rOldValue, rValue, getFetchDirection());
         case PROPERTY_ID_FETCHSIZE:
@@ -753,7 +752,6 @@ void ODatabaseMetaDataResultSet::setFastPropertyValue_NoBroadcast(
         case PROPERTY_ID_FETCHDIRECTION:
         case PROPERTY_ID_FETCHSIZE:
             throw Exception();
-            break;
         default:
             OSL_FAIL("setFastPropertyValue_NoBroadcast: Illegal handle value!");
     }
diff --git a/connectivity/source/drivers/ado/AGroup.cxx b/connectivity/source/drivers/ado/AGroup.cxx
index a7192db7c7ab..ea54a1ff035c 100644
--- a/connectivity/source/drivers/ado/AGroup.cxx
+++ b/connectivity/source/drivers/ado/AGroup.cxx
@@ -37,9 +37,8 @@ using namespace com::sun::star::sdbcx;
 // -------------------------------------------------------------------------
 void WpADOGroup::Create()
 {
-    HRESULT         hr = -1;
     ADOGroup* pGroup = NULL;
-    hr = CoCreateInstance(ADOS::CLSID_ADOGROUP_25,
+    HRESULT hr = CoCreateInstance(ADOS::CLSID_ADOGROUP_25,
                           NULL,
                           CLSCTX_INPROC_SERVER,
                           ADOS::IID_ADOGROUP_25,
diff --git a/connectivity/source/drivers/ado/Awrapado.cxx b/connectivity/source/drivers/ado/Awrapado.cxx
index 08535858cf55..b0cb037d8f27 100644
--- a/connectivity/source/drivers/ado/Awrapado.cxx
+++ b/connectivity/source/drivers/ado/Awrapado.cxx
@@ -36,9 +36,8 @@ using namespace connectivity::ado;
 
 void WpADOCatalog::Create()
 {
-    HRESULT         hr = -1;
     _ADOCatalog* pCommand;
-    hr = CoCreateInstance(ADOS::CLSID_ADOCATALOG_25,
+    HRESULT hr = CoCreateInstance(ADOS::CLSID_ADOCATALOG_25,
                           NULL,
                           CLSCTX_INPROC_SERVER,
                           ADOS::IID_ADOCATALOG_25,
diff --git a/connectivity/source/drivers/postgresql/pq_xcolumns.cxx b/connectivity/source/drivers/postgresql/pq_xcolumns.cxx
index 7d34de27b623..e0bd4adfec72 100644
--- a/connectivity/source/drivers/postgresql/pq_xcolumns.cxx
+++ b/connectivity/source/drivers/postgresql/pq_xcolumns.cxx
@@ -119,9 +119,7 @@ static Any isCurrency( const rtl::OUString & typeName )
 
 static Any isAutoIncrement( const rtl::OUString & defaultValue )
 {
-    sal_Bool ret = sal_False;
-
-    ret = defaultValue.matchAsciiL( RTL_CONSTASCII_STRINGPARAM( "nextval(" ) );
+    sal_Bool ret = defaultValue.matchAsciiL( RTL_CONSTASCII_STRINGPARAM( "nextval(" ) );
 //     printf( "%s %d\n",
 //             OUStringToOString(defaultValue, RTL_TEXTENCODING_ASCII_US).getStr(),
 //             ret );