Save empty tiles at the bottom of the pyramid, per mapbox/geojson-vt-…

…cpp#47 (comment) (#65)
mapbox · Jun 26, 2016 · 5368e63 · 5368e63
1 parent 5bd55d7
commit 5368e63
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 6 deletions.
diff --git a/src/index.js b/src/index.js
@@ -147,10 +147,12 @@ GeoJSONVT.prototype.splitTile = function (features, z, x, y, cz, cx, cy) {
 
         if (debug > 1) console.timeEnd('clipping');
 
-        if (tl) stack.push(tl, z + 1, x * 2,     y * 2);
-        if (bl) stack.push(bl, z + 1, x * 2,     y * 2 + 1);
-        if (tr) stack.push(tr, z + 1, x * 2 + 1, y * 2);
-        if (br) stack.push(br, z + 1, x * 2 + 1, y * 2 + 1);
+        if (features.length) {
+            stack.push(tl || [], z + 1, x * 2,     y * 2);
+            stack.push(bl || [], z + 1, x * 2,     y * 2 + 1);
+            stack.push(tr || [], z + 1, x * 2 + 1, y * 2);
+            stack.push(br || [], z + 1, x * 2 + 1, y * 2 + 1);
+        }
     }
 
     return solid;

diff --git a/test/fixtures/us-states-tiles.json b/test/fixtures/us-states-tiles.json
diff --git a/test/test-get-tile.js b/test/test-get-tile.js
@@ -27,7 +27,7 @@ test('getTile: us-states.json', function (t) {
     t.equal(index.getTile(11, 800, 400), null, 'non-existing tile');
     t.equal(index.getTile(-5, 123.25, 400.25), null, 'invalid tile');
 
-    t.equal(index.total, 29);
+    t.equal(index.total, 37);
 
     t.end();
 });