On Fri, Mar 18, 2022 at 7:04 AM Andy Shevchenko
<andriy.shevchenko@xxxxxxxxxxxxxxx> wrote:
>
> On Thu, Mar 17, 2022 at 11:11:21AM -0700, Nick Desaulniers wrote:
> > Our goal is to enable -Wformat for CC=clang. Please see also:
> > commit cbacb5ab0aa0 ("docs: printk-formats: Stop encouraging use of
> > unnecessary %h[xudi] and %hh[xudi]")
>
> Not that I agree on that commit for %h[h]x
>
> signed char ch = -1;
> printf("%x\n", ch);
> printf("%hhx\n", ch);
Will print:
ffffffff
ff
Maybe we should reconsider our recommendations for signed types?
It's probably worth adding `signed char` explicitly to
Documentation/core-api/printk-formats.rst, as it is a distinct type
from `char` in C.
--
Thanks,
~Nick Desaulniers
