RenderedTarget: Fix incorrect assertions

adazem009 · adazem009 · commit 8a5a00c4bc18 · 2024-02-25T19:04:19.000+01:00
diff --git a/src/renderedtarget.cpp b/src/renderedtarget.cpp
@@ -379,7 +379,7 @@ QRectF RenderedTarget::getBoundsForBubble() const
     if (rect.height() > slice)
         rect.setBottom(rect.top() - slice);
 
-    Q_ASSERT(rect.height() <= 8);
+    Q_ASSERT(rect.height() <= 8 || std::abs(rect.bottom()) == std::numeric_limits<double>::infinity() || std::abs(rect.top()) == std::numeric_limits<double>::infinity());
     return QRectF(QPointF(rect.left(), rect.top()), QPointF(rect.right(), rect.bottom()));
 }
 
@@ -441,7 +441,6 @@ void RenderedTarget::mouseReleaseEvent(QMouseEvent *event)
 {
     m_clicked = false;
     Q_ASSERT(m_mouseArea);
-    Q_ASSERT(m_engine);
 
     // Stop dragging
     if (m_mouseArea->draggedSprite() == this)

Original file line number	Diff line number	Diff line change
`@@ -379,7 +379,7 @@ QRectF RenderedTarget::getBoundsForBubble() const`
`379`	`379`	`if (rect.height() > slice)`
`380`	`380`	`rect.setBottom(rect.top() - slice);`
`381`	`381`
`382`		`- Q_ASSERT(rect.height() <= 8);`
	`382`	`+ Q_ASSERT(rect.height() <= 8 \|\| std::abs(rect.bottom()) == std::numeric_limits<double>::infinity() \|\| std::abs(rect.top()) == std::numeric_limits<double>::infinity());`
`383`	`383`	`return QRectF(QPointF(rect.left(), rect.top()), QPointF(rect.right(), rect.bottom()));`
`384`	`384`	`}`
`385`	`385`
`@@ -441,7 +441,6 @@ void RenderedTarget::mouseReleaseEvent(QMouseEvent *event)`
`441`	`441`	`{`
`442`	`442`	`m_clicked = false;`
`443`	`443`	`Q_ASSERT(m_mouseArea);`
`444`		`- Q_ASSERT(m_engine);`
`445`	`444`
`446`	`445`	`// Stop dragging`
`447`	`446`	`if (m_mouseArea->draggedSprite() == this)`