[core] There's only ever one icon quad

1 files changed, 130 insertions, 132 deletions

diff --git a/test/text/quads.test.cpp b/test/text/quads.test.cpp
index 69aba1a86b..42bc0f2048 100644
--- a/test/text/quads.test.cpp
+++ b/test/text/quads.test.cpp
@@ -22,23 +22,22 @@ TEST(getIconQuads, normal) {
     GeometryCoordinates line;
     Shaping shapedText;
 
-    SymbolQuads quads =
-        getIconQuads(anchor, shapedIcon, line, layout, SymbolPlacementType::Point, shapedText);
+    SymbolQuad quad =
+        getIconQuad(anchor, shapedIcon, line, layout, SymbolPlacementType::Point, shapedText);
 
-    ASSERT_EQ(quads.size(), 1u);
-    ASSERT_EQ(quads[0].anchorPoint.x, 2);
-    ASSERT_EQ(quads[0].anchorPoint.y, 3);
-    ASSERT_EQ(quads[0].tl.x, -8);
-    ASSERT_EQ(quads[0].tl.y, -6);
-    ASSERT_EQ(quads[0].tr.x, 7);
-    ASSERT_EQ(quads[0].tr.y, -6);
-    ASSERT_EQ(quads[0].bl.x, -8);
-    ASSERT_EQ(quads[0].bl.y, 5);
-    ASSERT_EQ(quads[0].br.x, 7);
-    ASSERT_EQ(quads[0].br.y, 5);
-    ASSERT_EQ(quads[0].anchorAngle, 0.0f);
-    ASSERT_EQ(quads[0].glyphAngle, 0.0f);
-    ASSERT_EQ(quads[0].minScale, 0.5f);
+    ASSERT_EQ(quad.anchorPoint.x, 2);
+    ASSERT_EQ(quad.anchorPoint.y, 3);
+    ASSERT_EQ(quad.tl.x, -8);
+    ASSERT_EQ(quad.tl.y, -6);
+    ASSERT_EQ(quad.tr.x, 7);
+    ASSERT_EQ(quad.tr.y, -6);
+    ASSERT_EQ(quad.bl.x, -8);
+    ASSERT_EQ(quad.bl.y, 5);
+    ASSERT_EQ(quad.br.x, 7);
+    ASSERT_EQ(quad.br.y, 5);
+    ASSERT_EQ(quad.anchorAngle, 0.0f);
+    ASSERT_EQ(quad.glyphAngle, 0.0f);
+    ASSERT_EQ(quad.minScale, 0.5f);
 }
 
 TEST(getIconQuads, style) {
@@ -61,23 +60,22 @@ TEST(getIconQuads, style) {
     // none
     {
         SymbolLayoutProperties::Evaluated layout;
-        SymbolQuads quads =
-            getIconQuads(anchor, shapedIcon, line, layout, SymbolPlacementType::Point, shapedText);
+        SymbolQuad quad =
+            getIconQuad(anchor, shapedIcon, line, layout, SymbolPlacementType::Point, shapedText);
 
-        ASSERT_EQ(quads.size(), 1u);
-        ASSERT_EQ(quads[0].anchorPoint.x, 0);
-        ASSERT_EQ(quads[0].anchorPoint.y, 0);
-        ASSERT_EQ(quads[0].tl.x, -11);
-        ASSERT_EQ(quads[0].tl.y, -11);
-        ASSERT_EQ(quads[0].tr.x, 9);
-        ASSERT_EQ(quads[0].tr.y, -11);
-        ASSERT_EQ(quads[0].bl.x, -11);
-        ASSERT_EQ(quads[0].bl.y, 9);
-        ASSERT_EQ(quads[0].br.x, 9);
-        ASSERT_EQ(quads[0].br.y, 9);
-        ASSERT_EQ(quads[0].anchorAngle, 0.0f);
-        ASSERT_EQ(quads[0].glyphAngle, 0.0f);
-        ASSERT_EQ(quads[0].minScale, 0.5f);
+        ASSERT_EQ(quad.anchorPoint.x, 0);
+        ASSERT_EQ(quad.anchorPoint.y, 0);
+        ASSERT_EQ(quad.tl.x, -11);
+        ASSERT_EQ(quad.tl.y, -11);
+        ASSERT_EQ(quad.tr.x, 9);
+        ASSERT_EQ(quad.tr.y, -11);
+        ASSERT_EQ(quad.bl.x, -11);
+        ASSERT_EQ(quad.bl.y, 9);
+        ASSERT_EQ(quad.br.x, 9);
+        ASSERT_EQ(quad.br.y, 9);
+        ASSERT_EQ(quad.anchorAngle, 0.0f);
+        ASSERT_EQ(quad.glyphAngle, 0.0f);
+        ASSERT_EQ(quad.minScale, 0.5f);
     }
 
     // width
@@ -85,17 +83,17 @@ TEST(getIconQuads, style) {
         SymbolLayoutProperties::Evaluated layout;
         layout.get<TextSize>() = 24.0f;
         layout.get<IconTextFit>() = IconTextFitType::Width;
-        SymbolQuads quads =
-            getIconQuads(anchor, shapedIcon, line, layout, SymbolPlacementType::Point, shapedText);
+        SymbolQuad quad =
+            getIconQuad(anchor, shapedIcon, line, layout, SymbolPlacementType::Point, shapedText);
 
-        ASSERT_EQ(quads[0].tl.x, -60);
-        ASSERT_EQ(quads[0].tl.y, 0);
-        ASSERT_EQ(quads[0].tr.x, 20);
-        ASSERT_EQ(quads[0].tr.y, 0);
-        ASSERT_EQ(quads[0].bl.x, -60);
-        ASSERT_EQ(quads[0].bl.y, 20);
-        ASSERT_EQ(quads[0].br.x, 20);
-        ASSERT_EQ(quads[0].br.y, 20);
+        ASSERT_EQ(quad.tl.x, -60);
+        ASSERT_EQ(quad.tl.y, 0);
+        ASSERT_EQ(quad.tr.x, 20);
+        ASSERT_EQ(quad.tr.y, 0);
+        ASSERT_EQ(quad.bl.x, -60);
+        ASSERT_EQ(quad.bl.y, 20);
+        ASSERT_EQ(quad.br.x, 20);
+        ASSERT_EQ(quad.br.y, 20);
     }
 
     // width x textSize
@@ -103,17 +101,17 @@ TEST(getIconQuads, style) {
         SymbolLayoutProperties::Evaluated layout;
         layout.get<TextSize>() = 12.0f;
         layout.get<IconTextFit>() = IconTextFitType::Width;
-        SymbolQuads quads =
-            getIconQuads(anchor, shapedIcon, line, layout, SymbolPlacementType::Point, shapedText);
+        SymbolQuad quad =
+            getIconQuad(anchor, shapedIcon, line, layout, SymbolPlacementType::Point, shapedText);
 
-        ASSERT_EQ(quads[0].tl.x, -30);
-        ASSERT_EQ(quads[0].tl.y, -5);
-        ASSERT_EQ(quads[0].tr.x, 10);
-        ASSERT_EQ(quads[0].tr.y, -5);
-        ASSERT_EQ(quads[0].bl.x, -30);
-        ASSERT_EQ(quads[0].bl.y, 15);
-        ASSERT_EQ(quads[0].br.x, 10);
-        ASSERT_EQ(quads[0].br.y, 15);
+        ASSERT_EQ(quad.tl.x, -30);
+        ASSERT_EQ(quad.tl.y, -5);
+        ASSERT_EQ(quad.tr.x, 10);
+        ASSERT_EQ(quad.tr.y, -5);
+        ASSERT_EQ(quad.bl.x, -30);
+        ASSERT_EQ(quad.bl.y, 15);
+        ASSERT_EQ(quad.br.x, 10);
+        ASSERT_EQ(quad.br.y, 15);
     }
 
     // width x textSize + padding
@@ -125,17 +123,17 @@ TEST(getIconQuads, style) {
         layout.get<IconTextFitPadding>()[1] = 10.0f;
         layout.get<IconTextFitPadding>()[2] = 5.0f;
         layout.get<IconTextFitPadding>()[3] = 10.0f;
-        SymbolQuads quads =
-            getIconQuads(anchor, shapedIcon, line, layout, SymbolPlacementType::Point, shapedText);
+        SymbolQuad quad =
+            getIconQuad(anchor, shapedIcon, line, layout, SymbolPlacementType::Point, shapedText);
 
-        ASSERT_EQ(quads[0].tl.x, -40);
-        ASSERT_EQ(quads[0].tl.y, -10);
-        ASSERT_EQ(quads[0].tr.x, 20);
-        ASSERT_EQ(quads[0].tr.y, -10);
-        ASSERT_EQ(quads[0].bl.x, -40);
-        ASSERT_EQ(quads[0].bl.y, 20);
-        ASSERT_EQ(quads[0].br.x, 20);
-        ASSERT_EQ(quads[0].br.y, 20);
+        ASSERT_EQ(quad.tl.x, -40);
+        ASSERT_EQ(quad.tl.y, -10);
+        ASSERT_EQ(quad.tr.x, 20);
+        ASSERT_EQ(quad.tr.y, -10);
+        ASSERT_EQ(quad.bl.x, -40);
+        ASSERT_EQ(quad.bl.y, 20);
+        ASSERT_EQ(quad.br.x, 20);
+        ASSERT_EQ(quad.br.y, 20);
     }
 
     // height
@@ -143,17 +141,17 @@ TEST(getIconQuads, style) {
         SymbolLayoutProperties::Evaluated layout;
         layout.get<TextSize>() = 24.0f;
         layout.get<IconTextFit>() = IconTextFitType::Height;
-        SymbolQuads quads =
-            getIconQuads(anchor, shapedIcon, line, layout, SymbolPlacementType::Point, shapedText);
+        SymbolQuad quad =
+            getIconQuad(anchor, shapedIcon, line, layout, SymbolPlacementType::Point, shapedText);
 
-        ASSERT_EQ(quads[0].tl.x, -30);
-        ASSERT_EQ(quads[0].tl.y, -10);
-        ASSERT_EQ(quads[0].tr.x, -10);
-        ASSERT_EQ(quads[0].tr.y, -10);
-        ASSERT_EQ(quads[0].bl.x, -30);
-        ASSERT_EQ(quads[0].bl.y, 30);
-        ASSERT_EQ(quads[0].br.x, -10);
-        ASSERT_EQ(quads[0].br.y, 30);
+        ASSERT_EQ(quad.tl.x, -30);
+        ASSERT_EQ(quad.tl.y, -10);
+        ASSERT_EQ(quad.tr.x, -10);
+        ASSERT_EQ(quad.tr.y, -10);
+        ASSERT_EQ(quad.bl.x, -30);
+        ASSERT_EQ(quad.bl.y, 30);
+        ASSERT_EQ(quad.br.x, -10);
+        ASSERT_EQ(quad.br.y, 30);
     }
 
     // height x textSize
@@ -161,17 +159,17 @@ TEST(getIconQuads, style) {
         SymbolLayoutProperties::Evaluated layout;
         layout.get<TextSize>() = 12.0f;
         layout.get<IconTextFit>() = IconTextFitType::Height;
-        SymbolQuads quads =
-            getIconQuads(anchor, shapedIcon, line, layout, SymbolPlacementType::Point, shapedText);
+        SymbolQuad quad =
+            getIconQuad(anchor, shapedIcon, line, layout, SymbolPlacementType::Point, shapedText);
 
-        ASSERT_EQ(quads[0].tl.x, -20);
-        ASSERT_EQ(quads[0].tl.y, -5);
-        ASSERT_EQ(quads[0].tr.x, 0);
-        ASSERT_EQ(quads[0].tr.y, -5);
-        ASSERT_EQ(quads[0].bl.x, -20);
-        ASSERT_EQ(quads[0].bl.y, 15);
-        ASSERT_EQ(quads[0].br.x, 0);
-        ASSERT_EQ(quads[0].br.y, 15);
+        ASSERT_EQ(quad.tl.x, -20);
+        ASSERT_EQ(quad.tl.y, -5);
+        ASSERT_EQ(quad.tr.x, 0);
+        ASSERT_EQ(quad.tr.y, -5);
+        ASSERT_EQ(quad.bl.x, -20);
+        ASSERT_EQ(quad.bl.y, 15);
+        ASSERT_EQ(quad.br.x, 0);
+        ASSERT_EQ(quad.br.y, 15);
     }
 
     // height x textSize + padding
@@ -183,17 +181,17 @@ TEST(getIconQuads, style) {
         layout.get<IconTextFitPadding>()[1] = 10.0f;
         layout.get<IconTextFitPadding>()[2] = 5.0f;
         layout.get<IconTextFitPadding>()[3] = 10.0f;
-        SymbolQuads quads =
-            getIconQuads(anchor, shapedIcon, line, layout, SymbolPlacementType::Point, shapedText);
+        SymbolQuad quad =
+            getIconQuad(anchor, shapedIcon, line, layout, SymbolPlacementType::Point, shapedText);
 
-        ASSERT_EQ(quads[0].tl.x, -30);
-        ASSERT_EQ(quads[0].tl.y, -10);
-        ASSERT_EQ(quads[0].tr.x, 10);
-        ASSERT_EQ(quads[0].tr.y, -10);
-        ASSERT_EQ(quads[0].bl.x, -30);
-        ASSERT_EQ(quads[0].bl.y, 20);
-        ASSERT_EQ(quads[0].br.x, 10);
-        ASSERT_EQ(quads[0].br.y, 20);
+        ASSERT_EQ(quad.tl.x, -30);
+        ASSERT_EQ(quad.tl.y, -10);
+        ASSERT_EQ(quad.tr.x, 10);
+        ASSERT_EQ(quad.tr.y, -10);
+        ASSERT_EQ(quad.bl.x, -30);
+        ASSERT_EQ(quad.bl.y, 20);
+        ASSERT_EQ(quad.br.x, 10);
+        ASSERT_EQ(quad.br.y, 20);
     }
 
     // both
@@ -201,17 +199,17 @@ TEST(getIconQuads, style) {
         SymbolLayoutProperties::Evaluated layout;
         layout.get<TextSize>() = 24.0f;
         layout.get<IconTextFit>() = IconTextFitType::Both;
-        SymbolQuads quads =
-            getIconQuads(anchor, shapedIcon, line, layout, SymbolPlacementType::Point, shapedText);
+        SymbolQuad quad =
+            getIconQuad(anchor, shapedIcon, line, layout, SymbolPlacementType::Point, shapedText);
 
-        ASSERT_EQ(quads[0].tl.x, -60);
-        ASSERT_EQ(quads[0].tl.y, -10);
-        ASSERT_EQ(quads[0].tr.x, 20);
-        ASSERT_EQ(quads[0].tr.y, -10);
-        ASSERT_EQ(quads[0].bl.x, -60);
-        ASSERT_EQ(quads[0].bl.y, 30);
-        ASSERT_EQ(quads[0].br.x, 20);
-        ASSERT_EQ(quads[0].br.y, 30);
+        ASSERT_EQ(quad.tl.x, -60);
+        ASSERT_EQ(quad.tl.y, -10);
+        ASSERT_EQ(quad.tr.x, 20);
+        ASSERT_EQ(quad.tr.y, -10);
+        ASSERT_EQ(quad.bl.x, -60);
+        ASSERT_EQ(quad.bl.y, 30);
+        ASSERT_EQ(quad.br.x, 20);
+        ASSERT_EQ(quad.br.y, 30);
     }
 
     // both x textSize
@@ -219,17 +217,17 @@ TEST(getIconQuads, style) {
         SymbolLayoutProperties::Evaluated layout;
         layout.get<TextSize>() = 12.0f;
         layout.get<IconTextFit>() = IconTextFitType::Both;
-        SymbolQuads quads =
-            getIconQuads(anchor, shapedIcon, line, layout, SymbolPlacementType::Point, shapedText);
+        SymbolQuad quad =
+            getIconQuad(anchor, shapedIcon, line, layout, SymbolPlacementType::Point, shapedText);
 
-        ASSERT_EQ(quads[0].tl.x, -30);
-        ASSERT_EQ(quads[0].tl.y, -5);
-        ASSERT_EQ(quads[0].tr.x, 10);
-        ASSERT_EQ(quads[0].tr.y, -5);
-        ASSERT_EQ(quads[0].bl.x, -30);
-        ASSERT_EQ(quads[0].bl.y, 15);
-        ASSERT_EQ(quads[0].br.x, 10);
-        ASSERT_EQ(quads[0].br.y, 15);
+        ASSERT_EQ(quad.tl.x, -30);
+        ASSERT_EQ(quad.tl.y, -5);
+        ASSERT_EQ(quad.tr.x, 10);
+        ASSERT_EQ(quad.tr.y, -5);
+        ASSERT_EQ(quad.bl.x, -30);
+        ASSERT_EQ(quad.bl.y, 15);
+        ASSERT_EQ(quad.br.x, 10);
+        ASSERT_EQ(quad.br.y, 15);
     }
 
     // both x textSize + padding
@@ -241,17 +239,17 @@ TEST(getIconQuads, style) {
         layout.get<IconTextFitPadding>()[1] = 10.0f;
         layout.get<IconTextFitPadding>()[2] = 5.0f;
         layout.get<IconTextFitPadding>()[3] = 10.0f;
-        SymbolQuads quads =
-            getIconQuads(anchor, shapedIcon, line, layout, SymbolPlacementType::Point, shapedText);
+        SymbolQuad quad =
+            getIconQuad(anchor, shapedIcon, line, layout, SymbolPlacementType::Point, shapedText);
 
-        ASSERT_EQ(quads[0].tl.x, -40);
-        ASSERT_EQ(quads[0].tl.y, -10);
-        ASSERT_EQ(quads[0].tr.x, 20);
-        ASSERT_EQ(quads[0].tr.y, -10);
-        ASSERT_EQ(quads[0].bl.x, -40);
-        ASSERT_EQ(quads[0].bl.y, 20);
-        ASSERT_EQ(quads[0].br.x, 20);
-        ASSERT_EQ(quads[0].br.y, 20);
+        ASSERT_EQ(quad.tl.x, -40);
+        ASSERT_EQ(quad.tl.y, -10);
+        ASSERT_EQ(quad.tr.x, 20);
+        ASSERT_EQ(quad.tr.y, -10);
+        ASSERT_EQ(quad.bl.x, -40);
+        ASSERT_EQ(quad.bl.y, 20);
+        ASSERT_EQ(quad.br.x, 20);
+        ASSERT_EQ(quad.br.y, 20);
     }
 
     // both x textSize + padding t/r/b/l
@@ -263,17 +261,17 @@ TEST(getIconQuads, style) {
         layout.get<IconTextFitPadding>()[1] = 5.0f;
         layout.get<IconTextFitPadding>()[2] = 10.0f;
         layout.get<IconTextFitPadding>()[3] = 15.0f;
-        SymbolQuads quads =
-            getIconQuads(anchor, shapedIcon, line, layout, SymbolPlacementType::Point, shapedText);
+        SymbolQuad quad =
+            getIconQuad(anchor, shapedIcon, line, layout, SymbolPlacementType::Point, shapedText);
 
-        ASSERT_EQ(quads[0].tl.x, -45);
-        ASSERT_EQ(quads[0].tl.y, -5);
-        ASSERT_EQ(quads[0].tr.x, 15);
-        ASSERT_EQ(quads[0].tr.y, -5);
-        ASSERT_EQ(quads[0].bl.x, -45);
-        ASSERT_EQ(quads[0].bl.y, 25);
-        ASSERT_EQ(quads[0].br.x, 15);
-        ASSERT_EQ(quads[0].br.y, 25);
+        ASSERT_EQ(quad.tl.x, -45);
+        ASSERT_EQ(quad.tl.y, -5);
+        ASSERT_EQ(quad.tr.x, 15);
+        ASSERT_EQ(quad.tr.y, -5);
+        ASSERT_EQ(quad.bl.x, -45);
+        ASSERT_EQ(quad.bl.y, 25);
+        ASSERT_EQ(quad.br.x, 15);
+        ASSERT_EQ(quad.br.y, 25);
     }
 }