From b2d4add4b8fde8f5a30cd591c8b36d53ee58d3fc Mon Sep 17 00:00:00 2001 From: Pawel Date: Thu, 26 Jul 2012 11:08:26 +0200 Subject: [PATCH] 0023349: There are identical sub-expressions 'aS1.ShapeType() != TopAbs_FACE' to the left and to the right of the '||' operator. The same shape was checked twice if it was a face. The other shape was omitted instead. --- src/BOPTest/BOPTest_BOPCommands.cxx | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/BOPTest/BOPTest_BOPCommands.cxx b/src/BOPTest/BOPTest_BOPCommands.cxx index 61b43cd668..83ccddd5ae 100755 --- a/src/BOPTest/BOPTest_BOPCommands.cxx +++ b/src/BOPTest/BOPTest_BOPCommands.cxx @@ -1508,7 +1508,7 @@ Standard_Integer bopsinf (Draw_Interpretor& di, } if (aS1.ShapeType()!=TopAbs_FACE || - aS1.ShapeType()!=TopAbs_FACE ) { + aS2.ShapeType()!=TopAbs_FACE ) { //printf(" Shapes #%d #%d are not faces\n", nF1, nF2 ); di << " Shapes #" << nF1 << " #" << nF2 << " are not faces\n"; return 0; @@ -1570,7 +1570,7 @@ Standard_Integer bopsonf (Draw_Interpretor& di, } if (aS1.ShapeType()!=TopAbs_FACE || - aS1.ShapeType()!=TopAbs_FACE ) { + aS2.ShapeType()!=TopAbs_FACE ) { //printf(" Shapes #%d #%d are not faces\n", nF1, nF2 ); di << " Shapes #" << nF1 << " #" << nF2 << " are not faces\n"; return 0;