Fix g++ compiler warnings

Author: coderarjob

example/add_parameterised_test.c

@@ -21,7 +21,7 @@ YT_TESTP (add, add_numbers, int16_t, int16_t, int32_t)
 
 YT_TEST (add, add_must_fail)
 {
-    YT_NEQ_SCALAR (add (65536, 1), 65537);
+    YT_NEQ_SCALAR (add ((uint16_t)65534, 1), 65535);
     YT_END();
 }
 

example/add_test.c

@@ -29,7 +29,7 @@ YT_TEST (add, add_two_positive_numbers)
 
 YT_TEST (add, add_must_fail)
 {
-    YT_NEQ_SCALAR (add (65536, 1), 65537);
+    YT_NEQ_SCALAR (add ((uint16_t)65534, 1), 65535);
     YT_END();
 }
 

example/basic_tests.c

@@ -33,7 +33,7 @@ YT_TEST (basic, scaler_tests_ints)
 
 YT_TEST (basic, string_tests)
 {
-    char* string1 = "Earth";
+    const char* string1 = "Earth";
 
     YT_EQ_STRING (string1, "Earth");  // Passes because string1 == "Earth"
     YT_NEQ_STRING (string1, "Mars");  // Passes because string1 != "Mars"

example/sensor_test.c

@@ -69,7 +69,7 @@ bool is_printing_complete_handler (int id)
 
     // * `is_printing_complete.invokeCount` shows the number of times this mocked function was
     // called. Starting from 1.
-    return (is_printing_complete_fake.invokeCount > stop_after_iteration);
+    return (is_printing_complete_fake.invokeCount > (unsigned)stop_after_iteration);
 }
 
 YT_TEST (printer, printer_success)

tests/basic_failures.c

@@ -28,26 +28,26 @@ YT_TEST (basic, scaler_tests_ints)
     YT_END();
 }
 
- YT_TEST (basic, string_tests)
+YT_TEST (basic, string_tests)
 {
-    char* string1 = "Earth";
+    const char* string1 = "Earth";
 
     YT_EQ_STRING (string1, "earth");  // Fails because string1 != "earth"
-    YT_NEQ_STRING (string1, "Earth");  // Fails because string1 == "Earth"
+    YT_NEQ_STRING (string1, "Earth"); // Fails because string1 == "Earth"
 
     YT_END();
 }
 
- YT_TEST (basic, memory_tests)
+YT_TEST (basic, memory_tests)
 {
 #define ARRAY_LEN(b) (sizeof (b) / sizeof (b[0]))
 
     char buffer1[] = { 0, 1, 2, 3 };
-    //char buffer2[] = { 0, 1, 2, 3 };
+    // char buffer2[] = { 0, 1, 2, 3 };
     char buffer3[] = { 0, 1, 2, 4 };
 
-    YT_EQ_MEM (buffer1, buffer3, ARRAY_LEN (buffer1));  // Fails because buffer1 != buffer3.
-    //YT_NEQ_MEM (buffer1, buffer2, ARRAY_LEN (buffer1)); // Fails because buffer1 == buffer2.
+    YT_EQ_MEM (buffer1, buffer3, ARRAY_LEN (buffer1)); // Fails because buffer1 != buffer3.
+    // YT_NEQ_MEM (buffer1, buffer2, ARRAY_LEN (buffer1)); // Fails because buffer1 == buffer2.
 
     YT_END();
 }

tests/basic_passing.c

@@ -31,7 +31,7 @@ YT_TEST (basic, scaler_tests_ints)
 
 YT_TEST (basic, string_tests)
 {
-    char* string1 = "Earth";
+    const char* string1 = "Earth";
 
     YT_EQ_STRING (string1, "Earth");  // Passes because string1 == string "Earth"
     YT_NEQ_STRING (string1, "Mars");  // Passes because string1 != "Mars"

yukti.h

@@ -289,7 +289,16 @@ void yt_reset(); // MUST BE DEFINED BY THE USER OF THIS HEADER FILE.
     YT__DEFINE_FUNC_BODY (YT__COUNT_ARGS (__VA_ARGS__), rt, f, __VA_ARGS__)
 
 // ----
-#define YT__DEFINE_FUNC_STRUCT(f) YT__STRUCT_TAG (f) YT__STRUCT_VAR (f) = { 0 }
+
+// clang-format off
+#ifdef __cplusplus
+    #define YT__ZEROED { }
+#else
+    #define YT__ZEROED { 0 }
+#endif /* __cplusplus */
+// clang-format on
+
+#define YT__DEFINE_FUNC_STRUCT(f) YT__STRUCT_TAG (f) YT__STRUCT_VAR (f) = YT__ZEROED;
 
 #define YT__DEFINE_FUNC_BODY_VOID(n, f, ...) \
     void f (YT__FUNC_PARAMS_X (__VA_ARGS__)) \
@@ -380,7 +389,8 @@ static YT__TestRecord* YT__current_testrecord = NULL;
 static uint32_t YT__total_test_count          = 0;
 static uint32_t YT__failed_test_count         = 0;
 
-static YT__TestRecord* YT__create_testRecord (char* testname, size_t test_count, size_t test_number)
+static YT__TestRecord* YT__create_testRecord (const char* testname, size_t test_count,
+                                              size_t test_number)
 {
     assert (testname != NULL);
 
@@ -457,7 +467,7 @@ static void YT__free_testRecord (YT__TestRecord* trecord)
         #define YT__teardown()              (void)0
         #define YT__ec_init()               (void)0
     #else
-        #define YT_IN_SEQUENCE(n) for (unsigned i = 0; i < (n); i++)
+        #define YT_IN_SEQUENCE(n) for (int i = 0; i < (n); i++)
 
         #define YT_MUST_NEVER_CALL(f, ...)                                                  \
             do {                                                                            \
@@ -916,11 +926,14 @@ static int YT__equal_string (const char* a, const char* b, int* i)
 
 static int YT__equal_mem (const void* a, const void* b, unsigned long size, int* i)
 {
+    uint8_t* ta = (uint8_t*)a;
+    uint8_t* tb = (uint8_t*)b;
+
     *i = 0;
-    while (size-- && *(uint8_t*)a++ == *(uint8_t*)b++)
+    while (size-- && *ta++ == *tb++)
         (*i)++;
 
-    return *(uint8_t*)--a == *(uint8_t*)--b;
+    return *--ta == *--tb;
 }
 
     /*