diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2017-05-16 10:42:03 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2017-05-16 12:09:38 +0200 |
| commit | 658eec1d5b96ea0a971fbdf531979480f72516a4 (patch) | |
| tree | 1ac9b520cf79ebd7befe3c76ae2686176fbee255 /basctl/source/basicide/baside3.cxx | |
| parent | 482ef1bfe95a32a6796ce5a3150f272845db7931 (diff) | |
loplugin:comparisonwithconstant in basctl
Change-Id: I0ac895e279e4feb47345e73ecef0d5ee1909c031
Reviewed-on: https://gerrit.libreoffice.org/37668
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'basctl/source/basicide/baside3.cxx')
| -rw-r--r-- | basctl/source/basicide/baside3.cxx | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/basctl/source/basicide/baside3.cxx b/basctl/source/basicide/baside3.cxx index 550bcd3637e9..fa499ae2dcc9 100644 --- a/basctl/source/basicide/baside3.cxx +++ b/basctl/source/basicide/baside3.cxx @@ -253,7 +253,7 @@ void DialogWindow::GetState( SfxItemSet& rSet ) } } - for ( sal_uInt16 nWh = aIter.FirstWhich(); 0 != nWh; nWh = aIter.NextWhich() ) + for ( sal_uInt16 nWh = aIter.FirstWhich(); nWh != 0; nWh = aIter.NextWhich() ) { switch ( nWh ) { @@ -913,19 +913,19 @@ bool implImportDialog( vcl::Window* pWin, const OUString& rCurPath, const Script ScopedVclPtrInstance< NameClashQueryBox > aQueryBox( pWin, aQueryBoxTitle, aQueryBoxText ); sal_uInt16 nRet = aQueryBox->Execute(); - if( RET_YES == nRet ) + if( nRet == RET_YES ) { // RET_YES == Rename, see NameClashQueryBox::NameClashQueryBox eNameClashMode = CLASH_RENAME_DIALOG; aNewDlgName = rDocument.createObjectName( E_DIALOGS, aLibName ); } - else if( RET_NO == nRet ) + else if( nRet == RET_NO ) { // RET_NO == Replace, see NameClashQueryBox::NameClashQueryBox eNameClashMode = CLASH_OVERWRITE_DIALOG; } - else if( RET_CANCEL == nRet ) + else if( nRet == RET_CANCEL ) { return bDone; } @@ -975,7 +975,7 @@ bool implImportDialog( vcl::Window* pWin, const OUString& rCurPath, const Script OUString aQueryBoxText(IDE_RESSTR(RID_STR_DLGIMP_MISMATCH_TEXT)); ScopedVclPtrInstance< LanguageMismatchQueryBox > aQueryBox( pWin, aQueryBoxTitle, aQueryBoxText ); sal_uInt16 nRet = aQueryBox->Execute(); - if( RET_YES == nRet ) + if( nRet == RET_YES ) { // RET_YES == Add, see LanguageMismatchQueryBox::LanguageMismatchQueryBox bAddDialogLanguagesToLib = true; @@ -985,7 +985,7 @@ bool implImportDialog( vcl::Window* pWin, const OUString& rCurPath, const Script //else if( RET_NO == nRet ) //{ //} - else if( RET_CANCEL == nRet ) + else if( nRet == RET_CANCEL ) { return bDone; } |