summaryrefslogtreecommitdiff
path: root/Week1-Pacman/src/demo.c
diff options
context:
space:
mode:
authorBoredGuy <osome3717@gmail.com>2024-12-17 11:11:59 +0300
committerBoredGuy <osome3717@gmail.com>2024-12-17 11:11:59 +0300
commitac7ba6dc94f6eb2d8cf9092391714247bb947148 (patch)
tree3352f8a6eeff07f18c4224f477ae5618aac21b30 /Week1-Pacman/src/demo.c
parentfc3032135f4da3662d6b727c70f22049d6e09231 (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.c28
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,