diff options
author | Yuval Peress <peress@chromium.org> | 2021-06-17 13:20:25 -0600 |
---|---|---|
committer | Commit Bot <commit-bot@chromium.org> | 2021-06-18 19:05:01 +0000 |
commit | 257acda4b812d01687710ac2565e60e8df203eb2 (patch) | |
tree | b4d8daca0a7d8a8f85b4d0390ea5793cfcb60c42 /test | |
parent | f1e0e13b2b71bf8f97b6da4e3eab27fd17000292 (diff) | |
download | chrome-ec-257acda4b812d01687710ac2565e60e8df203eb2.tar.gz |
zephyr: update snprintf signature to be compatible with zephyr 2.6
Also update the tests for printf since the documentation makes no
guarantee about negative size values.
BRANCH=none
BUG=b:190731415
TEST=build brya
TEST=make run-printf
Signed-off-by: Yuval Peress <peress@chromium.org>
Change-Id: I5e773362f1f30a1beb95284e589e49db3a1d8800
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2970989
Commit-Queue: Denis Brockus <dbrockus@chromium.org>
Reviewed-by: Denis Brockus <dbrockus@chromium.org>
Diffstat (limited to 'test')
-rw-r--r-- | test/printf.c | 10 |
1 files changed, 3 insertions, 7 deletions
diff --git a/test/printf.c b/test/printf.c index ae85837cd5..f7e9b9dd2d 100644 --- a/test/printf.c +++ b/test/printf.c @@ -97,10 +97,6 @@ test_static int test_vsnprintf_args(void) T(expect(/* expect an invalid args error */ EC_ERROR_INVAL, NO_BYTES_TOUCHED, - /* given -1 as output size limit */ - false, -1, "")); - T(expect(/* expect an invalid args error */ - EC_ERROR_INVAL, NO_BYTES_TOUCHED, /* given 0 as output size limit */ false, 0, "")); T(expect(/* expect SUCCESS */ @@ -209,13 +205,13 @@ test_static int test_vsnprintf_pointers(void) T(expect_success(err_str, "%P", ptr)); /* %p by itself is invalid */ T(expect(EC_ERROR_INVAL, NO_BYTES_TOUCHED, - false, -1, "%p")); + false, 0, "%p")); /* %p with an unknown suffix is invalid */ T(expect(EC_ERROR_INVAL, NO_BYTES_TOUCHED, - false, -1, "%p ")); + false, 0, "%p ")); /* %p with an unknown suffix is invalid */ T(expect(EC_ERROR_INVAL, NO_BYTES_TOUCHED, - false, -1, "%pQ")); + false, 0, "%pQ")); /* Test %pb, binary format */ T(expect_success("0", "%pb", BINARY_VALUE(val, 0))); |