diff options
author | Bogdan Buzea <buzea.bogdan@libreoffice.org> | 2024-10-18 17:39:10 +0200 |
---|---|---|
committer | David Gilbert <freedesktop@treblig.org> | 2024-10-19 19:49:40 +0200 |
commit | ba08d8f3a9b4421556284b76ed0635a8a62d7567 (patch) | |
tree | ac2671bdf9f14200287790ab41cf06e93d329996 /basctl | |
parent | 424329bb0b71fdb46b56be5638d5304dbef9e718 (diff) |
tdf#163486: PVS: Identical branches
Change-Id: If000f1e0bb01c6d551ef970edae8058e8020a202
V1037: Two or more case-branches perform the same actions. Check lines: 917, 945
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/175152
Tested-by: Jenkins
Reviewed-by: David Gilbert <freedesktop@treblig.org>
Diffstat (limited to 'basctl')
-rw-r--r-- | basctl/source/basicide/basides1.cxx | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/basctl/source/basicide/basides1.cxx b/basctl/source/basicide/basides1.cxx index e42c645ee9d8..1417b3799fb6 100644 --- a/basctl/source/basicide/basides1.cxx +++ b/basctl/source/basicide/basides1.cxx @@ -913,6 +913,8 @@ void Shell::GetState(SfxItemSet &rSet) } break; case SID_DOCINFO: + case SID_NEWWINDOW: + case SID_SAVEASDOC: { rSet.DisableItem( nWh ); } @@ -939,12 +941,6 @@ void Shell::GetState(SfxItemSet &rSet) rSet.DisableItem( nWh ); } break; - case SID_NEWWINDOW: - case SID_SAVEASDOC: - { - rSet.DisableItem( nWh ); - } - break; case SID_SIGNATURE: { SignatureState nState = SignatureState::NOSIGNATURES; |