christoph/cpp-masssprings
1
Fork 0
Code Releases Activity

fix compiler warnings

Browse Source
This commit is contained in:
Christoph Urlacher
2026-02-27 12:56:05 +01:00
parent 9d8b364330
commit db694838a7
3 changed files with 27 additions and 22 deletions
Download Patch File Download Diff File Expand all files Collapse all files

27
CMakeLists.txt
View File

@ -4,17 +4,6 @@ project(MassSprings)
set(CMAKE_CXX_STANDARD 26)
set(CMAKE_EXPORT_COMPILE_COMMANDS ON)
# set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wall -Wextra -Wfloat-equal -Wundef -Wshadow -Wpointer-arith -Wcast-align -Wno-unused-parameter -Wunreachable-code")
set(CMAKE_CXX_FLAGS_DEBUG "${CMAKE_CXX_FLAGS_DEBUG} -O2 -ggdb") # -fsanitize=address already fails on InitWindow(), -fsanitize=undefined, -fsanitize=leak
set(CMAKE_CXX_FLAGS_RELEASE "${CMAKE_CXX_FLAGS_RELEASE} -Ofast -march=native")
message("-- CMAKE_C_FLAGS: ${CMAKE_C_FLAGS}")
message("-- CMAKE_C_FLAGS_DEBUG: ${CMAKE_C_FLAGS_DEBUG}")
message("-- CMAKE_C_FLAGS_RELEASE: ${CMAKE_C_FLAGS_RELEASE}")
message("-- CMAKE_CXX_FLAGS: ${CMAKE_CXX_FLAGS}")
message("-- CMAKE_CXX_FLAGS_DEBUG: ${CMAKE_CXX_FLAGS_DEBUG}")
message("-- CMAKE_CXX_FLAGS_RELEASE: ${CMAKE_CXX_FLAGS_RELEASE}")
# Libraries
find_package(raylib REQUIRED)
set(LIBS raylib)
@ -43,6 +32,22 @@ if(NOT DEFINED DISABLE_TRACY)
list(APPEND FLAGS TRACY)
endif()
if (WIN32)
list(APPEND LIBS opengl32 gdi32 winmm)
endif()
# Set this after fetching tracy to hide tracy's warnings
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wall -Wextra -Wfloat-equal -Wundef -Wshadow -Wpointer-arith -Wcast-align -Wno-unused-parameter -Wunreachable-code")
set(CMAKE_CXX_FLAGS_DEBUG "${CMAKE_CXX_FLAGS_DEBUG} -O2 -ggdb") # -fsanitize=address already fails on InitWindow(), -fsanitize=undefined, -fsanitize=leak
set(CMAKE_CXX_FLAGS_RELEASE "${CMAKE_CXX_FLAGS_RELEASE} -Ofast")
message("-- CMAKE_C_FLAGS: ${CMAKE_C_FLAGS}")
message("-- CMAKE_C_FLAGS_DEBUG: ${CMAKE_C_FLAGS_DEBUG}")
message("-- CMAKE_C_FLAGS_RELEASE: ${CMAKE_C_FLAGS_RELEASE}")
message("-- CMAKE_CXX_FLAGS: ${CMAKE_CXX_FLAGS}")
message("-- CMAKE_CXX_FLAGS_DEBUG: ${CMAKE_CXX_FLAGS_DEBUG}")
message("-- CMAKE_CXX_FLAGS_RELEASE: ${CMAKE_CXX_FLAGS_RELEASE}")
# Headers + Sources
include_directories(include)
set(SOURCES

4
include/state.hpp
View File

@ -47,9 +47,9 @@ public:
bool edited = false;
public:
StateManager(ThreadedPhysics &_physics, const std::string &preset_file)
StateManager(ThreadedPhysics &_physics, const std::string &_preset_file)
: physics(_physics) {
ParsePresetFile(preset_file);
ParsePresetFile(_preset_file);
current_state = presets.at(current_preset);
ClearGraph();
}

18
src/gui.cpp
View File

@ -253,13 +253,13 @@ auto Gui::DrawButton(Rectangle bounds, const std::string &label, Color color,
SetDefaultStyle(style_default);
SetComponentStyle(BUTTON, style_button);
const int state = GuiGetState();
const int _state = GuiGetState();
if (!enabled || WindowOpen()) {
GuiSetState(STATE_DISABLED);
}
int pressed = GuiButton(bounds, label.data());
if (!enabled || WindowOpen()) {
GuiSetState(state);
GuiSetState(_state);
}
// Restore original styling
@ -296,14 +296,14 @@ auto Gui::DrawToggleSlider(Rectangle bounds, const std::string &off_label,
SetComponentStyle(SLIDER, style_slider);
SetComponentStyle(TOGGLE, style_toggle);
const int state = GuiGetState();
const int _state = GuiGetState();
if (!enabled || WindowOpen()) {
GuiSetState(STATE_DISABLED);
}
int pressed = GuiToggleSlider(
bounds, std::format("{};{}", off_label, on_label).data(), active);
if (!enabled || WindowOpen()) {
GuiSetState(state);
GuiSetState(_state);
}
// Restore original styling
@ -343,13 +343,13 @@ auto Gui::DrawSpinner(Rectangle bounds, const std::string &label, int *value,
SetComponentStyle(VALUEBOX, style_valuebox);
SetComponentStyle(BUTTON, style_button);
const int state = GuiGetState();
const int _state = GuiGetState();
if (!enabled || WindowOpen()) {
GuiSetState(STATE_DISABLED);
}
int pressed = GuiSpinner(bounds, "", label.data(), value, min, max, false);
if (!enabled || WindowOpen()) {
GuiSetState(state);
GuiSetState(_state);
}
// Restore original styling
@ -382,13 +382,13 @@ auto Gui::DrawLabel(Rectangle bounds, const std::string &text, Color color,
SetDefaultStyle(style_default);
SetComponentStyle(LABEL, style_label);
const int state = GuiGetState();
const int _state = GuiGetState();
if (!enabled || WindowOpen()) {
GuiSetState(STATE_DISABLED);
}
int pressed = GuiLabel(bounds, text.data());
if (!enabled || WindowOpen()) {
GuiSetState(state);
GuiSetState(_state);
}
// Restore original styling
@ -516,7 +516,7 @@ auto Gui::DrawCameraControls(Color color) const -> void {
int projection = camera.projection == CAMERA_ORTHOGRAPHIC;
DrawMenuToggleSlider(0, 3, 1, 1, "Perspective (Alt)", "Orthographic (Alt)",
&projection, color);
if (projection != camera.projection == CAMERA_ORTHOGRAPHIC) {
if (projection != (camera.projection == CAMERA_ORTHOGRAPHIC)) {
input.ToggleCameraProjection();
}
}