diff options
author | BoredGuy <osome3717@gmail.com> | 2024-12-17 11:11:59 +0300 |
---|---|---|
committer | BoredGuy <osome3717@gmail.com> | 2024-12-17 11:11:59 +0300 |
commit | ac7ba6dc94f6eb2d8cf9092391714247bb947148 (patch) | |
tree | 3352f8a6eeff07f18c4224f477ae5618aac21b30 /Week1-Pacman/src/demo.c | |
parent | fc3032135f4da3662d6b727c70f22049d6e09231 (diff) |
Fix embarrassing bug
- The position and size of the source rectangle for animations was
dependent on the display scale, this is incredibly stupid and shouldn't have happened.
Diffstat (limited to 'Week1-Pacman/src/demo.c')
-rw-r--r-- | Week1-Pacman/src/demo.c | 28 |
1 files changed, 4 insertions, 24 deletions
diff --git a/Week1-Pacman/src/demo.c b/Week1-Pacman/src/demo.c index 85ee2a3..2ff021e 100644 --- a/Week1-Pacman/src/demo.c +++ b/Week1-Pacman/src/demo.c @@ -19,18 +19,8 @@ void demo_rendercopy(struct demo* demo, const float scale_x = demo->display_scale_x; const float scale_y = demo->display_scale_y; - SDL_Rect new_srect; SDL_Rect new_drect; - if(s_rect) { - new_srect = (SDL_Rect){ - .x = s_rect->x * scale_x, - .y = s_rect->y * scale_y, - .w = s_rect->w * scale_x, - .h = s_rect->h * scale_y - }; - } - if(d_rect) { new_drect = (SDL_Rect){ .x = d_rect->x * scale_x, @@ -40,9 +30,9 @@ void demo_rendercopy(struct demo* demo, }; } - //Handle edge case where s_rect or d_rect are NULL + //Handle edge case where d_rect is NULL SDL_RenderCopy(demo->ren, texture, - s_rect ? &new_srect : NULL, + s_rect, d_rect ? &new_drect : NULL); } @@ -56,18 +46,8 @@ void demo_rendercopy_ex(struct demo* demo, const float scale_x = demo->display_scale_x; const float scale_y = demo->display_scale_y; - SDL_Rect new_srect; SDL_Rect new_drect; - if(s_rect) { - new_srect = (SDL_Rect){ - .x = s_rect->x * scale_x, - .y = s_rect->y * scale_y, - .w = s_rect->w * scale_x, - .h = s_rect->h * scale_y - }; - } - if(d_rect) { new_drect = (SDL_Rect){ .x = d_rect->x * scale_x, @@ -77,9 +57,9 @@ void demo_rendercopy_ex(struct demo* demo, }; } - //Handle edge case where s_rect or d_rect are NULL + //Handle edge case where d_rect is NULL SDL_RenderCopyEx(demo->ren, texture, - s_rect ? &new_srect : NULL, + s_rect, d_rect ? &new_drect : NULL, angle, center, |