fix compare issues

diff --git a/CMakeLists.txt b/CMakeLists.txt
index 6b8c41ad7bf9e631515283581e7304810bcf7989..1c39f309f008a1d21f26836b495920fabc6717cf 100644 (file)
--- 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 587cecb61f09f7e4885887744bb98cd1fe2fb300..c520e31fc619e75bd709e3703f9f1683aa26b7c1 100644 (file)
--- 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 1d29a64994d906dd7655ca46b53a2ca6a64416e6..522d77a262b14b4f14f62ce0dbb4dcc71aaf9a33 100644 (file)
--- 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) {