bug#73068: printf: please implement POSIX:2024 argument reordering

2024-09-13 Thread Pádraig Brady
On 13/09/2024 13:55, Pádraig Brady wrote: On 12/09/2024 20:33, Paul Eggert wrote: On 2024-09-12 12:03, Pádraig Brady wrote: This is tricky enough, that we should be as restrictive as possible here, so I may resort to strspn(f, "0123456789") to parse instead. I'll think a bit about it. The c

bug#73068: printf: please implement POSIX:2024 argument reordering

2024-09-13 Thread Pádraig Brady
On 12/09/2024 20:33, Paul Eggert wrote: On 2024-09-12 12:03, Pádraig Brady wrote: This is tricky enough, that we should be as restrictive as possible here, so I may resort to strspn(f, "0123456789") to parse instead. I'll think a bit about it. The code's also assuming INT_MAX < INTMAX_MAX, w

bug#73068: printf: please implement POSIX:2024 argument reordering

2024-09-12 Thread Paul Eggert
On 2024-09-12 12:03, Pádraig Brady wrote: This is tricky enough, that we should be as restrictive as possible here, so I may resort to strspn(f, "0123456789") to parse instead. I'll think a bit about it. The code's also assuming INT_MAX < INTMAX_MAX, which POSIX doesn't require. You could pu

bug#73068: printf: please implement POSIX:2024 argument reordering

2024-09-12 Thread Pádraig Brady
On 12/09/2024 18:40, Collin Funk wrote: Hi Pádraig, Pádraig Brady writes: I'll apply the attached sometime tomorrow. Marking this as done. Patch looks good, thanks. One small comment, though. +#define GET_CURR_ARG(POS) \ +do {

bug#73068: printf: please implement POSIX:2024 argument reordering

2024-09-12 Thread Pádraig Brady
On 12/09/2024 18:06, Bruno Haible wrote: Pádraig Brady wrote: I'll apply the attached sometime tomorrow. Nice! Thank you. There seems to be a typo in the unit test, though: It defines a shell function 'printf_checki_err' but the function it then invokes is 'printf_check_err'. Hah, good catc

bug#73068: printf: please implement POSIX:2024 argument reordering

2024-09-12 Thread Collin Funk
Hi Pádraig, Pádraig Brady writes: > I'll apply the attached sometime tomorrow. > > Marking this as done. Patch looks good, thanks. One small comment, though. > +#define GET_CURR_ARG(POS)\ > +do { \ > + char *arge;

bug#73068: printf: please implement POSIX:2024 argument reordering

2024-09-12 Thread Bruno Haible
Pádraig Brady wrote: > I'll apply the attached sometime tomorrow. Nice! Thank you. There seems to be a typo in the unit test, though: It defines a shell function 'printf_checki_err' but the function it then invokes is 'printf_check_err'. Bruno

bug#73068: printf: please implement POSIX:2024 argument reordering

2024-09-12 Thread Pádraig Brady
On 09/09/2024 19:30, Pádraig Brady wrote: On 06/09/2024 15:06, Bruno Haible wrote: Hi, POSIX:2024 specifies that printf(1) should support numbered conversion specifications: https://pubs.opengroup.org/onlinepubs/9799919799/utilities/printf.html https://austingroupbugs.net/view.php?id=1592 Coul

bug#73068: printf: please implement POSIX:2024 argument reordering

2024-09-09 Thread Pádraig Brady
On 06/09/2024 15:06, Bruno Haible wrote: Hi, POSIX:2024 specifies that printf(1) should support numbered conversion specifications: https://pubs.opengroup.org/onlinepubs/9799919799/utilities/printf.html https://austingroupbugs.net/view.php?id=1592 Could this support please be added to GNU coreu

bug#73068: printf: please implement POSIX:2024 argument reordering

2024-09-06 Thread Bruno Haible
Hi, POSIX:2024 specifies that printf(1) should support numbered conversion specifications: https://pubs.opengroup.org/onlinepubs/9799919799/utilities/printf.html https://austingroupbugs.net/view.php?id=1592 Could this support please be added to GNU coreutils? As of coreutils 9.5, I still get: