Increase coverage and CI fixes

Author: SamuelMarks

c89stringutils/c89stringutils_string_extras.c

@@ -20,7 +20,6 @@
 #include <errno.h>
 #include <ctype.h>
 #include <limits.h> /* for INT_MAX */
-/* clang-format on */
 #ifndef RSIZE_MAX
 #define RSIZE_MAX ((size_t) - 1 >> 1)
 #endif
@@ -255,6 +254,7 @@ C89STRINGUTILS_EXPORT void c89stringutils_log_debug(const char *fmt, ...) {
     disable : 4244 4702) /* conversion from int to char, unreachable code */
 #endif
 #include "stb_sprintf.h"
+/* clang-format on */
 #if defined(_MSC_VER) && _MSC_VER >= 1300
 #pragma warning(pop)
 #endif

c89stringutils/c89stringutils_string_extras.h

@@ -496,16 +496,16 @@ c89stringutils_vsnprintf(char *s, size_t n, const char *format, va_list arg)
  */
 #if defined(C89STRINGUTILS_HAVE__STRNICMP)
 #define c89stringutils_strncasecmp(s1, s2, n)                                  \
-  (((s1) == NULL && (s2) == NULL) ? 0                                          \
-   : ((s1) == NULL)               ? -1                                         \
-   : ((s2) == NULL)               ? 1                                          \
-                                  : _strnicmp((s1), (s2), (n)))
+  ((((const void *)(s1)) == NULL && ((const void *)(s2)) == NULL) ? 0          \
+   : (((const void *)(s1)) == NULL)                               ? -1         \
+   : (((const void *)(s2)) == NULL)                               ? 1          \
+                                    : _strnicmp((s1), (s2), (n)))
 #elif defined(C89STRINGUTILS_HAVE_STRNCASECMP)
 #define c89stringutils_strncasecmp(s1, s2, n)                                  \
-  (((s1) == NULL && (s2) == NULL) ? 0                                          \
-   : ((s1) == NULL)               ? -1                                         \
-   : ((s2) == NULL)               ? 1                                          \
-                                  : strncasecmp((s1), (s2), (n)))
+  ((((const void *)(s1)) == NULL && ((const void *)(s2)) == NULL) ? 0          \
+   : (((const void *)(s1)) == NULL)                               ? -1         \
+   : (((const void *)(s2)) == NULL)                               ? 1          \
+                                    : strncasecmp((s1), (s2), (n)))
 #else
 extern C89STRINGUTILS_EXPORT int
 c89stringutils_strncasecmp(const char *s1, const char *s2, size_t n);
@@ -520,16 +520,16 @@ c89stringutils_strncasecmp(const char *s1, const char *s2, size_t n);
  */
 #if defined(C89STRINGUTILS_HAVE__STRICMP)
 #define c89stringutils_strcasecmp(s1, s2)                                      \
-  (((s1) == NULL && (s2) == NULL) ? 0                                          \
-   : ((s1) == NULL)               ? -1                                         \
-   : ((s2) == NULL)               ? 1                                          \
-                                  : _stricmp((s1), (s2)))
+  ((((const void *)(s1)) == NULL && ((const void *)(s2)) == NULL) ? 0          \
+   : (((const void *)(s1)) == NULL)                               ? -1         \
+   : (((const void *)(s2)) == NULL)                               ? 1          \
+                                    : _stricmp((s1), (s2)))
 #elif defined(C89STRINGUTILS_HAVE_STRCASECMP)
 #define c89stringutils_strcasecmp(s1, s2)                                      \
-  (((s1) == NULL && (s2) == NULL) ? 0                                          \
-   : ((s1) == NULL)               ? -1                                         \
-   : ((s2) == NULL)               ? 1                                          \
-                                  : strcasecmp((s1), (s2)))
+  ((((const void *)(s1)) == NULL && ((const void *)(s2)) == NULL) ? 0          \
+   : (((const void *)(s1)) == NULL)                               ? -1         \
+   : (((const void *)(s2)) == NULL)                               ? 1          \
+                                    : strcasecmp((s1), (s2)))
 #else
 extern C89STRINGUTILS_EXPORT int c89stringutils_strcasecmp(const char *s1,
                                                            const char *s2);
@@ -546,7 +546,9 @@ extern C89STRINGUTILS_EXPORT int c89stringutils_strcasecmp(const char *s1,
  */
 #if defined(C89STRINGUTILS_HAVE_STRNSTR)
 #define c89stringutils_strnstr(b, t, l)                                        \
-  (((b) == NULL || (t) == NULL) ? NULL : strnstr((b), (t), (l)))
+  ((((const void *)(b)) == NULL || ((const void *)(t)) == NULL)                \
+       ? NULL                                                                  \
+       : strnstr((b), (t), (l)))
 #else
 extern C89STRINGUTILS_EXPORT char *c89stringutils_strnstr(const char *buffer,
                                                           const char *target,
@@ -562,7 +564,9 @@ extern C89STRINGUTILS_EXPORT char *c89stringutils_strnstr(const char *buffer,
  */
 #if defined(C89STRINGUTILS_HAVE_STRCASESTR)
 #define c89stringutils_strcasestr(h, n)                                        \
-  (((h) == NULL || (n) == NULL) ? NULL : strcasestr((h), (n)))
+  ((((const void *)(h)) == NULL || ((const void *)(n)) == NULL)                \
+       ? NULL                                                                  \
+       : strcasestr((h), (n)))
 #else
 extern C89STRINGUTILS_EXPORT char *c89stringutils_strcasestr(const char *h,
                                                              const char *n);

c89stringutils/stb_sprintf.h

@@ -1,3 +1,6 @@
+#ifdef __cplusplus
+extern "C" {
+#endif /* __cplusplus */
 /* clang-format off */
 #if defined(__GNUC__) || defined(__clang__)
 #pragma GCC diagnostic push
@@ -1920,3 +1923,7 @@ WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
 #pragma GCC diagnostic pop
 #endif
 /* clang-format on */
+
+#ifdef __cplusplus
+}
+#endif /* __cplusplus */