Re: [PATCH v16 03/14] lib: vsprintf: use _parse_integer() instead of _parse_integer_limit()
From: Rodrigo Alencar
Date: Thu Jun 04 2026 - 06:13:56 EST
On 26/06/04 10:58AM, Rodrigo Alencar via B4 Relay wrote:
> From: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>
>
> Use _parse_integer() that allows optional arguments to be explicitly
> initialised.
...
> diff --git a/lib/kstrtox.h b/lib/kstrtox.h
> index 00cf3255bdd2..ff84fe434a00 100644
> --- a/lib/kstrtox.h
> +++ b/lib/kstrtox.h
> @@ -12,6 +12,9 @@ unsigned int _parse_integer_limit(const char *s, unsigned int base, unsigned lon
> #define _parse_integer0(s, base, res, ...) \
> _parse_integer_limit(s, base, res, INT_MAX)
removed the semicolon here...
>
> +#define _parse_integer1(s, base, res, max_chars, ...) \
> + _parse_integer_limit(s, base, res, max_chars)
and here, as per sashiko's feedback:
https://sashiko.dev/#/patchset/20260602203706.103449-1-andriy.shevchenko%40linux.intel.com?part=1
https://sashiko.dev/#/patchset/20260602203706.103449-1-andriy.shevchenko%40linux.intel.com?part=2
> +
> #define _parse_integer(s, base, res, ...) \
> CONCATENATE(_parse_integer, COUNT_ARGS(__VA_ARGS__))(s, base, res, __VA_ARGS__)
--
Kind regards,
Rodrigo Alencar