comparison view/AlignmentView.cpp @ 1620:dc0e47f234a2 tip

Fix some erroneous uses of reference frame where we intended "above"-view frame

         for (const auto &km: m_fromAboveMap) {
 
             sv_frame_t af = km.first;
             sv_frame_t bf = km.second;
             
-            if (af < m_leftmostAbove || af > m_rightmostAbove) {
+            if (af < m_leftmostAbove) {
+#ifdef DEBUG_ALIGNMENT_VIEW
+                SVCERR << "AlignmentView: af " << af << " < m_leftmostAbove " << m_leftmostAbove << endl;
+#endif
+                continue;
+            }
+            if (af > m_rightmostAbove) {
+#ifdef DEBUG_ALIGNMENT_VIEW
+                SVCERR << "AlignmentView: af " << af << " > m_rightmostAbove " << m_rightmostAbove << endl;
+#endif
                 continue;
             }
 
             int ax = m_above->getXForFrame(af);
             int bx = m_below->getXForFrame(bf);

     // just all map to the same first/last destination frame. But we
     // don't want to display their mappings, as they're just noise.)
     m_leftmostAbove = -1;
     m_rightmostAbove = -1;
 
-    sv_frame_t prevRf = -1;
+    sv_frame_t prevAf = -1;
     sv_frame_t prevBf = -1;
     
-    foreach (sv_frame_t f, keyFrames) {
-
-        sv_frame_t rf = m_above->alignToReference(f);
+    foreach (sv_frame_t af, keyFrames) {
+
+        sv_frame_t rf = m_above->alignToReference(af);
         sv_frame_t bf = m_below->alignFromReference(rf);
 
         if (prevBf > 0 && bf > prevBf) {
-            if (m_leftmostAbove < 0 && prevBf > 0 && bf > prevBf) {
-                m_leftmostAbove = prevRf;
-            }
-            m_rightmostAbove = rf;
-        }
-        prevRf = rf;
+            if (m_leftmostAbove < 0) {
+                m_leftmostAbove = prevAf;
+            }
+            m_rightmostAbove = af;
+        }
+        prevAf = af;
         prevBf = bf;
         
         bool mappedSomething = false;
         
         if (resolution > 1) {
             if (keyFramesBelow.find(bf) == keyFramesBelow.end()) {
 
-                sv_frame_t f1 = f + resolution;
-                sv_frame_t rf1 = m_above->alignToReference(f1);
+                sv_frame_t af1 = af + resolution;
+                sv_frame_t rf1 = m_above->alignToReference(af1);
                 sv_frame_t bf1 = m_below->alignFromReference(rf1);
 
                 for (sv_frame_t probe = bf + 1; probe <= bf1; ++probe) {
                     if (keyFramesBelow.find(probe) != keyFramesBelow.end()) {
-                        m_fromAboveMap.insert({ f, probe });
+                        m_fromAboveMap.insert({ af, probe });
                         mappedSomething = true;
                     }
                 }
             }
         }
 
         if (!mappedSomething) {
-            m_fromAboveMap.insert({ f, bf });
+            m_fromAboveMap.insert({ af, bf });
         }
     }
 
 #ifdef DEBUG_ALIGNMENT_VIEW
     SVCERR << "AlignmentView " << getId() << "::buildMaps: have "