View Issue Details

IDProjectCategoryView StatusLast Update
0024508CommunityOCCT:Modeling Algorithmspublic2017-05-28 18:17
Reporterbarbier Assigned Toifv 
PrioritynormalSeverityminor 
Status closedResolutionfixed 
PlatformAOSL 
Fixed in Version6.9.0 
Summary0024508: Suspect shadowed variable in BRepFeat_MakeLinearForm.cxx
DescriptionHello,

Lines 314-336 look like (Sliding is of type Standard_Boolean):

  if(Sliding) {
    Standard_Boolean Sliding = Standard_False;
    Handle(Geom_Surface) s = BRep_Tool::Surface(FirstFace);
    if (s->DynamicType() ==
    STANDARD_TYPE(Geom_RectangularTrimmedSurface)) {
      s = Handle(Geom_RectangularTrimmedSurface)::
    DownCast(s)->BasisSurface();
    }
    if(s->DynamicType() == STANDARD_TYPE(Geom_Plane) ||
       s->DynamicType() == STANDARD_TYPE(Geom_CylindricalSurface)) {
      // if plane or cylinder : sliding is possible
      Sliding = Standard_True;
    }
  }

  if(Sliding) {
  ...

But another Sliding variable is defined in the inner scope, and thus it is not modified in the outer scope.
TagsNo tags attached.
Test case number

Relationships

related to 0025734 closedbugmaster Open CASCADE GCC warnings in Android build 

Activities

abv

2017-05-28 18:17

manager   ~0066687

Fixed by 0025734

Issue History

Date Modified Username Field Change
2014-01-02 01:42 barbier New Issue
2014-01-02 01:42 barbier Assigned To => ifv
2017-05-28 18:16 abv Relationship added related to 0025734
2017-05-28 18:17 abv Note Added: 0066687
2017-05-28 18:17 abv Status new => closed
2017-05-28 18:17 abv Fixed in Version => 6.9.0
2017-05-28 18:17 abv Resolution open => fixed