fix boolean operators

2 files changed, 4 insertions, 4 deletions

diff --git a/src/text/collision.cpp b/src/text/collision.cpp
index 70712f0651..aa4cf1c7a2 100644
--- a/src/text/collision.cpp
+++ b/src/text/collision.cpp
@@ -112,7 +112,7 @@ float Collision::getPlacementScale(const GlyphBoxes &glyphs, float minPlacementS
         const float pad = glyph.padding;
 
         if (anchor.x < 0 || anchor.x > 4096 || anchor.y < 0 || anchor.y > 4096) {
-            return -1;
+            return 0;
         }
 
         float minScale = std::fmax(minPlacementScale, glyph.minScale);
@@ -142,7 +142,7 @@ float Collision::getPlacementScale(const GlyphBoxes &glyphs, float minPlacementS
                 // NOTE: this isn't right because there can be glyphs with
                 // the same anchor but differing box offsets.
                 if (na == oa) {
-                    return -1;
+                    return 0;
                 }
 
                 // todo: unhardcode the 8 = tileExtent/tileSize
@@ -175,7 +175,7 @@ float Collision::getPlacementScale(const GlyphBoxes &glyphs, float minPlacementS
                 }
 
                 if (minPlacementScale > maxPlacementScale) {
-                    return -1;
+                    return 0;
                 }
             }
         }
diff --git a/src/text/placement.cpp b/src/text/placement.cpp
index 448b892934..23e2a0ee87 100644
--- a/src/text/placement.cpp
+++ b/src/text/placement.cpp
@@ -225,7 +225,7 @@ Placement Placement::getGlyphs(Anchor &anchor, const vec2<float> &origin, const
         if (!glyph)
             continue;
 
-        if (!(rect && rect.w > 0 && rect.h > 0))
+        if (!rect)
             continue;
 
         const float x = (origin.x + shape.x + glyph.metrics.left - buffer + rect.w / 2) * boxScale;