From 4130665b2c8004ea3f33ae6e6c036805f67a0dae Mon Sep 17 00:00:00 2001 From: Bastian Dehn Date: Fri, 10 Oct 2025 09:36:25 +0200 Subject: [PATCH] fix compare issues --- CMakeLists.txt | 4 ++-- libs/spandisc/src/splitter.c | 4 ++-- src/discspan.c | 2 +- 3 files changed, 5 insertions(+), 5 deletions(-) diff --git a/CMakeLists.txt b/CMakeLists.txt index 6b8c41a..1c39f30 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -4,9 +4,9 @@ set(CMAKE_BUILD_TYPE "Debug" CACHE STRING "Build Type [Debug|Release]") message(STATUS "CMAKE_BUILD_TYPE: ${CMAKE_BUILD_TYPE}") if("${CMAKE_BUILD_TYPE}" STREQUAL "Debug") - set(CMAKE_C_FLAGS "-Wall -std=c99 -g -fsanitize=address -fprofile-arcs -ftest-coverage") + set(CMAKE_C_FLAGS "-std=c99 -Wall -Wextra -pedantic -g -fsanitize=address -fprofile-arcs -ftest-coverage") else() - set(CMAKE_C_FLAGS "-Werror -std=c99") + set(CMAKE_C_FLAGS "-std=c99 -Werror -Wextra -pedantic") endif() project(discspan) diff --git a/libs/spandisc/src/splitter.c b/libs/spandisc/src/splitter.c index 587cecb..c520e31 100644 --- a/libs/spandisc/src/splitter.c +++ b/libs/spandisc/src/splitter.c @@ -48,7 +48,7 @@ void free_medium(medium_t *medium) void free_medium_complete(medium_t* medium) { - for (int i = 0; i < medium->length; i++) { + for (uint32_t i = 0; i < medium->length; i++) { free_file_size(medium->filearray[i]); medium->filearray[i] = NULL; } @@ -162,7 +162,7 @@ medium_t* splitter_to_disc(medium_t* srcdata, disc->nr = srcdata->nr; disc->freespace = srcdata->freespace; - for (int i = 0; i < srcdata->length; i++) { + for (uint32_t i = 0; i < srcdata->length; i++) { if (check(disc, srcdata->filearray[i])) add_item_medium(disc, srcdata->filearray[i]); else diff --git a/src/discspan.c b/src/discspan.c index 1d29a64..522d77a 100644 --- a/src/discspan.c +++ b/src/discspan.c @@ -72,7 +72,7 @@ int main(int argc, char* argv[]) int32_t opt_index = 0; int32_t opt_value = 0; - for (uint32_t i = 0; i < argc; i++) { + for (int32_t i = 0; i < argc; i++) { opt_index = 0; opt_value = getopt_long(argc, argv, "i:o:59bcfhv", loption, &opt_index); switch (opt_value) { -- 2.47.3