comparison layer/ScrollableMagRangeCache.cpp @ 1411:6dc3a36d9794

Sample a column only if it is set (avoids flash of mis-ranged material when scrolling with view normalisation on)

 */
 
 #include "ScrollableMagRangeCache.h"
 
 #include "base/HitCount.h"
+#include "base/Debug.h"
 
 #include <iostream>
 using namespace std;
 
 //#define DEBUG_SCROLLABLE_MAG_RANGE_CACHE 1

     
     int dx = (v->getXForFrame(m_startFrame) -
               v->getXForFrame(newStartFrame));
 
 #ifdef DEBUG_SCROLLABLE_MAG_RANGE_CACHE
-    cerr << "ScrollableMagRangeCache::scrollTo: start frame " << m_startFrame
-         << " -> " << newStartFrame << ", dx = " << dx << endl;
+    SVDEBUG << "ScrollableMagRangeCache::scrollTo: start frame " << m_startFrame
+            << " -> " << newStartFrame << ", dx = " << dx << endl;
 #endif
 
     if (m_startFrame == newStartFrame) {
         // haven't moved
         count.hit();

                          m_ranges.begin() + dx, m_ranges.end());
         m_ranges = newRanges;
     }
 
 #ifdef DEBUG_SCROLLABLE_MAG_RANGE_CACHE
-    cerr << "maxes (" << m_ranges.size() << ") now: ";
+    SVDEBUG << "maxes (" << m_ranges.size() << ") now: ";
     for (int i = 0; in_range_for(m_ranges, i); ++i) {
-        cerr << m_ranges[i].getMax() << " ";
+        SVDEBUG << m_ranges[i].getMax() << " ";
     }
-    cerr << endl;
+    SVDEBUG << endl;
 #endif
 }
 
 MagnitudeRange
 ScrollableMagRangeCache::getRange(int x, int count) const
 {
     MagnitudeRange r;
 #ifdef DEBUG_SCROLLABLE_MAG_RANGE_CACHE
-    cerr << "ScrollableMagRangeCache::getRange(" << x << ", " << count << ")" << endl;
+    SVDEBUG << "ScrollableMagRangeCache::getRange(" << x << ", " << count << ")" << endl;
 #endif
     for (int i = 0; i < count; ++i) {
-        r.sample(m_ranges.at(x + i));
+        const auto &cr = m_ranges.at(x + i);
+        if (cr.isSet()) {
+            r.sample(cr);
+        }
+#ifdef DEBUG_SCROLLABLE_MAG_RANGE_CACHE
+        SVDEBUG << cr.getMin() << "->" << cr.getMax() << " ";
+#endif
     }
+#ifdef DEBUG_SCROLLABLE_MAG_RANGE_CACHE
+    SVDEBUG << endl;
+#endif
     return r;
 }
 
 void
 ScrollableMagRangeCache::sampleColumn(int column, const MagnitudeRange &r)
 {
     if (!in_range_for(m_ranges, column)) {
-        cerr << "ERROR: ScrollableMagRangeCache::sampleColumn: column " << column
-             << " is out of range for cache of width " << m_ranges.size()
-             << " (with start frame " << m_startFrame << ")" << endl;
+        SVCERR << "ERROR: ScrollableMagRangeCache::sampleColumn: column " << column
+               << " is out of range for cache of width " << m_ranges.size()
+               << " (with start frame " << m_startFrame << ")" << endl;
         throw logic_error("column out of range");
     } else {
         m_ranges[column].sample(r);
     }
 }