diff options
| author | Jett Rink <jettrink@chromium.org> | 2019-08-06 12:46:41 -0600 |
|---|---|---|
| committer | Commit Bot <commit-bot@chromium.org> | 2019-08-09 21:50:53 +0000 |
| commit | e53d941195f48f0d2c5afdec44f19eafc254d4d9 (patch) | |
| tree | 69746cf7718459dba2d0aa20644b45b3d9ae25bd | |
| parent | 1d75f18c5f386d75313b71db316fcf99993fcc9f (diff) | |
| download | chrome-ec-e53d941195f48f0d2c5afdec44f19eafc254d4d9.tar.gz | |
test: make TEST_EQ and TEST_NE use %d as format
When using TEST_EQ, the checkpatch.pl script complains about
space around a ...., %d). When we are testing for equality,
we are most likely be using a integer
BRANCH=none
BUG=none
TEST=Uses of TEST_EQ work as expected when displaying errors
Change-Id: Iedc69ee8b3e98fdcb2243f9dde445425f2714818
Signed-off-by: Jett Rink <jettrink@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/1739584
Reviewed-by: Denis Brockus <dbrockus@chromium.org>
| -rw-r--r-- | include/test_util.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/include/test_util.h b/include/test_util.h index 6134897fa1..78f25ee9b3 100644 --- a/include/test_util.h +++ b/include/test_util.h @@ -50,8 +50,8 @@ } \ } while (0) -#define TEST_EQ(a, b, fmt) TEST_OPERATOR(a, b, ==, fmt) -#define TEST_NE(a, b, fmt) TEST_OPERATOR(a, b, !=, fmt) +#define TEST_EQ(a, b) TEST_OPERATOR(a, b, ==, %d) +#define TEST_NE(a, b) TEST_OPERATOR(a, b, !=, %d) #define TEST_BITS_SET(a, bits) TEST_OPERATOR(a & bits, bits, ==, %u) #define TEST_BITS_CLEARED(a, bits) TEST_OPERATOR(a & bits, 0, ==, %u) |