diff options
author | Noel Grandin <noel.grandin@collabora.co.uk> | 2017-04-05 12:26:29 +0200 |
---|---|---|
committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2017-04-05 12:21:19 +0000 |
commit | 572e9cde744e2b482a04ff4b457ad2cfde5f70d2 (patch) | |
tree | 78e93df178de6d17d55eef7c3907781aa131ff7c /sd | |
parent | sc: remove unused private field (diff) | |
download | core-572e9cde744e2b482a04ff4b457ad2cfde5f70d2.tar.gz core-572e9cde744e2b482a04ff4b457ad2cfde5f70d2.zip |
loplugin:redundantcast find c-style bool casts
Change-Id: I3237b93babc67de12c3771aa84766c2141ca93b2
Reviewed-on: https://gerrit.libreoffice.org/36137
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'sd')
-rw-r--r-- | sd/source/core/sdpage.cxx | 2 | ||||
-rw-r--r-- | sd/source/filter/ppt/pptatom.hxx | 2 | ||||
-rw-r--r-- | sd/source/ui/dlg/masterlayoutdlg.cxx | 8 | ||||
-rw-r--r-- | sd/source/ui/func/futext.cxx | 2 | ||||
-rw-r--r-- | sd/source/ui/unoidl/UnoDocumentSettings.cxx | 2 |
5 files changed, 8 insertions, 8 deletions
diff --git a/sd/source/core/sdpage.cxx b/sd/source/core/sdpage.cxx index afb247e441b2..3d5577608368 100644 --- a/sd/source/core/sdpage.cxx +++ b/sd/source/core/sdpage.cxx @@ -2937,7 +2937,7 @@ bool SdPage::RestoreDefaultText( SdrObject* pObj ) // not changed object. if(pTextObj && pTextObj->GetOutlinerParaObject() - && pTextObj->GetOutlinerParaObject()->IsVertical() != (bool)bVertical) + && pTextObj->GetOutlinerParaObject()->IsVertical() != bVertical) { ::tools::Rectangle aObjectRect = pTextObj->GetSnapRect(); pTextObj->GetOutlinerParaObject()->SetVertical(bVertical); diff --git a/sd/source/filter/ppt/pptatom.hxx b/sd/source/filter/ppt/pptatom.hxx index c23e662442b1..c03882ed6994 100644 --- a/sd/source/filter/ppt/pptatom.hxx +++ b/sd/source/filter/ppt/pptatom.hxx @@ -97,7 +97,7 @@ inline const Atom* Atom::findNextChildAtom( const Atom* pLast ) inline bool Atom::isContainer() const { - return (bool)maRecordHeader.IsContainer(); + return maRecordHeader.IsContainer(); } inline bool Atom::seekToContent() const diff --git a/sd/source/ui/dlg/masterlayoutdlg.cxx b/sd/source/ui/dlg/masterlayoutdlg.cxx index 62f8a1578f60..7508b4c49dae 100644 --- a/sd/source/ui/dlg/masterlayoutdlg.cxx +++ b/sd/source/ui/dlg/masterlayoutdlg.cxx @@ -100,7 +100,7 @@ void MasterLayoutDialog::applyChanges() { mpDoc->BegUndo(GetText()); - if( (mpCurrentPage->GetPageKind() != PageKind::Standard) && (mbOldHeader != (bool) mpCBHeader->IsChecked() ) ) + if( (mpCurrentPage->GetPageKind() != PageKind::Standard) && (mbOldHeader != mpCBHeader->IsChecked() ) ) { if( mbOldHeader ) remove( PRESOBJ_HEADER ); @@ -108,7 +108,7 @@ void MasterLayoutDialog::applyChanges() create( PRESOBJ_HEADER ); } - if( mbOldFooter != (bool) mpCBFooter->IsChecked() ) + if( mbOldFooter != mpCBFooter->IsChecked() ) { if( mbOldFooter ) remove( PRESOBJ_FOOTER ); @@ -116,7 +116,7 @@ void MasterLayoutDialog::applyChanges() create( PRESOBJ_FOOTER ); } - if( mbOldDate != (bool) mpCBDate->IsChecked() ) + if( mbOldDate != mpCBDate->IsChecked() ) { if( mbOldDate ) remove( PRESOBJ_DATETIME ); @@ -124,7 +124,7 @@ void MasterLayoutDialog::applyChanges() create( PRESOBJ_DATETIME ); } - if( mbOldPageNumber != (bool) mpCBPageNumber->IsChecked() ) + if( mbOldPageNumber != mpCBPageNumber->IsChecked() ) { if( mbOldPageNumber ) remove( PRESOBJ_SLIDENUMBER ); diff --git a/sd/source/ui/func/futext.cxx b/sd/source/ui/func/futext.cxx index 9f1f57a02df2..4f9955ceea3a 100644 --- a/sd/source/ui/func/futext.cxx +++ b/sd/source/ui/func/futext.cxx @@ -699,7 +699,7 @@ bool FuText::MouseButtonUp(const MouseEvent& rMEvt) pPara = GetTextObj()->GetOutlinerParaObject(); } - if(pPara && (bool)bVertical != pPara->IsVertical()) + if(pPara && bVertical != pPara->IsVertical()) { // set ParaObject orientation accordingly pPara->SetVertical(bVertical); diff --git a/sd/source/ui/unoidl/UnoDocumentSettings.cxx b/sd/source/ui/unoidl/UnoDocumentSettings.cxx index aacfa2b133c8..6d54f1cb47e9 100644 --- a/sd/source/ui/unoidl/UnoDocumentSettings.cxx +++ b/sd/source/ui/unoidl/UnoDocumentSettings.cxx @@ -252,7 +252,7 @@ void DocumentSettings::AssignURL( XPropertyListType t, const Any* pValue, bool *pOk, bool *pChanged ) { OUString aURL; - if( !(bool)( *pValue >>= aURL ) ) + if( !( *pValue >>= aURL ) ) return; if( LoadList( t, aURL, ""/*TODO?*/, uno::Reference< embed::XStorage >() ) ) |