diff options
| author | Stephan Bergmann <sbergman@redhat.com> | 2020-05-28 08:26:51 +0200 |
|---|---|---|
| committer | Stephan Bergmann <sbergman@redhat.com> | 2020-05-28 09:48:09 +0200 |
| commit | e84d05483cc5f30161dc5b15f93fb3bf71ed636e (patch) | |
| tree | 2a343319cb4c1e06060b632de4fc840ba96bb70f /connectivity | |
| parent | 412ff42bf0a6d078b9a6fd1daa8d0eb0b9a78c4a (diff) | |
Make loplugin:simplifypointertobool handle parenthesized expressions
...as discussed as an open TODO in the commit message of
fe6cce01c88d045a1fcf09acf049c34c22299b02 "Fix loplugin:simplifypointertobool for
libstdc++ std::shared_ptr". The necessary changes across the code base have
been done fully automatically with the rewriting plugin on Linux. (All those
changes apparently involve uses of macro arguments wrapped in parentheses in the
macro body, but always in conditionally-converted-to-bool contexts. In other
contexts, such automatic rewriting would add the "bool" to the macro body, which
would be wrong in general, but we apparently get away with that sloppy coding
for now.)
The parenExprs_ stack that fe6cce01c88d045a1fcf09acf049c34c22299b02 had
introduced to treat such (then-undetected, it had turned out) parenthesized
cases now turns out to not be needed after all.
Change-Id: I2021f61c2e2805be7e18b38edf8744d186cac3cb
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/95010
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
Diffstat (limited to 'connectivity')
| -rw-r--r-- | connectivity/source/drivers/hsqldb/HStorageAccess.cxx | 4 | ||||
| -rw-r--r-- | connectivity/source/drivers/hsqldb/StorageNativeInputStream.cxx | 4 |
2 files changed, 4 insertions, 4 deletions
diff --git a/connectivity/source/drivers/hsqldb/HStorageAccess.cxx b/connectivity/source/drivers/hsqldb/HStorageAccess.cxx index 60e3fd0ee868..7bb868d2ff1a 100644 --- a/connectivity/source/drivers/hsqldb/HStorageAccess.cxx +++ b/connectivity/source/drivers/hsqldb/HStorageAccess.cxx @@ -106,7 +106,7 @@ extern "C" SAL_JNI_EXPORT jlong JNICALL Java_com_sun_star_sdbcx_comp_hsqldb_Nati #endif std::shared_ptr<StreamHelper> pHelper = StorageContainer::getRegisteredStream(env,name,key); - OSL_ENSURE(pHelper.get(),"No stream helper!"); + OSL_ENSURE(pHelper,"No stream helper!"); jlong nReturn = pHelper ? pHelper->getSeek()->getPosition() : jlong(0); #ifdef HSQLDB_DBG @@ -130,7 +130,7 @@ extern "C" SAL_JNI_EXPORT jlong JNICALL Java_com_sun_star_sdbcx_comp_hsqldb_Nati #endif std::shared_ptr<StreamHelper> pHelper = StorageContainer::getRegisteredStream(env,name,key); - OSL_ENSURE(pHelper.get(),"No stream helper!"); + OSL_ENSURE(pHelper,"No stream helper!"); jlong nReturn = pHelper ? pHelper->getSeek()->getLength() :jlong(0); #ifdef HSQLDB_DBG diff --git a/connectivity/source/drivers/hsqldb/StorageNativeInputStream.cxx b/connectivity/source/drivers/hsqldb/StorageNativeInputStream.cxx index bdafac4ee3ab..b16eaf742e1c 100644 --- a/connectivity/source/drivers/hsqldb/StorageNativeInputStream.cxx +++ b/connectivity/source/drivers/hsqldb/StorageNativeInputStream.cxx @@ -140,7 +140,7 @@ extern "C" SAL_JNI_EXPORT jlong JNICALL Java_com_sun_star_sdbcx_comp_hsqldb_Stor "n < 0"); std::shared_ptr<StreamHelper> pHelper = StorageContainer::getRegisteredStream(env,name,key); - OSL_ENSURE(pHelper.get(),"No stream helper!"); + OSL_ENSURE(pHelper,"No stream helper!"); if ( pHelper ) { Reference<XInputStream> xIn = pHelper->getInputStream(); @@ -202,7 +202,7 @@ extern "C" SAL_JNI_EXPORT jint JNICALL Java_com_sun_star_sdbcx_comp_hsqldb_Stora #endif std::shared_ptr<StreamHelper> pHelper = StorageContainer::getRegisteredStream(env,name,key); - OSL_ENSURE(pHelper.get(),"No stream helper!"); + OSL_ENSURE(pHelper,"No stream helper!"); Reference<XInputStream> xIn = pHelper ? pHelper->getInputStream() : Reference<XInputStream>(); if ( xIn.is() ) { |