OpenCascade/occt
1
0
Fork 0
mirror of https://git.dev.opencascade.org/repos/occt.git synced 2025-04-21 10:13:43 +03:00
Code

0023349: There are identical sub-expressions 'aS1.ShapeType() != TopAbs_FACE' to the left and to the right of the '||' operator.

Browse Source 
The same shape was checked twice if it was a face. The other shape was omitted instead.
This commit is contained in:
Pawel 2012-07-26 11:08:26 +02:00 committed by Pawel Kowalski
parent 8fb480b35a
commit b2d4add4b8
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/BOPTest/BOPTest_BOPCommands.cxx
View File

@ -1508,7 +1508,7 @@ Standard_Integer bopsinf (Draw_Interpretor& di,
} }
if (aS1.ShapeType()!=TopAbs_FACE || if (aS1.ShapeType()!=TopAbs_FACE ||
aS1.ShapeType()!=TopAbs_FACE ) { aS2.ShapeType()!=TopAbs_FACE ) {
//printf(" Shapes #%d #%d are not faces\n", nF1, nF2 ); //printf(" Shapes #%d #%d are not faces\n", nF1, nF2 );
di << " Shapes #" << nF1 << " #" << nF2 << " are not faces\n"; di << " Shapes #" << nF1 << " #" << nF2 << " are not faces\n";
return 0; return 0;
@ -1570,7 +1570,7 @@ Standard_Integer bopsonf (Draw_Interpretor& di,
} }
if (aS1.ShapeType()!=TopAbs_FACE || if (aS1.ShapeType()!=TopAbs_FACE ||
aS1.ShapeType()!=TopAbs_FACE ) { aS2.ShapeType()!=TopAbs_FACE ) {
//printf(" Shapes #%d #%d are not faces\n", nF1, nF2 ); //printf(" Shapes #%d #%d are not faces\n", nF1, nF2 );
di << " Shapes #" << nF1 << " #" << nF2 << " are not faces\n"; di << " Shapes #" << nF1 << " #" << nF2 << " are not faces\n";
return 0; return 0;