MantisBT - Open CASCADE
View Issue Details
0025749Open CASCADE[OCCT] OCCT:Codingpublic2015-01-26 11:282019-07-10 19:27
ski 
abv 
normalminor 
assignedopen 
[OCCT] 6.8.0 
[OCCT] 7.5.0* 
0025749: Coding, HLRBRep - Eliminate GCC warning -Wswitch in Android build
It is necessary to eliminate GCC warning -Wswitch in file HLRBRep_Curve.cxx:

HLRBRep_Curve.cxx: In member function 'Standard_Real HLRBRep_Curve::Parameter2d(Standard_Real) const':
HLRBRep_Curve.cxx:62:10: warning: enumeration value 'GeomAbs_Circle' not handled in switch [-Wswitch]
   switch (myType)
HLRBRep_Curve.cxx:62:10: warning: enumeration value 'GeomAbs_Hyperbola' not handled in switch [-Wswitch]
HLRBRep_Curve.cxx:62:10: warning: enumeration value 'GeomAbs_Parabola' not handled in switch [-Wswitch]
HLRBRep_Curve.cxx:62:10: warning: enumeration value 'GeomAbs_BezierCurve' not handled in switch [-Wswitch]
HLRBRep_Curve.cxx:62:10: warning: enumeration value 'GeomAbs_BSplineCurve' not handled in switch [-Wswitch]
HLRBRep_Curve.cxx:62:10: warning: enumeration value 'GeomAbs_OtherCurve' not handled in switch [-Wswitch]

It seems that GeomAbs_Circle should be handled like GeomAbs_Ellipse.
No test.
No tags attached.
related to 0025734closed bugmaster Open CASCADE GCC warnings in Android build 
Issue History
2015-01-26 11:28skiNew Issue
2015-01-26 11:28skiAssigned To => abv
2015-01-26 12:40abvRelationship addedrelated to 0025734
2015-01-26 12:42abvNote Added: 0036641
2015-01-26 12:42abvAssigned Toabv => jgv
2015-01-26 12:42abvStatusnew => assigned
2015-02-25 16:01gitNote Added: 0037870
2015-02-25 16:03jgvNote Added: 0037871
2015-02-25 16:03jgvAssigned Tojgv => kgv
2015-02-25 16:03jgvStatusassigned => resolved
2015-02-25 16:03jgvSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=9465#r9465
2015-02-25 16:38abvNote Added: 0037875
2015-02-25 16:38abvAssigned Tokgv => abv
2015-02-25 16:38abvStatusresolved => assigned
2015-04-19 19:13abvTarget Version => 7.1.0
2015-09-25 00:46abvRelationship addedrelated to 0026236
2015-09-25 19:50abvSummaryEliminate GCC warning -Wswitch in Android build => Coding, HLRBRep - Eliminate GCC warning -Wswitch in Android build
2016-01-06 13:55abvNote Added: 0049701
2016-11-03 17:21abvTarget Version7.1.0 => 7.2.0
2017-07-27 09:43abvTarget Version7.2.0 => 7.4.0
2019-07-10 19:27abvTarget Version7.4.0 => 7.5.0*

Notes
(0036641)
abv   
2015-01-26 12:42   
Julia, could you please check the reported issue and suggest how it should be corrected?
(0037870)
git   
2015-02-25 16:01   
Branch CR25749 has been created by jgv.

SHA-1: 2714656ef1ea0c9173f2d72cdb976ba1cf6d9d70


Detailed log of new commits:

Author: jgv
Date: Wed Feb 25 16:00:57 2015 +0300

    0025749: Eliminate GCC warning -Wswitch in Android build
(0037871)
jgv   
2015-02-25 16:03   
Please review the branch CR25749.
(0037875)
abv   
2015-02-25 16:38   
The goal is not to suppress the warning but to ensure that the code is correct, and this seems no be not resolved: by the code, it seems that circles are possible in HLR, but it is not clear if they are handled properly at the place where warning is generated.
(0049701)
abv   
2016-01-06 13:55   
Note that warning has been silenced by patch for 0026254, however the reported values are still not explicitly handles in switch