Reduce compiler warnings for -Wall -Wpedantic -Wextra

Author: MaJerle

CMakeLists.txt

@@ -23,30 +23,28 @@ target_sources(${PROJECT_NAME} PUBLIC
 
     # win32 port
     ${CMAKE_CURRENT_LIST_DIR}/lwmem/src/system/lwmem_sys_win32.c
-    )
+)
 
 # Add key include paths
 target_include_directories(${PROJECT_NAME} PUBLIC
     ${CMAKE_CURRENT_LIST_DIR}
     ${CMAKE_CURRENT_LIST_DIR}/dev
-    )
+)
 
 # Compilation definition information
 target_compile_definitions(${PROJECT_NAME} PUBLIC
     WIN32
     _DEBUG
     CONSOLE
     LWMEM_DEV
-    )
+)
 
-# Compiler and linker options
+# Compiler options
 target_compile_options(${PROJECT_NAME} PRIVATE
-    ${CPU_PARAMETERS}
     -Wall
     -Wextra
     -Wpedantic
-    -Wno-unused-parameter
-    )
+)
 
 # Add subdir with lwmem and link to the project
 add_subdirectory("lwmem")

lwmem/src/lwmem/lwmem.c

@@ -245,7 +245,12 @@ prv_insert_free_block(lwmem_t* const lw, lwmem_block_t* nb) {
      * By doing this, we protect data left by app
      * and we make sure new allocations cannot see old information
      */
-    LWMEM_MEMSET(LWMEM_GET_PTR_FROM_BLOCK(nb), 0x00, nb->size - LWMEM_BLOCK_META_SIZE);
+    if (nb != NULL) {
+        void* p = LWMEM_GET_PTR_FROM_BLOCK(nb);
+        if (p != NULL) {
+            LWMEM_MEMSET(p, 0x00, nb->size - LWMEM_BLOCK_META_SIZE);
+        }
+    }
 #endif /* LWMEM_CFG_RESET_MEMORY */
 
     /*
@@ -1102,6 +1107,9 @@ lwmem_debug_print(uint8_t print_alloc, uint8_t print_free) {
     size_t block_size;
     lwmem_block_t* block;
 
+    (void)print_alloc;
+    (void)print_free;
+
     printf("|-------|------------------|--------|------|------------------|-----------------|\r\n");
     printf("| Block |          Address | IsFree | Size | MaxUserAllocSize | Meta            |\r\n");
     printf("|-------|------------------|--------|------|------------------|-----------------|\r\n");

tests/lwmem_test.c

@@ -54,8 +54,8 @@ lw_c_regions[] = {
 
 void
 lwmem_test_run(void) {
-    void* ptr_1, * ptr_2, * ptr_3, * ptr_4;
-    void* ptr_c_1, * ptr_c_2, * ptr_c_3, * ptr_c_4;
+    void* ptr_1, * ptr_2, * ptr_3;
+    void* ptr_c_1, * ptr_c_2, * ptr_c_3;
 
     /* Initialize default lwmem instance */
     /* Use one of 2 possible function calls: */