Reordering decimation clean to avoid allocation

chartjs · Apr 12, 2021 · 4bd7e87 · 4bd7e87
1 parent 5b3f260
commit 4bd7e87
Showing 1 changed file with 7 additions and 7 deletions.
diff --git a/src/plugins/plugin.decimation.js b/src/plugins/plugin.decimation.js
@@ -233,6 +233,13 @@ export default {
         return;
       }
 
+      let {start, count} = getStartAndCountOfVisiblePointsSimplified(meta, data);
+      if (count <= 4 * availableWidth) {
+        // No decimation is required until we are above this threshold
+        cleanDecimatedDataset(dataset);
+        return;
+      }
+
       if (isNullOrUndef(_data)) {
         // First time we are seeing this dataset
         // We override the 'data' property with a setter that stores the
@@ -251,13 +258,6 @@ export default {
         });
       }
 
-      let {start, count} = getStartAndCountOfVisiblePointsSimplified(meta, data);
-      if (count <= 4 * availableWidth) {
-        // No decimation is required until we are above this threshold
-        cleanDecimatedDataset(dataset);
-        return;
-      }
-
       // Point the chart to the decimated data
       let decimated;
       switch (options.algorithm) {