From defd1edb6f06c8ea6e73c85665ed716ba5836dee Mon Sep 17 00:00:00 2001 From: Andrew Lanzone Date: Sat, 26 Jul 2025 00:59:43 -0700 Subject: [PATCH] Fix clang error and unsigned bug --- apps/openmw/mwgui/alchemywindow.cpp | 2 +- apps/openmw/mwgui/journalwindow.cpp | 3 ++- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/apps/openmw/mwgui/alchemywindow.cpp b/apps/openmw/mwgui/alchemywindow.cpp index d0931057cd..a9aec50448 100644 --- a/apps/openmw/mwgui/alchemywindow.cpp +++ b/apps/openmw/mwgui/alchemywindow.cpp @@ -589,7 +589,7 @@ namespace MWGui if (arg.button == SDL_CONTROLLER_BUTTON_B) { // Remove active ingredients or close the window, starting with right-most slot. - for (size_t i = mIngredients.size() - 1; i >= 0; --i) + for (int i = mIngredients.size() - 1; i >= 0; --i) { if (mIngredients[i]->isUserString("ToolTipType")) { diff --git a/apps/openmw/mwgui/journalwindow.cpp b/apps/openmw/mwgui/journalwindow.cpp index 1ce9f8b024..817ed95975 100644 --- a/apps/openmw/mwgui/journalwindow.cpp +++ b/apps/openmw/mwgui/journalwindow.cpp @@ -901,7 +901,8 @@ namespace } else if (arg.button == SDL_CONTROLLER_BUTTON_DPAD_LEFT || arg.button == SDL_CONTROLLER_BUTTON_LEFTSHOULDER) notifyPrevPage(getWidget(PrevPageBTN)); - else if (arg.button == SDL_CONTROLLER_BUTTON_DPAD_RIGHT || arg.button == SDL_CONTROLLER_BUTTON_RIGHTSHOULDER) + else if (arg.button == SDL_CONTROLLER_BUTTON_DPAD_RIGHT + || arg.button == SDL_CONTROLLER_BUTTON_RIGHTSHOULDER) notifyNextPage(getWidget(NextPageBTN)); return true;