comparison src/Finder.cpp @ 86:f07b9b7f1ab6 refactors

Previous commit was a mistake: the ahead-of-time business is in Finder::getExpandDirection. In fact we were failing to swap advance directions in forward path when writing to the "other" finder. This does not actually affect the backward path calculation, but it does mean we can restore the sanity check.

                     dir = Matcher::AdvanceOther;
                 }
             }
 
             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;
                     err.r = r;
                     err.c = c;