diff options
| author | Mike Kaganski <mike.kaganski@collabora.com> | 2018-11-11 18:04:43 +0300 |
|---|---|---|
| committer | Mike Kaganski <mike.kaganski@collabora.com> | 2018-11-11 19:38:25 +0100 |
| commit | 6b67f867cca1cb62ca045b60635f6da0a347c4f2 (patch) | |
| tree | 726c4b4b59dded0cb2cc2b852636d10170b79505 /dbaccess | |
| parent | b7f022234589c84ddb8566ddf7d91701eba55d3d (diff) | |
tdf#120703 PVS: V560 A part of conditional expression is always true/false
Change-Id: I1027242192dfd2be4918094d59f2a8f8c8fec041
Reviewed-on: https://gerrit.libreoffice.org/63266
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
Diffstat (limited to 'dbaccess')
| -rw-r--r-- | dbaccess/source/ui/app/AppController.cxx | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/dbaccess/source/ui/app/AppController.cxx b/dbaccess/source/ui/app/AppController.cxx index 0958cda048e8..0915c666c0fa 100644 --- a/dbaccess/source/ui/app/AppController.cxx +++ b/dbaccess/source/ui/app/AppController.cxx @@ -571,7 +571,7 @@ FeatureState OApplicationController::GetState(sal_uInt16 _nId) const break; case ID_BROWSER_CUT: aReturn.bEnabled = !isDataSourceReadOnly() && getContainer()->getSelectionCount() >= 1; - aReturn.bEnabled = aReturn.bEnabled && ( !(ID_BROWSER_CUT == _nId && getContainer()->getElementType() == E_TABLE) || getContainer()->isCutAllowed() ); + aReturn.bEnabled = aReturn.bEnabled && (getContainer()->getElementType() != E_TABLE || getContainer()->isCutAllowed()); break; case ID_BROWSER_PASTE: switch( getContainer()->getElementType() ) |