Messages in this thread | | | Subject | Re: [PATCH 0/2] printf: add support for %de | From | Rasmus Villemoes <> | Date | Tue, 21 Jan 2020 11:27:34 +0100 |
| |
On 20/01/2020 10.32, Andy Shevchenko wrote: > On Mon, Jan 20, 2020 at 10:57 AM Uwe Kleine-König > <u.kleine-koenig@pengutronix.de> wrote: > >> this is an reiteration of my patch from some time ago that introduced >> %dE with the same semantic. Back then this resulted in the support for >> %pe which was less contentious. >> > >> I still consider %de (now with a small 'e' to match %pe) useful. > > Please, don't spread the extensions over the standard specifiers. The > %p* extensions are enough for my opinion. > NAK. >
As I think I already mentioned (and what led me to do the %pe), I'm with Andy and Joe here, I don't think modifying the behaviour of stuff other than %p is a good idea - especially not when it's only about saving a few characters to avoid the ERR_PTR() wrapping.
Rasmus
| |