From 3c49fdc99d9177a2751361f3151cc562aa1f77a3 Mon Sep 17 00:00:00 2001 From: John Firebaugh Date: Mon, 16 May 2016 16:33:24 -0700 Subject: [core] Feature IDs are always integers --- include/mbgl/util/feature.hpp | 2 +- platform/node/src/node_feature.cpp | 2 +- src/mbgl/geometry/feature_index.cpp | 6 +----- 3 files changed, 3 insertions(+), 7 deletions(-) diff --git a/include/mbgl/util/feature.hpp b/include/mbgl/util/feature.hpp index c9dbc31769..7747d34ee9 100644 --- a/include/mbgl/util/feature.hpp +++ b/include/mbgl/util/feature.hpp @@ -13,7 +13,7 @@ public: Feature(geometry_type&& geometry_) : mapbox::geometry::feature { std::move(geometry_) } {} - optional id {}; + optional id; }; } // namespace mbgl diff --git a/platform/node/src/node_feature.cpp b/platform/node/src/node_feature.cpp index 7175f9d1fa..1a5e31fe97 100644 --- a/platform/node/src/node_feature.cpp +++ b/platform/node/src/node_feature.cpp @@ -159,7 +159,7 @@ v8::Local toJS(const Feature& feature) { Nan::Set(result, Nan::New("properties").ToLocalChecked(), toJS(feature.properties)); if (feature.id) { - Nan::Set(result, Nan::New("id").ToLocalChecked(), toJS(*feature.id)); + Nan::Set(result, Nan::New("id").ToLocalChecked(), Nan::New(double(*feature.id))); } return scope.Escape(result); diff --git a/src/mbgl/geometry/feature_index.cpp b/src/mbgl/geometry/feature_index.cpp index d3d523af0a..fb68b572b1 100644 --- a/src/mbgl/geometry/feature_index.cpp +++ b/src/mbgl/geometry/feature_index.cpp @@ -123,11 +123,7 @@ void FeatureIndex::addFeature( Feature feature { mapbox::geometry::point() }; feature.properties = geometryTileFeature->getProperties(); - - optional id = geometryTileFeature->getID(); - if (id) { - feature.id = Value(*id); - } + feature.id = geometryTileFeature->getID(); result[layerID].push_back(std::move(feature)); } -- cgit v1.2.1