Skip to content

Commit

Permalink
all: Fixup some new warnings/tidy checks
Browse files Browse the repository at this point in the history
  • Loading branch information
@goldsteinn
goldsteinn committed Sep 18, 2024
1 parent c3427db commit 5a3e412
Show file tree
Hide file tree
Showing 13 changed files with 125 additions and 67 deletions.
3 changes: 2 additions & 1 deletion CMakeLists.txt
View file
Original file line number Diff line number Diff line change
Expand Up @@ -92,7 +92,8 @@ if(CMAKE_PROJECT_NAME STREQUAL PROJECT_NAME)
--src=${PROJECT_SOURCE_DIR}/src/eztest/eztest.h
--prefix=${PROJECT_BINARY_DIR}/${EZTEST_INTERNAL_PREFIX_} --dst=eztest.h
--trim-pragmas
DEPENDS ${PROJECT_SOURCE_DIR}/src/eztest/eztest-api.h
DEPENDS ${PROJECT_SOURCE_DIR}/src/eztest/eztest-algo.h
${PROJECT_SOURCE_DIR}/src/eztest/eztest-api.h
${PROJECT_SOURCE_DIR}/src/eztest/eztest-arch.h
${PROJECT_SOURCE_DIR}/src/eztest/eztest-asserts-impl.h
${PROJECT_SOURCE_DIR}/src/eztest/eztest-c-asserts-impl.h
Expand Down
17 changes: 17 additions & 0 deletions src/eztest/eztest-algo.h
View file
Original file line number Diff line number Diff line change
@@ -0,0 +1,17 @@
#ifndef EZTEST_D_EZTEST_D_EZTEST_ALGO_H_
#define EZTEST_D_EZTEST_D_EZTEST_ALGO_H_

#include "eztest-lang.h"

#if EZTEST_CXX_LANG_
/* NOLINTBEGIN(llvmlibc-restrict-system-libc-headers) */
# include <algorithm>
/* NOLINTEND(llvmlibc-restrict-system-libc-headers) */
# define EZTEST_MIN_(eztest_lhs, eztest_rhs) \
EZTEST_STD_NS_ min(eztest_lhs, eztest_rhs)
#else
# define EZTEST_MIN_(eztest_lhs, eztest_rhs) \
((eztest_lhs) < (eztest_rhs) ? (eztest_lhs) : (eztest_rhs))
#endif

#endif
11 changes: 6 additions & 5 deletions src/eztest/eztest-containers.h
View file
Original file line number Diff line number Diff line change
Expand Up @@ -6,16 +6,17 @@


#if EZTEST_CXX_LANG_ >= 2011
/* NOLINTBEGIN(llvmlibc-restrict-system-libc-headers) */
# include <array>
/* NOLINTEND(llvmlibc-restrict-system-libc-headers) */
# define EZTEST_ARR_BUILDER_(name, T, k_n) EZTEST_STD_NS_ array<T, k_n> name
# define EZTEST_ARR_T_INIT_ \
{ \
{ 0 } \
}
{ \
{ 0 } \
}
#else
# define EZTEST_ARR_BUILDER_(name, T, k_n) T name[k_n]
# define EZTEST_ARR_T_INIT_ \
{ 0 }
# define EZTEST_ARR_T_INIT_ { 0 }
#endif

