!valid always returns false and hence is unnecessary

Signed-off-by: Surajjalpun2002 <[email protected]>
OpenChemistry · Oct 19, 2023 · f3d76c1 · f3d76c1
1 parent 3cb9696
commit f3d76c1
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/avogadro/core/dihedraliterator.cpp b/avogadro/core/dihedraliterator.cpp
@@ -82,7 +82,7 @@ Dihedral DihedralIterator::operator++()
     if (valid) {
       // we have a valid current dihedral, try to find a new "d"
       for (const auto maybeD : graph.neighbors(c)) {
-        if (maybeD != a && maybeD != b && (!valid || maybeD > d)) {
+        if (maybeD != a && maybeD != b && maybeD > d) {
           m_current = make_tuple(a, b, c, maybeD);
           return m_current;
         }