mbox series

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

Message ID 20250209-nolibc-scanf-v2-0-c29dea32f1cd@weissschuh.net (mailing list archive)
Headers show
Series tools/nolibc: add support for [v]sscanf() | expand

Message

Thomas Weißschuh Feb. 9, 2025, 8:27 p.m. UTC
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,

Comments

Willy Tarreau Feb. 10, 2025, 6:29 a.m. UTC | #1
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