Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Crash fix: handle dragging a selection rectangle up/left correctly. #1433

Merged
merged 1 commit into from
Jun 16, 2020

Conversation

divVerent
Copy link
Contributor

Previously, this created a Rectf instance with negative size, crashing
the filled rectangle drawing routine.

I previously considered rather making the drawing routine support this
case, but then I started to wonder how to define the rounded rectangle
radius in that case... and then I saw the pre-existing assert in the
other Rectf constructor and decided that the intention was to not allow
this in the first place. So, dragging code fixed instead.

…ft correctly.

Previously, this created a Rectf instance with negative size, crashing
the filled rectangle drawing routine.

I previously considered rather making the drawing routine support this
case, but then I started to wonder how to define the rounded rectangle
radius in that case... and then I saw the pre-existing assert in the
other Rectf constructor and decided that the intention was to not allow
this in the first place. So, dragging code fixed instead.
@Zwatotem Zwatotem added category:code status:needs-review Work needs to be reviewed by other people type:bug labels Jun 15, 2020
@tobbi tobbi merged commit 54905db into SuperTux:master Jun 16, 2020
@divVerent divVerent deleted the divVerent/crash-fix-2 branch June 16, 2020 19:06
@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.

4 participants