0023294: Identical sub-expression in 'if-clause'
authorPawel Kowalski <pawel-kowalski@wp.pl>
Fri, 10 Aug 2012 06:45:42 +0000 (10:45 +0400)
committerPawel Kowalski <pawel-kowalski@wp.pl>
Fri, 10 Aug 2012 06:45:42 +0000 (10:45 +0400)
Correcting probable copy-paste bug.
Test case offset shape_type_i D7 was modified

src/BOP/BOP_ArgumentAnalyzer.cxx
tests/offset/shape_type_i/D7

index 5c73809..e0e1559 100755 (executable)
@@ -928,7 +928,7 @@ Standard_Boolean TestSubShapeType(const TopAbs_ShapeEnum theT1,
       }
     }
     if((aT1==TopAbs_WIRE && aT2==TopAbs_SHELL) ||
-       (aT1==TopAbs_WIRE && aT2==TopAbs_SHELL)) {
+       (aT2==TopAbs_WIRE && aT1==TopAbs_SHELL)) {
       if(!BOP_WireShell::CheckArgTypes(aT1,aT2,theOP)) {
         return Standard_False;
       }
index fa221d1..39e5562 100644 (file)
@@ -1,5 +1,6 @@
 puts "TODO OCC23068 Mandriva2010: Error: bsection of result and s is not equal zero"
 puts "TODO OCC23068 Mandriva2010: Error : The volume of result shape "
+puts "TODO ?OCC23068 Mandriva2010: Faulty shapes in variables faulty_1 to faulty_2"
 ellipse w1 0 0 0 15 10
 mkedge w1 w1 0 pi/2
 trotate w1 0 0 0 1 0 0 90