Index: DamEngine/trunk/src/Deltares.DamEngine.Data/Geometry/GeometryGenerator.cs =================================================================== diff -u -r5208 -r5214 --- DamEngine/trunk/src/Deltares.DamEngine.Data/Geometry/GeometryGenerator.cs (.../GeometryGenerator.cs) (revision 5208) +++ DamEngine/trunk/src/Deltares.DamEngine.Data/Geometry/GeometryGenerator.cs (.../GeometryGenerator.cs) (revision 5214) @@ -1066,9 +1066,13 @@ } //if (flag3 && flag4 || flag1 && flag2 || flag1 | flag2 && flag3 || flag4) - //ToDo: Check if the above condition is correct, was | operator intended instead of || operator? + //ToDo bka: Check if the above condition is correct, was | operator intended instead of || operator? //Next line is the same as above line, but with || operator instead of | operator if (flag3 && flag4 || flag1 && flag2 || ((flag1 || flag2) && flag3) || flag4) + //The solo flag4 at the end is strange, because at the start there is flag3 && flag4, you could leave that out: + //if (flag1 && flag2 || ((flag1 || flag2) && flag3) || flag4) + //Probably something else was meant, but that could give a different result, e.g. + //if (flag3 && flag4 || flag1 && flag2 || (flag1 || flag2) && (flag3 || flag4)) { if (flag1 && !flag2 && !flag5 && !flag7) {