Archived issue #0023294
Identical sub-expression in 'if-clause'
Description
BOP_ArgumentAnalyzer.cxx, line 930
if((aT1==TopAbs_WIRE && aT2==TopAbs_SHELL) ||
(aT1==TopAbs_WIRE && aT2==TopAbs_SHELL)) {
should probably read
if((aT1==TopAbs_WIRE && aT2==TopAbs_SHELL) ||
(aT2==TopAbs_WIRE && aT1==TopAbs_SHELL)) {
if((aT1==TopAbs_WIRE && aT2==TopAbs_SHELL) ||
(aT1==TopAbs_WIRE && aT2==TopAbs_SHELL)) {
should probably read
if((aT1==TopAbs_WIRE && aT2==TopAbs_SHELL) ||
(aT2==TopAbs_WIRE && aT1==TopAbs_SHELL)) {
Public activity
3 archived notes
Participants are labeled by their role within this record.
Corresponding git branch pushed. Please review.
Reviewed. Please test.
Dear Commenter 2,
Branch CR23294 (and products from GIT master) was compiled on Linux and Windows platforms and tested.
Regressions:
Not detected
Improvements:
Not detected
Testing case:
Not needed
Branch CR23294 (and products from GIT master) was compiled on Linux and Windows platforms and tested.
Regressions:
Not detected
Improvements:
Not detected
Testing case:
Not needed
Related records