0023294: Identical sub-expression in 'if-clause'

Correcting probable copy-paste bug.
Test case offset shape_type_i D7 was modified

diff --git a/src/BOP/BOP_ArgumentAnalyzer.cxx b/src/BOP/BOP_ArgumentAnalyzer.cxx
index 5c7380901ccfb1e855ed6489fd2e27b70f642383..e0e155982e5d1e871926ac5c4cabf508117287bc 100755 (executable)
--- a/src/BOP/BOP_ArgumentAnalyzer.cxx
+++ b/src/BOP/BOP_ArgumentAnalyzer.cxx
@@ -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;
       }

diff --git a/tests/offset/shape_type_i/D7 b/tests/offset/shape_type_i/D7
index fa221d1f40a8d71da6d2ed7b883b47a790e764fb..39e55621b90ad4cc58eba97626a683d9eff3ed80 100644 (file)
--- a/tests/offset/shape_type_i/D7
+++ b/tests/offset/shape_type_i/D7
@@ -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