Message ID | 20220119014315.1938157-3-sjg@chromium.org |
---|---|
State | RFC |
Delegated to: | Tom Rini |
Headers | show |
Series | Initial implementation of standard boot | expand |
On 1/19/22 02:42, Simon Glass wrote: > This function currently has no tests. Add some. > > Signed-off-by: Simon Glass <sjg@chromium.org> > --- > > (no changes since v1) > > include/vsprintf.h | 4 ++-- > test/str_ut.c | 19 +++++++++++++++++++ > 2 files changed, 21 insertions(+), 2 deletions(-) > > diff --git a/include/vsprintf.h b/include/vsprintf.h > index 8bfafa0d33f..01d2248e04d 100644 > --- a/include/vsprintf.h > +++ b/include/vsprintf.h > @@ -99,7 +99,7 @@ long long simple_strtoll(const char *cp, char **endp, unsigned int base); > * For example, "abc123" would return 123. > * > * @str: String to exxamine > - * @return training number if found, else -1 > + * @return trailing number if found, else -1 %s/@return/Return:/ Cf. https://www.kernel.org/doc/html/latest/doc-guide/kernel-doc.html#function-documentation Best regards Heinrich > */ > long trailing_strtol(const char *str); > > @@ -114,7 +114,7 @@ long trailing_strtol(const char *str); > * @str: String to exxamine > * @end: Pointer to end of string to examine, or NULL to use the > * whole string > - * @return training number if found, else -1 > + * @return trailing number if found, else -1 > */ > long trailing_strtoln(const char *str, const char *end); > > diff --git a/test/str_ut.c b/test/str_ut.c > index 6c817f6f72f..9674a59f2a6 100644 > --- a/test/str_ut.c > +++ b/test/str_ut.c > @@ -242,6 +242,25 @@ static int str_xtoa(struct unit_test_state *uts) > } > STR_TEST(str_xtoa, 0); > > +static int str_trailing(struct unit_test_state *uts) > +{ > + char str1[] = "abc123def"; > + > + ut_asserteq(-1, trailing_strtol("")); > + ut_asserteq(-1, trailing_strtol("123")); > + ut_asserteq(123, trailing_strtol("abc123")); > + ut_asserteq(4, trailing_strtol("12c4")); > + ut_asserteq(-1, trailing_strtol("abd")); > + ut_asserteq(-1, trailing_strtol("abc123def")); > + > + ut_asserteq(-1, trailing_strtoln(str1, NULL)); > + ut_asserteq(123, trailing_strtoln(str1, str1 + 6)); > + ut_asserteq(-1, trailing_strtoln(str1, str1 + 9)); > + > + return 0; > +} > +STR_TEST(str_trailing, 0); > + > int do_ut_str(struct cmd_tbl *cmdtp, int flag, int argc, char *const argv[]) > { > struct unit_test *tests = UNIT_TEST_SUITE_START(str_test);
diff --git a/include/vsprintf.h b/include/vsprintf.h index 8bfafa0d33f..01d2248e04d 100644 --- a/include/vsprintf.h +++ b/include/vsprintf.h @@ -99,7 +99,7 @@ long long simple_strtoll(const char *cp, char **endp, unsigned int base); * For example, "abc123" would return 123. * * @str: String to exxamine - * @return training number if found, else -1 + * @return trailing number if found, else -1 */ long trailing_strtol(const char *str); @@ -114,7 +114,7 @@ long trailing_strtol(const char *str); * @str: String to exxamine * @end: Pointer to end of string to examine, or NULL to use the * whole string - * @return training number if found, else -1 + * @return trailing number if found, else -1 */ long trailing_strtoln(const char *str, const char *end); diff --git a/test/str_ut.c b/test/str_ut.c index 6c817f6f72f..9674a59f2a6 100644 --- a/test/str_ut.c +++ b/test/str_ut.c @@ -242,6 +242,25 @@ static int str_xtoa(struct unit_test_state *uts) } STR_TEST(str_xtoa, 0); +static int str_trailing(struct unit_test_state *uts) +{ + char str1[] = "abc123def"; + + ut_asserteq(-1, trailing_strtol("")); + ut_asserteq(-1, trailing_strtol("123")); + ut_asserteq(123, trailing_strtol("abc123")); + ut_asserteq(4, trailing_strtol("12c4")); + ut_asserteq(-1, trailing_strtol("abd")); + ut_asserteq(-1, trailing_strtol("abc123def")); + + ut_asserteq(-1, trailing_strtoln(str1, NULL)); + ut_asserteq(123, trailing_strtoln(str1, str1 + 6)); + ut_asserteq(-1, trailing_strtoln(str1, str1 + 9)); + + return 0; +} +STR_TEST(str_trailing, 0); + int do_ut_str(struct cmd_tbl *cmdtp, int flag, int argc, char *const argv[]) { struct unit_test *tests = UNIT_TEST_SUITE_START(str_test);
This function currently has no tests. Add some. Signed-off-by: Simon Glass <sjg@chromium.org> --- (no changes since v1) include/vsprintf.h | 4 ++-- test/str_ut.c | 19 +++++++++++++++++++ 2 files changed, 21 insertions(+), 2 deletions(-)