Skip to content

Fix assertion failure in triangle drawing on the SDL backend. - #1437

Merged
tobbi merged 1 commit into
SuperTux:masterfrom
divVerent:divVerent/assertion-fix
Jun 19, 2020
Merged

tobbi merged 1 commit into
SuperTux:masterfrom
divVerent:divVerent/assertion-fix

Conversation

@divVerent

Copy link
Copy Markdown
Contributor

Fixed by changing the type of an edge from Rectf to std::pair<Vector, Vector>.

This assertion was introduced in
fa31cab
(and if this causes more issues, we may want to retract this assertion
or turn it into a debug print for now).

Shouldn't affect optimized builds as asserts normally are off when
optimizing via -DNDEBUG.

Fixed by changing the type of an edge from Rectf to std::pair<Vector, Vector>.

This assertion was introduced in
SuperTux@fa31cab
(and if this causes more issues, we may want to retract this assertion
or turn it into a debug print for now).

Shouldn't affect optimized builds as asserts normally are off when
optimizing via -DNDEBUG.
@tobbi tobbi added category:code status:needs-review Work needs to be reviewed by other people labels Jun 17, 2020
@tobbi
tobbi merged commit 09e4482 into SuperTux:master Jun 19, 2020
@mrkubax10 mrkubax10 removed the status:needs-review Work needs to be reviewed by other people label Jul 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants