diff options
| author | Bogdan Buzea <buzea.bogdan@libreoffice.org> | 2024-10-18 17:06:12 +0200 |
|---|---|---|
| committer | David Gilbert <freedesktop@treblig.org> | 2024-11-16 15:17:55 +0100 |
| commit | adcc06a4564164b8b0a73ce4254d54f144675bc9 (patch) | |
| tree | 6a5b7c1418dfbfb96b7550c8c49bf59e5a8198f2 /chart2/source | |
| parent | 03e4b7c48d6294520fbe914fe3675108a9935769 (diff) | |
tdf#163486: PVS: Identical branches
V1037 Two or more case-branches perform the same actions. Check lines: 146, 158. This will still be the case, the code being just at the moment similar.
Change-Id: Iceca3c717ccc6ee3ea865d3a469418ff6e8accd7
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/175146
Reviewed-by: David Gilbert <freedesktop@treblig.org>
Tested-by: Jenkins
Diffstat (limited to 'chart2/source')
| -rw-r--r-- | chart2/source/controller/dialogs/ChartResourceGroups.cxx | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/chart2/source/controller/dialogs/ChartResourceGroups.cxx b/chart2/source/controller/dialogs/ChartResourceGroups.cxx index 1e43eb481a36..f773eefd7062 100644 --- a/chart2/source/controller/dialogs/ChartResourceGroups.cxx +++ b/chart2/source/controller/dialogs/ChartResourceGroups.cxx @@ -142,9 +142,6 @@ void StackingResourceGroup::fillControls(const ChartTypeParameter& rParameter) != GlobalStackMode_STACK_Z); //todo remove this condition if z stacking radio button is really used switch (rParameter.eStackMode) { - case GlobalStackMode_STACK_Y: - m_xRB_Stack_Y->set_active(true); - break; case GlobalStackMode_STACK_Y_PERCENT: m_xRB_Stack_Y_Percent->set_active(true); break; @@ -157,6 +154,7 @@ void StackingResourceGroup::fillControls(const ChartTypeParameter& rParameter) */ m_xRB_Stack_Y->set_active(true); break; + case GlobalStackMode_STACK_Y: default: m_xRB_Stack_Y->set_active(true); break; |