0031699: IVtkOCC_ShapePickerAlgo.cxx is missing a break in a switch/case starting at line 132

View Issue Details [ Jump to Notes ]

[ Issue History ] [ Print ]

Project

Category

View Status

Date Submitted

Last Update

0031699

Community

[OCCT] OCCT:VIS

public

2020-08-04 19:29

2021-03-29 16:18

Reporter

jensgw

Assigned To

bugmaster

Priority

normal

Severity

minor

Status

closed

Resolution

fixed

Platform

OS Version

Product Version

[OCCT] 7.3.0

Target Version

[OCCT] 7.5.0

Fixed in Version

[OCCT] 7.5.0

Summary

0031699: IVtkOCC_ShapePickerAlgo.cxx is missing a break in a switch/case starting at line 132

Description

IVtkOCC_ShapePickerAlgo.cxx is missing a break in a switch/case starting at line 132.

It looks like the case SelectMgr_TOU_Full is missing a break.
RebuildObjectsTree() is called already, so a fallthrough to SelectMgr_TOU_Partial should not be needed.
If the fallthrough is intended, this error report is proof that such coding practice can lead to confusion and I would ask you to refactor.

switch (aSel->UpdateStatus())
    {
      case SelectMgr_TOU_Full:
        // Recompute the sensitive primitives which correspond to the mode.
        myViewerSelector->RemoveSelectionOfObject (aSelObj, aSelObj->Selection (theMode));
        aSelObj->RecomputePrimitives (theMode);
        myViewerSelector->AddSelectionToObject (aSelObj, aSelObj->Selection (theMode));
        myViewerSelector->RebuildObjectsTree();
        myViewerSelector->RebuildSensitivesTree (aSelObj);
// here should be a break;
      case SelectMgr_TOU_Partial:
        {
          if (aSelObj->HasTransformation())
          {
            myViewerSelector->RebuildObjectsTree();
          }
          break;
        }
      default:
        break;
    }

Tags

No tags attached.

Test case number

Attached Files

Relationships

related to

0031765

closed

bugmaster

Open CASCADE

Coding Rules - eliminate GCC compiler warnings -Wcatch-value in IVtk

Notes
(0093431) jensgw (reporter) 2020-08-04 19:32	Product Version is 7.4.0. But that can not be selected in the dropdown.

(0099752) kgv (developer) 2021-03-26 06:41	Fixed by 0031765, please close the issue.

Issue History
Date Modified	Username	Field	Change
2020-08-04 19:29	jensgw	New Issue
2020-08-04 19:29	jensgw	Assigned To	=> ssv
2020-08-04 19:32	jensgw	Note Added: 0093431
2020-08-17 15:38	kgv	Relationship added	related to 0028988
2020-09-14 14:07	kgv	Relationship added	related to 0031765
2021-03-26 06:41	kgv	Note Added: 0099752
2021-03-26 06:41	kgv	Assigned To	ssv => bugmaster
2021-03-26 06:41	kgv	Status	new => feedback
2021-03-26 06:41	kgv	Resolution	open => fixed
2021-03-26 06:41	kgv	Fixed in Version	=> 7.5.0
2021-03-26 06:41	kgv	Target Version	=> 7.5.0
2021-03-29 16:18	bugmaster	Status	feedback => closed