#endif
4 changes: 2 additions & 2 deletions src/eztest/eztest-cxx-asserts-impl.h
View file
Original file line number Diff line number Diff line change
Expand Up @@ -344,7 +344,7 @@ class eztest_cxx_binop_streq_t : public EZTEST_NS_ eztest_cxx_print_var_t {
eztest_cxx_run(const EZTEST_STD_NS_ string & eztest_op0,
const EZTEST_STD_NS_ string_view & eztest_op1) {
EZTEST_DISABLE_WAGGREGATE_RETURN_
EZTEST_STD_NS_ string_view eztest_op0_sv(eztest_op0);
const EZTEST_STD_NS_ string_view eztest_op0_sv(eztest_op0);
EZTEST_REENABLE_WAGGREGATE_RETURN_
return EZTEST_STD_NS_ operator==(eztest_op0_sv, eztest_op1);
}
Expand All @@ -353,7 +353,7 @@ class eztest_cxx_binop_streq_t : public EZTEST_NS_ eztest_cxx_print_var_t {
eztest_cxx_run(const EZTEST_STD_NS_ string_view & eztest_op0,
const EZTEST_STD_NS_ string & eztest_op1) {
EZTEST_DISABLE_WAGGREGATE_RETURN_
EZTEST_STD_NS_ string_view eztest_op1_sv(eztest_op1);
const EZTEST_STD_NS_ string_view eztest_op1_sv(eztest_op1);
EZTEST_REENABLE_WAGGREGATE_RETURN_
return EZTEST_STD_NS_ operator==(eztest_op0, eztest_op1_sv);
}
Expand Down
48 changes: 24 additions & 24 deletions src/eztest/eztest-lang.h
View file
Original file line number Diff line number Diff line change
Expand Up @@ -10,20 +10,20 @@
# define EZTEST_STRUCT_

# define EZTEST_NAMESPACE_BEGIN_ \
EZTEST_DISABLE_WNAMESPACES_ \
namespace eztest {
EZTEST_DISABLE_WNAMESPACES_ \
namespace eztest {

# define EZTEST_NAMESPACE_END_ \
} \
EZTEST_REENABLE_WNAMESPACES_
} \
EZTEST_REENABLE_WNAMESPACES_

# define EZTEST_NS_ \
/* NOLINTBEGIN(llvmlibc-callee-namespace) */ \
eztest::/* NOLINTEND(llvmlibc-callee-namespace) */
/* NOLINTBEGIN(llvmlibc-callee-namespace) */ \
eztest::/* NOLINTEND(llvmlibc-callee-namespace) */

# define EZTEST_STD_NS_ \
/* NOLINTBEGIN(llvmlibc-callee-namespace) */ \
std::/* NOLINTEND(llvmlibc-callee-namespace) */
/* NOLINTBEGIN(llvmlibc-callee-namespace) */ \
std::/* NOLINTEND(llvmlibc-callee-namespace) */

# define EZTEST_VOID_ARG_

Expand All @@ -32,34 +32,34 @@
# endif

# define EZTEST_PRIVATE_CXX_INL_ /* NOLINTBEGIN(llvmlibc-inl*-func*-decl) */ \
static EZTEST_INLINE_ /* NOLINTEND(llvmlibc-inl*-func*-decl) */
static EZTEST_INLINE_ /* NOLINTEND(llvmlibc-inl*-func*-decl) */


# define EZTEST_PRIVATE_ \
/* NOLINTBEGIN(llvmlibc-inl*-func*-decl) */ \
static /* NOLINTEND(llvmlibc-inl*-func*-decl) */
/* NOLINTBEGIN(llvmlibc-inl*-func*-decl) */ \
static /* NOLINTEND(llvmlibc-inl*-func*-decl) */


# if EZTEST_CXX_LANG_ >= 2011
# define EZTEST_NULL_ \
EZTEST_DISABLE_WCXX98_COMPAT_ nullptr EZTEST_REENABLE_WCXX98_COMPAT_
EZTEST_DISABLE_WCXX98_COMPAT_ nullptr EZTEST_REENABLE_WCXX98_COMPAT_
# else
# include <cstddef> /* NULL. */
# define EZTEST_NULL_ NULL
# endif

# define EZTEST_CAST_(type, expr) static_cast<type>(expr)
# define EZTEST_REINTERPRET_CAST_(type, var) \
/* NOLINTBEGIN(cppcoreguidelines-pro-type-reinterpret-cast) */ \
reinterpret_cast<type>( \
var) /* NOLINTEND(cppcoreguidelines-pro-type-reinterpret-cast) */
/* NOLINTBEGIN(cppcoreguidelines-pro-type-reinterpret-cast) */ \
reinterpret_cast<type>( \
var) /* NOLINTEND(cppcoreguidelines-pro-type-reinterpret-cast) */

# if EZTEST_CXX_LANG_ >= 2011
# define EZTEST_STATIC_ASSERT_MSG_(cond, msg) \
EZTEST_DISABLE_WCXX98_COMPAT_ static_assert(cond, msg) \
EZTEST_REENABLE_WCXX98_COMPAT_
EZTEST_DISABLE_WCXX98_COMPAT_ static_assert(cond, msg) \
EZTEST_REENABLE_WCXX98_COMPAT_
# define EZTEST_AUTO_(eztest_unused_A0) \
EZTEST_DISABLE_WCXX98_COMPAT_ auto EZTEST_REENABLE_WCXX98_COMPAT_
EZTEST_DISABLE_WCXX98_COMPAT_ auto EZTEST_REENABLE_WCXX98_COMPAT_
# endif


Expand All @@ -79,7 +79,7 @@
# endif

# define EZTEST_PRIVATE_ /* NOLINTBEGIN(llvmlibc-inline-function-decl) */ \
static /* NOLINTEND(llvmlibc-inline-function-decl) */
static /* NOLINTEND(llvmlibc-inline-function-decl) */
# define EZTEST_PRIVATE_CXX_INL_ EZTEST_PRIVATE_


Expand All @@ -102,9 +102,9 @@

#ifndef EZTEST_STATIC_ASSERT_MSG_
# define EZTEST_STATIC_ASSERT_MSG_(cond, msg) \
struct EZTEST_UNIQUE_NAME_(eztest_static_assertion) { \
char EZTEST_UNIQUE_NAME_(eztest_x_)[(cond) ? 1 : -1]; \
}
struct EZTEST_UNIQUE_NAME_(eztest_static_assertion) { \
char EZTEST_UNIQUE_NAME_(eztest_x_)[(cond) ? 1 : -1]; \
}

#endif

Expand All @@ -116,9 +116,9 @@
#define EZTEST_STRUCT_NS_ EZTEST_STRUCT_ EZTEST_NS_

#define EZTEST_UNIMPLEMENTED_(name) \
EZTEST_STATIC_ASSERT_MSG_(0, name " is unimplemented")
EZTEST_STATIC_ASSERT_MSG_(0, name " is unimplemented")
#define EZTEST_STATIC_ASSERT_(cond) \
EZTEST_STATIC_ASSERT_MSG_(cond, EZTEST_STRINGIFY_(cond))
EZTEST_STATIC_ASSERT_MSG_(cond, EZTEST_STRINGIFY_(cond))


#endif
49 changes: 26 additions & 23 deletions src/eztest/eztest-libc.h
View file
Original file line number Diff line number Diff line change
Expand Up @@ -6,27 +6,30 @@
#include "eztest-lang.h"
#include "eztest-libc-incs.h"


#define EZTEST_EXIT_ EZTEST_STD_NS_ exit
/* TODO: Wrap the call to exit in a mutex. */
#define EZTEST_EXIT_ /* NOLINTBEGIN(concurrency-mt-unsafe) */ \
EZTEST_STD_NS_ exit /* NOLINTEND(concurrency-mt-unsafe) */
EZTEST_DISABLE_WCXX98_COMPAT_PEDANTIC_

#define EZTEST_PRINTF_(...) \
/* NOLINTBEGIN(clang-ana*-sec*.insecureAPI.Depr*OrUnsafeBufferHandling) */ \
/* NOLINTBEGIN(cppcoreguide*-pro-type-vararg,hicpp-vararg) */ \
EZTEST_STD_NS_ fprintf( \
EZTEST_STDOUT_, \
__VA_ARGS__) /* NOLINTEND(cppcoreguide*-pro-type-vararg,hicpp-vararg) */ \
/* NOLINTBEGIN(clang-ana*-sec*.insecureAPI.Depr*OrUnsafeBufferHandling) */ \
/* NOLINTBEGIN(cppcoreguide*-pro-type-vararg,hicpp-vararg) */ \
EZTEST_STD_NS_ fprintf( \
EZTEST_STDOUT_, \
__VA_ARGS__) /* NOLINTEND(cppcoreguide*-pro-type-vararg,hicpp-vararg) \
*/ \
\
/* NOLINTEND(clang-ana*-sec*.insecureAPI.Depr*OrUnsafeBufferHandling) */
/* NOLINTEND(clang-ana*-sec*.insecureAPI.Depr*OrUnsafeBufferHandling) \
*/

EZTEST_REENABLE_WCXX98_COMPAT_PEDANTIC_

#define EZTEST_VFPRINTF_ EZTEST_STD_NS_ vfprintf
#define EZTEST_FFLUSH_ EZTEST_STD_NS_ fflush
#define EZTEST_ERRNO_ errno
#define EZTEST_STDOUT_ \
EZTEST_DISABLE_WDISABLED_MACRO_EXPANSION_ stdout \
EZTEST_REENABLE_WDISABLED_MACRO_EXPANSION_
EZTEST_DISABLE_WDISABLED_MACRO_EXPANSION_ stdout \
EZTEST_REENABLE_WDISABLED_MACRO_EXPANSION_

/* strerror is safe here. We have complete control of the threads
and know in the parent proc there will only be 1 thread (the
Expand All @@ -37,17 +40,17 @@ EZTEST_REENABLE_WCXX98_COMPAT_PEDANTIC_
# define EZTEST_STRERROR_R_(errnum, buf, bufsz) strerror_r(errnum, buf, bufsz)
# else
# define EZTEST_STRERROR_R_(errnum, buf, bufsz) \
(strerror_r(errnum, buf, bufsz) == 0 ? &((buf)[0]) : "Unknown error")
(strerror_r(errnum, buf, bufsz) == 0 ? &((buf)[0]) : "Unknown error")
# endif
#else
# if EZTEST_CXX_LANG_
# define EZTEST_STRERROR_R_(errnum, buf, bufsz) \
/* NOLINTBEGIN(concurrency-mt-unsafe) */ \
std::strerror(errnum) /* NOLINTEND(concurrency-mt-unsafe) */
/* NOLINTBEGIN(concurrency-mt-unsafe) */ \
std::strerror(errnum) /* NOLINTEND(concurrency-mt-unsafe) */
# else
# define EZTEST_STRERROR_R_(errnum, buf, bufsz) \
/* NOLINTBEGIN(concurrency-mt-unsafe) */ \
strerror(errnum) /* NOLINTEND(concurrency-mt-unsafe) */
/* NOLINTBEGIN(concurrency-mt-unsafe) */ \
strerror(errnum) /* NOLINTEND(concurrency-mt-unsafe) */
# endif
#endif

Expand Down Expand Up @@ -85,16 +88,16 @@ EZTEST_NAMESPACE_END_

#define EZTEST_STRCMP_(lhs, rhs) EZTEST_STD_NS_ strcmp(lhs, rhs)
#define EZTEST_STARTSWITH_(prefix, str) \
(EZTEST_STD_NS_ strncmp(prefix, str, EZTEST_STD_NS_ strlen(prefix)) == 0)
(EZTEST_STD_NS_ strncmp(prefix, str, EZTEST_STD_NS_ strlen(prefix)) == 0)

#define EZTEST_MEMCPY_(dst, src, sz) EZTEST_STD_NS_ memcpy(dst, src, sz)
#define EZTEST_MEMSET_(dst, val, sz) EZTEST_STD_NS_ memset(dst, val, sz)
#define EZTEST_MEMCMP_(s0, s1, sz) EZTEST_STD_NS_ memcmp(s0, s1, sz)

#if EZTEST_POSIX_VERSION_ >= 200809L || EZTEST_GNU_SOURCE_ != 0
# define EZTEST_STRNLEN_(str, maxlen) \
/* NOLINTBEGIN(llvmlibc-callee-namespace) */ \
strnlen(str, maxlen) /* NOLINTEND(llvmlibc-callee-namespace) */
/* NOLINTBEGIN(llvmlibc-callee-namespace) */ \
strnlen(str, maxlen) /* NOLINTEND(llvmlibc-callee-namespace) */

#else
EZTEST_NAMESPACE_BEGIN_
Expand All @@ -119,11 +122,11 @@ EZTEST_NAMESPACE_END_

#if EZTEST_GNU_SOURCE_ != 0
# define EZTEST_STRCASECMP_(lhs, rhs) \
/* NOLINTBEGIN(llvmlibc-callee-namespace) */ strcasecmp( \
lhs, rhs) /* NOLINTEND(llvmlibc-callee-namespace) */
/* NOLINTBEGIN(llvmlibc-callee-namespace) */ strcasecmp( \
lhs, rhs) /* NOLINTEND(llvmlibc-callee-namespace) */
# define EZTEST_STRNCASECMP_(lhs, rhs, len) \
/* NOLINTBEGIN(llvmlibc-callee-namespace) */ strncasecmp( \
lhs, rhs, len) /* NOLINTEND(llvmlibc-callee-namespace) */
/* NOLINTBEGIN(llvmlibc-callee-namespace) */ strncasecmp( \
lhs, rhs, len) /* NOLINTEND(llvmlibc-callee-namespace) */

#else
EZTEST_NAMESPACE_BEGIN_
Expand Down Expand Up @@ -190,7 +193,7 @@ EZTEST_REENABLE_WUNUSED_FUNCTION_
EZTEST_NAMESPACE_END_
# define EZTEST_STRCASECMP_(lhs, rhs) EZTEST_NS_ eztest_strcasecmp(lhs, rhs)
# define EZTEST_STRNCASECMP_(lhs, rhs, len) \
EZTEST_NS_ eztest_strncasecmp(lhs, rhs, len)
EZTEST_NS_ eztest_strncasecmp(lhs, rhs, len)

#endif
#endif
12 changes: 11 additions & 1 deletion src/eztest/eztest-results.h
View file
Original file line number Diff line number Diff line change
Expand Up @@ -19,14 +19,15 @@ struct eztest_results_t {
EZTEST_REENABLE_WPADDED_
#define EZTEST_RESULTS_T_ EZTEST_STRUCT_NS_ eztest_results_t
#define EZTEST_RESULTS_T_INIT_ \
{ EZTEST_DURATION_T_INIT_, EZTEST_ARR_T_INIT_, 0 }
{ EZTEST_DURATION_T_INIT_, EZTEST_ARR_T_INIT_, 0 }


EZTEST_PRIVATE_ unsigned
eztest_results_count_failed(const EZTEST_RESULTS_T_ * eztest_results) {

unsigned eztest_cnt = 0;
EZTEST_DISABLE_WUNSAFE_BUFFER_USAGE_
/* NOLINTBEGIN(llvmlibc-callee-namespace) */
eztest_cnt +=
eztest_results->eztest_stats_[EZTEST_NS_ eztest_k_status_fail];
eztest_cnt +=
Expand All @@ -35,6 +36,7 @@ eztest_results_count_failed(const EZTEST_RESULTS_T_ * eztest_results) {
eztest_results->eztest_stats_[EZTEST_NS_ eztest_k_status_fail_unknown];
eztest_cnt +=
eztest_results->eztest_stats_[EZTEST_NS_ eztest_k_status_fail_timeout];
/* NOLINTEND(llvmlibc-callee-namespace) */
EZTEST_REENABLE_WUNSAFE_BUFFER_USAGE_
return eztest_cnt;
}
Expand All @@ -44,7 +46,9 @@ EZTEST_PRIVATE_ unsigned
eztest_results_count_passed(const EZTEST_RESULTS_T_ * eztest_results) {

EZTEST_DISABLE_WUNSAFE_BUFFER_USAGE_
/* NOLINTBEGIN(llvmlibc-callee-namespace) */
return eztest_results->eztest_stats_[EZTEST_NS_ eztest_k_status_passed];
/* NOLINTEND(llvmlibc-callee-namespace) */
EZTEST_REENABLE_WUNSAFE_BUFFER_USAGE_
}

Expand All @@ -53,7 +57,9 @@ EZTEST_PRIVATE_ unsigned
eztest_results_count_disabled(const EZTEST_RESULTS_T_ * eztest_results) {

EZTEST_DISABLE_WUNSAFE_BUFFER_USAGE_
/* NOLINTBEGIN(llvmlibc-callee-namespace) */
return eztest_results->eztest_stats_[EZTEST_NS_ eztest_k_status_disabled];
/* NOLINTEND(llvmlibc-callee-namespace) */
EZTEST_REENABLE_WUNSAFE_BUFFER_USAGE_
}

Expand All @@ -62,7 +68,9 @@ EZTEST_PRIVATE_ unsigned
eztest_results_count_unknown(const EZTEST_RESULTS_T_ * eztest_results) {

EZTEST_DISABLE_WUNSAFE_BUFFER_USAGE_
/* NOLINTBEGIN(llvmlibc-callee-namespace) */
return eztest_results->eztest_stats_[EZTEST_NS_ eztest_k_status_unknown];
/* NOLINTEND(llvmlibc-callee-namespace) */
EZTEST_REENABLE_WUNSAFE_BUFFER_USAGE_
}

Expand All @@ -72,12 +80,14 @@ eztest_results_count_internal_errors(const EZTEST_RESULTS_T_ * eztest_results) {

unsigned eztest_cnt = 0;
EZTEST_DISABLE_WUNSAFE_BUFFER_USAGE_
/* NOLINTBEGIN(llvmlibc-callee-namespace) */
eztest_cnt +=
eztest_results
->eztest_stats_[EZTEST_NS_ eztest_k_status_internal_error];
eztest_cnt +=
eztest_results
->eztest_stats_[EZTEST_NS_ eztest_k_status_internal_fatal_error];
/* NOLINTEND(llvmlibc-callee-namespace) */
EZTEST_REENABLE_WUNSAFE_BUFFER_USAGE_
return eztest_cnt;
}
Expand Down
11 changes: 8 additions & 3 deletions src/eztest/eztest-run.h
View file
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
#ifndef EZTEST_D_EZTEST_D_EZTEST_RUN_H_
#define EZTEST_D_EZTEST_D_EZTEST_RUN_H_

#include "eztest-algo.h"
#include "eztest-duration.h"
#include "eztest-fail.h"
#include "eztest-group-it.h"
Expand Down Expand Up @@ -183,13 +184,17 @@ eztest_run_tests(const EZTEST_LIST_T_ * eztest_tests_base,
return 1;
}
}
if (eztest_run_result > EZTEST_NS_ eztest_k_status_unknown) {
eztest_run_result = EZTEST_NS_ eztest_k_status_unknown;
}
/* Clamp return value. */
eztest_run_result = EZTEST_MIN_(
eztest_run_result,
EZTEST_CAST_(unsigned, EZTEST_NS_ eztest_k_status_unknown));

eztest_test->eztest_status_ = eztest_run_result;
EZTEST_DISABLE_WUNSAFE_BUFFER_USAGE_
/* NOLINTBEGIN(cppcoreguidelines-pro-bounds-constant-array-index) */
/* NOLINTBEGIN(llvmlibc-callee-namespace) */
++eztest_results->eztest_stats_[eztest_run_result];
/* NOLINTEND(llvmlibc-callee-namespace) */
/* NOLINTEND(cppcoreguidelines-pro-bounds-constant-array-index) */
EZTEST_REENABLE_WUNSAFE_BUFFER_USAGE_
}
Expand Down
Loading

0 comments on commit 5a3e412

Please sign in to comment.