From 7445feb845428b46c4c2dae68e450fea1657ef0a Mon Sep 17 00:00:00 2001
From: Eric Fischer <enf@pobox.com>
Date: Fri, 14 Jul 2017 17:59:24 -0700
Subject: [PATCH] Work in the direction of tracking representative attribute
 values

---
 geojson.cpp | 13 +++++++++++++
 mbtiles.hpp | 11 ++++++-----
 2 files changed, 19 insertions(+), 5 deletions(-)

diff --git a/geojson.cpp b/geojson.cpp
index 05b865f..1a9a7bd 100644
--- a/geojson.cpp
+++ b/geojson.cpp
@@ -349,6 +349,7 @@ int serialize_geometry(json_object *geometry, json_object *properties, json_obje
 
 			metakey[m] = properties->keys[i]->string;
 
+			bool track = false;
 			if (properties->values[i] != NULL) {
 				int vt = properties->values[i]->type;
 				std::string val;
@@ -407,20 +408,24 @@ int serialize_geometry(json_object *geometry, json_object *properties, json_obje
 						exit(EXIT_FAILURE);
 					}
 					m++;
+					track = true;
 				} else if (vt == JSON_NUMBER) {
 					tas.type = metatype[m] = mvt_double;
 					metaval[m] = val;
 					m++;
+					track = true;
 				} else if (vt == JSON_TRUE || vt == JSON_FALSE) {
 					tas.type = metatype[m] = mvt_bool;
 					metaval[m] = val;
 					m++;
+					track = true;
 				} else if (vt == JSON_NULL) {
 					;
 				} else {
 					tas.type = metatype[m] = mvt_string;
 					metaval[m] = val;
 					m++;
+					track = true;
 				}
 			}
 
@@ -428,6 +433,14 @@ int serialize_geometry(json_object *geometry, json_object *properties, json_obje
 				auto fk = layermap->find(layername);
 				fk->second.file_keys.insert(tas);
 			}
+
+			if (track) {
+				type_and_string attrib;
+				attrib.type = metatype[m - 1];
+				attrib.string = metaval[m - 1];
+
+				// XXX increment in type_and_string_stats.sample_values
+			}
 		}
 	}
 
diff --git a/mbtiles.hpp b/mbtiles.hpp
index fe0ac67..e25e615 100644
--- a/mbtiles.hpp
+++ b/mbtiles.hpp
@@ -8,17 +8,18 @@ struct type_and_string {
 	int type;
 	std::string string;
 
-	size_t attribute_count = 0;
-	std::set<mvt_value> sample_values;
+	bool operator<(const type_and_string &o) const;
+};
+
+struct type_and_string_stats {
+	std::set<type_and_string> sample_values;
 	double min = INFINITY;
 	double max = -INFINITY;
-
-	bool operator<(const type_and_string &o) const;
 };
 
 struct layermap_entry {
 	size_t id;
-	std::set<type_and_string> file_keys;
+	std::set<type_and_string> file_keys;  // XXX map to type_and_string_stats
 	int minzoom;
 	int maxzoom;