From 16a9c16d9f99bd8be9f1bb15442b001fdf2ea759 Mon Sep 17 00:00:00 2001
From: thejch <66577496+thejch@users.noreply.github.com>
Date: Sat, 30 Mar 2024 18:14:26 -0700
Subject: renderer/animations: Fix various inaccurate damage tracking issues
 and offsets (#5297)

---
 src/helpers/Box.cpp | 25 ++++++++++++++-----------
 1 file changed, 14 insertions(+), 11 deletions(-)

(limited to 'src/helpers/Box.cpp')

diff --git a/src/helpers/Box.cpp b/src/helpers/Box.cpp
index e9e0eeeb..8470b509 100644
--- a/src/helpers/Box.cpp
+++ b/src/helpers/Box.cpp
@@ -106,6 +106,11 @@ CBox& CBox::expand(const double& value) {
     w += value * 2.0;
     h += value * 2.0;
 
+    if (w <= 0 || h <= 0) {
+        w = 0;
+        h = 0;
+    }
+
     return *this;
 }
 
@@ -116,22 +121,20 @@ CBox& CBox::noNegativeSize() {
     return *this;
 }
 
-CBox& CBox::intersection(const CBox other) {
-    const float newTop    = std::max(y, other.y);
+CBox CBox::intersection(const CBox other) const {
+    const float newX      = std::max(x, other.x);
+    const float newY      = std::max(y, other.y);
     const float newBottom = std::min(y + h, other.y + other.h);
-    const float newLeft   = std::max(x, other.x);
     const float newRight  = std::min(x + w, other.x + other.w);
-    y                     = newTop;
-    x                     = newLeft;
-    w                     = newRight - newLeft;
-    h                     = newBottom - newTop;
+    float       newW      = newRight - newX;
+    float       newH      = newBottom - newY;
 
-    if (w <= 0 || h <= 0) {
-        w = 0;
-        h = 0;
+    if (newW <= 0 || newH <= 0) {
+        newW = 0;
+        newH = 0;
     }
 
-    return *this;
+    return {newX, newY, newW, newH};
 }
 
 CBox CBox::roundInternal() {
-- 
cgit v1.2.3