From e0ab754714f5d7d91fb91394b646cf162c6fe2f5 Mon Sep 17 00:00:00 2001 From: Eric Fischer Date: Mon, 19 Mar 2018 13:46:50 -0700 Subject: [PATCH] Nested attributes do work in Geobuf. Remove coercion to string. --- Makefile | 2 +- geobuf.cpp | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/Makefile b/Makefile index 2a82f45..fa18227 100644 --- a/Makefile +++ b/Makefile @@ -93,7 +93,7 @@ test: tippecanoe tippecanoe-decode $(addsuffix .check,$(TESTS)) raw-tiles-test p rm $@.out $@.mbtiles # Don't test overflow with geobuf, because it fails (https://github.com/mapbox/geobuf/issues/87) -nogeobuf = tests/overflow/out/-z0.json tests/dateline/out/-z5.json +nogeobuf = tests/overflow/out/-z0.json geobuf-test: tippecanoe-json-tool $(addsuffix .checkbuf,$(filter-out $(nogeobuf),$(TESTS))) # For quicker address sanitizer build, hope that regular JSON parsing is tested enough by parallel and join tests diff --git a/geobuf.cpp b/geobuf.cpp index 0f10f2c..e6fc114 100644 --- a/geobuf.cpp +++ b/geobuf.cpp @@ -77,7 +77,7 @@ serial_val readValue(protozero::pbf_reader &pbf) { break; case 6: - sv.type = mvt_string; // stringified JSON + sv.type = mvt_hash; // stringified JSON sv.s = pbf.get_string(); if (sv.s == "null") {