Merge pull request #102778 from KoBeWi/line_zero

Repiteo · Repiteo · commit d408264fdef9 · 2025-02-12T12:56:39.000-06:00
Prevent errors when Line2D has 0 length
diff --git a/scene/2d/line_2d.cpp b/scene/2d/line_2d.cpp
@@ -299,6 +299,9 @@ void Line2D::_draw() {
 	}
 
 	lb.build();
+	if (lb.indices.is_empty()) {
+		return;
+	}
 
 	RS::get_singleton()->canvas_item_add_triangle_array(
 			get_canvas_item(),
diff --git a/scene/2d/line_builder.cpp b/scene/2d/line_builder.cpp
@@ -112,6 +112,11 @@ void LineBuilder::build() {
 		}
 	}
 
+	if (Math::is_zero_approx(total_distance)) {
+		// Zero-length line, nothing to build.
+		return;
+	}
+
 	if (_interpolate_color) {
 		color0 = gradient->get_color(0);
 	} else {

Original file line number	Diff line number	Diff line change
`@@ -299,6 +299,9 @@ void Line2D::_draw() {`
`299`	`299`	`}`
`300`	`300`
`301`	`301`	`lb.build();`
	`302`	`+ if (lb.indices.is_empty()) {`
	`303`	`+ return;`
	`304`	`+ }`
`302`	`305`
`303`	`306`	`RS::get_singleton()->canvas_item_add_triangle_array(`
`304`	`307`	`get_canvas_item(),`
Original file line number	Diff line number	Diff line change
`@@ -112,6 +112,11 @@ void LineBuilder::build() {`
`112`	`112`	`}`
`113`	`113`	`}`
`114`	`114`
	`115`	`+ if (Math::is_zero_approx(total_distance)) {`
	`116`	`+ // Zero-length line, nothing to build.`
	`117`	`+ return;`
	`118`	`+ }`
	`119`	`+`
`115`	`120`	`if (_interpolate_color) {`
`116`	`121`	`color0 = gradient->get_color(0);`
`117`	`122`	`} else {`