Changeset 11915 in josm for trunk

Timestamp:

2017-04-15T02:27:51+02:00 (7 years ago)

Author:

bastiK

Message:

see #7427 - run consistency test in debug mode

File:

• trunk/src/org/openstreetmap/josm/tools/ImageWarp.java (modified) (5 diffs)

trunk/src/org/openstreetmap/josm/tools/ImageWarp.java

@@ -10 +10 @@
 import java.util.Map;
 import java.util.Set;
+
+import org.openstreetmap.josm.Main;
 
 /**
@@ -44 +46 @@
         private final Map<Integer, Map<Integer, Point2D>> cache;
 
-        private static final boolean CONSISTENCY_TEST = false;
+        private final boolean consistencyTest;
         private final Set<Integer> deletedRows;
 
@@ -56 +58 @@
             this.stride = stride;
             this.cache = new HashMap<>();
-            if (CONSISTENCY_TEST) {
+            this.consistencyTest = Main.isDebugEnabled();
+            if (consistencyTest) {
                 deletedRows = new HashSet<>();
             } else {
@@ -91 +94 @@
 
         private Map<Integer, Point2D> getRow(int yIdx) {
-            cleanUp(yIdx - 2);
+            cleanUp(yIdx - 3);
             Map<Integer, Point2D> row = cache.get(yIdx);
             if (row == null) {
                 row = new HashMap<>();
                 cache.put(yIdx, row);
-                if (CONSISTENCY_TEST) {
+                if (consistencyTest) {
                     // should not create a row that has been deleted before
                     if (deletedRows.contains(yIdx)) throw new AssertionError();
-                    // only ever cache 2 rows at once
-                    if (cache.size() > 2) throw new AssertionError();
+                    // only ever cache 3 rows at once
+                    if (cache.size() > 3) throw new AssertionError();
                 }
             }
@@ -109 +112 @@
         private void cleanUp(int yIdx) {
             Map<Integer, Point2D> del = cache.remove(yIdx);
-            if (CONSISTENCY_TEST && del != null) {
+            if (consistencyTest && del != null) {
                 // should delete each row only once
                 if (deletedRows.contains(yIdx)) throw new AssertionError();