From 05acbfc1f1c03b32bb5e9030220445e99df0ad23 Mon Sep 17 00:00:00 2001 From: Toby Driscoll Date: Tue, 30 Jun 2026 11:57:48 -0400 Subject: [PATCH 1/2] Build Rectangle negation via canonical constructor Negating a Rectangle went through the lowercase rectangle(-vertices(r)) builder, while every other arithmetic op uses Rectangle(center, radii, rotation) directly. Negation is a point reflection through the origin: center -> -center, radii and rotation unchanged (the vertex-offset set is closed under negation), so it can use the canonical constructor too. Co-Authored-By: Claude Opus 4.8 --- src/polygons.jl | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/polygons.jl b/src/polygons.jl index dedbf84..3f039a6 100644 --- a/src/polygons.jl +++ b/src/polygons.jl @@ -335,7 +335,7 @@ inv(r::Rectangle) = inv(Polygon(r)) # these provide shortcuts Base.:+(r::Rectangle, z::Number) = Rectangle(r.center + z, r.radii, r.rotation) Base.:+(z::Number, r::Rectangle) = r + z -Base.:-(r::Rectangle) = rectangle(-vertices(r)) +Base.:-(r::Rectangle) = Rectangle(-r.center, r.radii, r.rotation) Base.:-(r::Rectangle, z::Number) = Rectangle(r.center - z, r.radii, r.rotation) Base.:-(z::Number, r::Rectangle) = (-r) + z From 20ae65d61a777aef51b108ebfb7796e53feddcd7 Mon Sep 17 00:00:00 2001 From: Toby Driscoll Date: Tue, 30 Jun 2026 11:59:05 -0400 Subject: [PATCH 2/2] changed CI to include LTS, not fixed versions --- .github/workflows/CI.yml | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/.github/workflows/CI.yml b/.github/workflows/CI.yml index 9b49721..3e35185 100644 --- a/.github/workflows/CI.yml +++ b/.github/workflows/CI.yml @@ -23,8 +23,7 @@ jobs: fail-fast: false matrix: version: - - '1.10' - - '1.11' + - 'lts' - '1' os: - ubuntu-latest