diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2019-03-05 13:44:17 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2019-03-06 06:47:06 +0100 |
| commit | a2e3705d8b3b05ae664d54b762d6ff72927d5e48 (patch) | |
| tree | 6ac31f9c9b419b12dbf065845a6941c143f57e2c /framework | |
| parent | f423ac2d3bdba4263f1f41e31e7e2b7715afdd6e (diff) | |
loplugin:unnecessaryparen improve member expression
Change-Id: I304621018cb1e2a47e478e86df4229bcf2176741
Reviewed-on: https://gerrit.libreoffice.org/68757
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'framework')
| -rw-r--r-- | framework/source/services/autorecovery.cxx | 10 | ||||
| -rw-r--r-- | framework/source/xml/acceleratorconfigurationreader.cxx | 5 |
2 files changed, 3 insertions, 12 deletions
diff --git a/framework/source/services/autorecovery.cxx b/framework/source/services/autorecovery.cxx index 4f8210d405b1..778a9228d4d7 100644 --- a/framework/source/services/autorecovery.cxx +++ b/framework/source/services/autorecovery.cxx @@ -2158,19 +2158,13 @@ void AutoRecovery::implts_stopListening() xGlobalEventBroadcaster.set(m_xNewDocBroadcaster, css::uno::UNO_QUERY); } /* SAFE */ - if ( - (xGlobalEventBroadcaster.is()) && - (m_bListenForDocEvents ) - ) + if (xGlobalEventBroadcaster.is() && m_bListenForDocEvents) { xGlobalEventBroadcaster->removeDocumentEventListener(m_xNewDocBroadcasterListener); m_bListenForDocEvents = false; } - if ( - (xCFG.is() ) && - (m_bListenForConfigChanges) - ) + if (xCFG.is() && m_bListenForConfigChanges) { xCFG->removeChangesListener(m_xRecoveryCFGListener); m_bListenForConfigChanges = false; diff --git a/framework/source/xml/acceleratorconfigurationreader.cxx b/framework/source/xml/acceleratorconfigurationreader.cxx index ea945756c6db..6c50e40cf284 100644 --- a/framework/source/xml/acceleratorconfigurationreader.cxx +++ b/framework/source/xml/acceleratorconfigurationreader.cxx @@ -75,10 +75,7 @@ void SAL_CALL AcceleratorConfigurationReader::endDocument() // The xml file seems to be corrupted. // Because we found no end-tags ... at least for // one list or item. - if ( - (m_bInsideAcceleratorList) || - (m_bInsideAcceleratorItem) - ) + if (m_bInsideAcceleratorList || m_bInsideAcceleratorItem) { THROW_PARSEEXCEPTION("No matching start or end element 'acceleratorlist' found!") } |