clang tidy

impeller/entity/geometry/fill_path_geometry.cc

@@ -22,7 +22,8 @@ GeometryResult FillPathGeometry::GetPositionBuffer(
     RenderPass& pass) const {
   auto& host_buffer = renderer.GetTransientsBuffer();
 
-  if (path_.GetBoundingBox()->IsEmpty()) {
+  const auto& bounding_box = path_.GetBoundingBox();
+  if (bounding_box.has_value() && bounding_box->IsEmpty()) {
     return GeometryResult{
         .type = PrimitiveType::kTriangle,
         .vertex_buffer =
@@ -93,7 +94,8 @@ GeometryResult FillPathGeometry::GetPositionUVBuffer(
     RenderPass& pass) const {
   using VS = TextureFillVertexShader;
 
-  if (path_.GetBoundingBox()->IsEmpty()) {
+  const auto& bounding_box = path_.GetBoundingBox();
+  if (bounding_box.has_value() && bounding_box->IsEmpty()) {
     return GeometryResult{
         .type = PrimitiveType::kTriangle,
         .vertex_buffer =

impeller/tessellator/tessellator_unittests.cc

@@ -17,7 +17,9 @@ TEST(TessellatorTest, TessellatorBuilderReturnsCorrectResultStatus) {
   // Zero points.
   {
     Tessellator t;
-    auto path = PathBuilder{}.TakePath(FillType::kOdd);
+    auto path = PathBuilder{}
+                    .AddRect(Rect::MakeLTRB(0, 0, 100, 100))
+                    .TakePath(FillType::kOdd);
     Tessellator::Result result = t.Tessellate(
         path, 1.0f,
         [](const float* vertices, size_t vertices_count,