[v2] tools/nolibc: add support for [v]sscanf()

[PATCH v2 0/2] tools/nolibc: add support for [v]sscanf()

Thomas Weißschuh posted 2 patches 12 months ago

Diff against v1
Download series mbox

tools/include/nolibc/stdio.h                 | 98 ++++++++++++++++++++++++++++
tools/testing/selftests/kselftest.h          |  5 --
tools/testing/selftests/nolibc/nolibc-test.c | 68 +++++++++++++++++++
3 files changed, 166 insertions(+), 5 deletions(-)

Expand all Fold all

[PATCH v2 0/2] tools/nolibc: add support for [v]sscanf()

Posted by Thomas Weißschuh 12 months ago

The implementation is limited and only supports numeric arguments.

Signed-off-by: Thomas Weißschuh <linux@weissschuh.net>
---
Changes in v2:
- Return __LINE__ from different testcases to directly point to the
  failed testcase
- Add some comments
- Expand commit message
- Link to v1: https://lore.kernel.org/r/20240731-nolibc-scanf-v1-0-f71bcc4abb9e@weissschuh.net

---
Thomas Weißschuh (2):
      tools/nolibc: add support for [v]sscanf()
      Revert "selftests: kselftest: Fix build failure with NOLIBC"

 tools/include/nolibc/stdio.h                 | 98 ++++++++++++++++++++++++++++
 tools/testing/selftests/kselftest.h          |  5 --
 tools/testing/selftests/nolibc/nolibc-test.c | 68 +++++++++++++++++++
 3 files changed, 166 insertions(+), 5 deletions(-)
---
base-commit: 665fa8dea90d9fbc0e7137c7e1315d6f7e15757e
change-id: 20240414-nolibc-scanf-f1db6930d0c6

Best regards,
-- 
Thomas Weißschuh <linux@weissschuh.net>

Re: [PATCH v2 0/2] tools/nolibc: add support for [v]sscanf()

Posted by Willy Tarreau 12 months ago

On Sun, Feb 09, 2025 at 09:27:11PM +0100, Thomas Weißschuh wrote:
> The implementation is limited and only supports numeric arguments.
> 
> Signed-off-by: Thomas Weißschuh <linux@weissschuh.net>
> ---
> Changes in v2:
> - Return __LINE__ from different testcases to directly point to the
>   failed testcase
> - Add some comments
> - Expand commit message
> - Link to v1: https://lore.kernel.org/r/20240731-nolibc-scanf-v1-0-f71bcc4abb9e@weissschuh.net

Looks good. For the whole series:

Acked-by: Willy Tarreau <w@1wt.eu>

Thanks!
Willy