# HG changeset patch
# User Chris Cannam
# Date 1417087637 0
# Node ID a540b26db2287c9ed188d4df8fc4dcd2fc44bdd4
# Parent  de7034e93dd0bc6717de7013631279211eda8941
Ah, no, of course the advance differs for the forward path (it is based on partial information). But the cost shouldn't.

diff -r de7034e93dd0 -r a540b26db228 src/Finder.cpp
--- a/src/Finder.cpp	Thu Nov 27 10:53:00 2014 +0000
+++ b/src/Finder.cpp	Thu Nov 27 11:27:17 2014 +0000
@@ -222,17 +222,6 @@
             }
 
             if (dir != Matcher::AdvanceNone) {
-                if (m_m->getAdvance(r, c) != dir) {
-                    cerr << "WrongAdvance found" << endl;
-                    err.type = ErrorPosition::WrongAdvance;
-                    err.r = r;
-                    err.c = c;
-                    err.costWas = m_m->getPathCost(r, c);
-                    err.costShouldBe = updateTo;
-                    err.advanceWas = m_m->getAdvance(r, c);
-                    err.advanceShouldBe = dir;
-                    return err;
-                }
                 if (m_m->getPathCost(r, c) != updateTo) {
                     cerr << "WrongCost found" << endl;
                     err.type = ErrorPosition::WrongCost;
diff -r de7034e93dd0 -r a540b26db228 src/Finder.h
--- a/src/Finder.h	Thu Nov 27 10:53:00 2014 +0000
+++ b/src/Finder.h	Thu Nov 27 11:27:17 2014 +0000
@@ -68,7 +68,7 @@
 protected:
 #ifdef PERFORM_ERROR_CHECKS
     struct ErrorPosition {
-        enum Type { NoError = 0, WrongCost, WrongAdvance, NoAdvance };
+        enum Type { NoError = 0, WrongCost, NoAdvance };
         ErrorPosition() : type(NoError) { }
         Type type;
         int r;