From ea83a9d3a32a9dc931c149e37434654677e1b259 Mon Sep 17 00:00:00 2001 From: Erica Fischer Date: Thu, 8 May 2025 10:19:41 -0700 Subject: [PATCH] Another case where it could try to drop even after already limiting. --- .../out/-z2_--convert-polygons-to-label-points.json | 1 - tile.cpp | 2 ++ 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/tests/ne_110m_admin_0_countries/out/-z2_--convert-polygons-to-label-points.json b/tests/ne_110m_admin_0_countries/out/-z2_--convert-polygons-to-label-points.json index 01201ee7..505b7ade 100644 --- a/tests/ne_110m_admin_0_countries/out/-z2_--convert-polygons-to-label-points.json +++ b/tests/ne_110m_admin_0_countries/out/-z2_--convert-polygons-to-label-points.json @@ -9,7 +9,6 @@ "maxzoom": "2", "minzoom": "0", "name": "tests/ne_110m_admin_0_countries/out/-z2_--convert-polygons-to-label-points.json.check.mbtiles", -"strategies": "[{},{},{\"tiny_polygons\":1}]", "type": "overlay", "version": "2" }, "features": [ diff --git a/tile.cpp b/tile.cpp index 58c234ac..850bb760 100644 --- a/tile.cpp +++ b/tile.cpp @@ -2504,6 +2504,7 @@ long long write_tile(decompressor *geoms, std::atomic *geompos_in, ch // this is maxzoom; ok to stop early still because they said to limit abruptly layer_features.resize(limit_tile_feature_count_at_maxzoom); too_many_features = false; // don't try to drop; we have already truncated + too_many_bytes = false; // don't try to drop; we have already truncated skipped = 0; // doesn't matter that we skipped features; we have truncated } } else if (limit_tile_feature_count != 0) { @@ -2511,6 +2512,7 @@ long long write_tile(decompressor *geoms, std::atomic *geompos_in, ch can_stop_early = false; layer_features.resize(limit_tile_feature_count); too_many_features = false; // don't try to drop; we have already truncated + too_many_bytes = false; // don't try to drop; we have already truncated skipped = 0; // doesn't matter that we skipped features; we have truncated } }