diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-10-18 08:27:05 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-10-18 09:46:28 +0200 |
| commit | 5fb66ae5595b7435e8954df31473fad15a74b8c2 (patch) | |
| tree | 3b3f0ce3eafa10557a7e78b10851c97ee16c7ebf /sfx2 | |
| parent | 181a1b36ac728e3a43e054496ceb53fd3315abdb (diff) | |
clang-tidy readability-simplify-boolean-expr
Change-Id: I78fa01a6c803dec782488490b730af3a11814d64
Reviewed-on: https://gerrit.libreoffice.org/61902
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'sfx2')
| -rw-r--r-- | sfx2/source/appl/appopen.cxx | 7 | ||||
| -rw-r--r-- | sfx2/source/view/frame.cxx | 6 |
2 files changed, 4 insertions, 9 deletions
diff --git a/sfx2/source/appl/appopen.cxx b/sfx2/source/appl/appopen.cxx index 632a2f985268..1c16308dbd4e 100644 --- a/sfx2/source/appl/appopen.cxx +++ b/sfx2/source/appl/appopen.cxx @@ -555,11 +555,8 @@ bool lcl_isFilterNativelySupported(const SfxFilter& rFilter) return true; const OUString& aName = rFilter.GetFilterName(); - if (aName.startsWith("MS Excel")) - // We can handle all Excel variants natively. - return true; - - return false; + // We can handle all Excel variants natively. + return aName.startsWith("MS Excel"); } } diff --git a/sfx2/source/view/frame.cxx b/sfx2/source/view/frame.cxx index a14885f40752..6fd9832b2b4d 100644 --- a/sfx2/source/view/frame.cxx +++ b/sfx2/source/view/frame.cxx @@ -163,10 +163,8 @@ void SfxFrame::DoClose_Impl() bool SfxFrame::DocIsModified_Impl() { - if ( pImpl->pCurrentViewFrame && pImpl->pCurrentViewFrame->GetObjectShell() && - pImpl->pCurrentViewFrame->GetObjectShell()->IsModified() ) - return true; - return false; + return pImpl->pCurrentViewFrame && pImpl->pCurrentViewFrame->GetObjectShell() && + pImpl->pCurrentViewFrame->GetObjectShell()->IsModified(); } bool SfxFrame::PrepareClose_Impl( bool bUI ) |