Re: [PATCH v4 12/20] vsprintf: add new `%pA` format specifier

From: Andy Shevchenko
Date: Mon Feb 14 2022 - 08:55:11 EST


On Mon, Feb 14, 2022 at 01:27:48PM +0100, Miguel Ojeda wrote:
> On Mon, Feb 14, 2022 at 11:19 AM Andy Shevchenko
> <andriy.shevchenko@xxxxxxxxxxxxxxx> wrote:
> >
> > Not sure I understand this...
> >
> > ...together with this in the current SoB chain.
>
> From my reading of `Documentation/process/submitting-patches.rst`,
> this is the case I have to use:
>
> ```
> Example of a patch submitted by a Co-developed-by: author::
>
> From: From Author <from@xxxxxxxxxxxxxxxxxx>
>
> <changelog>
>
> Co-developed-by: Random Co-Author <random@xxxxxxxxxxxxxxxxxxxx>
> Signed-off-by: Random Co-Author <random@xxxxxxxxxxxxxxxxxxxx>
> Signed-off-by: From Author <from@xxxxxxxxxxxxxxxxxx>
> Co-developed-by: Submitting Co-Author <sub@xxxxxxxxxxxxxxxxxxxx>
> Signed-off-by: Submitting Co-Author <sub@xxxxxxxxxxxxxxxxxxxx>
> ```
>
> Do you think another case applies?

I see, thanks for elaboration!

...

> > I'm wondering if you considered to use %pV.
>
> Please see Rasmus' reply, i.e. we are using Rust's own formatting
> machinery (the compiler validates the format string and creates an
> object that represents the formatting to be done).

Yes, I read that.

--
With Best Regards,
Andy Shevchenko