diff options
author | P. J. McDermott <pj@pehjota.net> | 2023-10-16 14:53:56 (EDT) |
---|---|---|
committer | P. J. McDermott <pj@pehjota.net> | 2023-10-16 14:53:56 (EDT) |
commit | 111d144174adf2e7ed7272912315eb60d9e0fd26 (patch) | |
tree | 4dea8e6e2b9c34468aee026338247467d6432192 /src/tk/button.c | |
parent | c5e354b1ff6e15fc25ce39b5c009c0488ffe8824 (diff) | |
download | mazefight-111d144174adf2e7ed7272912315eb60d9e0fd26.zip mazefight-111d144174adf2e7ed7272912315eb60d9e0fd26.tar.gz mazefight-111d144174adf2e7ed7272912315eb60d9e0fd26.tar.bz2 |
tk: Use goto for render errors
Diffstat (limited to 'src/tk/button.c')
-rw-r--r-- | src/tk/button.c | 15 |
1 files changed, 7 insertions, 8 deletions
diff --git a/src/tk/button.c b/src/tk/button.c index fa0cb65..911c024 100644 --- a/src/tk/button.c +++ b/src/tk/button.c @@ -124,10 +124,7 @@ _mftk_button_render(struct mftk_widget *w, SDL_Renderer *renderer, int x, int y) if (SDL_SetRenderDrawColor(renderer, border.r, border.g, border.b, border.a) < 0 || SDL_RenderFillRect(renderer, &rect) < 0) { - SDL_LogError(SDL_LOG_CATEGORY_APPLICATION, - "Couldn't render widget: %s", - SDL_GetError()); - return -1; + goto err; } rect.x += b->border; @@ -138,10 +135,7 @@ _mftk_button_render(struct mftk_widget *w, SDL_Renderer *renderer, int x, int y) MFTK_COLOR_BACK_R, MFTK_COLOR_BACK_G, MFTK_COLOR_BACK_B, MFTK_COLOR_BACK_A) < 0 || SDL_RenderFillRect(renderer, &rect) < 0) { - SDL_LogError(SDL_LOG_CATEGORY_APPLICATION, - "Couldn't render widget: %s", - SDL_GetError()); - return -1; + goto err; } if (mftk_widget_render(b->label, renderer, @@ -150,6 +144,11 @@ _mftk_button_render(struct mftk_widget *w, SDL_Renderer *renderer, int x, int y) } return 0; + + err: + SDL_LogError(SDL_LOG_CATEGORY_APPLICATION, + "Couldn't render widget: %s", SDL_GetError()); + return -1; } static void |