Do not use structured binding

Author: geonove

ddsketch/include/ddsketch.hpp

@@ -20,10 +20,6 @@
 #ifndef _DDSKETCH_HPP_
 #define _DDSKETCH_HPP_
 
-#include <utility>
-#include <type_traits>
-#include <variant>
-#include "store_factory.hpp"
 #include "common_defs.hpp"
 #include "memory_operations.hpp"
 

ddsketch/include/dense_store_impl.hpp

@@ -21,7 +21,6 @@
 #define DENSE_STORE_IMPL_HPP
 
 #include <algorithm>
-#include <filesystem>
 
 #include "common_defs.hpp"
 

ddsketch/test/DDSketchTest.cpp

@@ -97,7 +97,7 @@ void test_adding(SketchType& sketch, const std::vector<double>& values, double r
     // Test individual additions
     sketch.clear();
     for (const double& value : values) {
-        sketch.update(value);
+      sketch.update(value);
     }
     assert_encodes(sketch, values, relative_accuracy);
 
@@ -106,11 +106,13 @@ void test_adding(SketchType& sketch, const std::vector<double>& values, double r
     auto sketch_weighted(sketch);
     std::map<double, int> value_counts;
     for (const double& value : values) {
-        value_counts[value]++;
+      value_counts[value]++;
     }
 
-    for (const auto& [value, count] : value_counts) {
-        sketch_weighted.update(value, count);
+    for (const auto& value_count : value_counts) {
+      double value = value_count.first;
+      double count = value_count.second;
+      sketch_weighted.update(value, count);
     }
     assert_encodes(sketch_weighted, values, relative_accuracy);
 }