Re: [PATCH v1] MAINTAINERS: Sort entries in database for VSPRINTF

From: Andy Shevchenko
Date: Mon Feb 10 2020 - 09:51:33 EST


On Mon, Feb 10, 2020 at 03:21:55PM +0100, Petr Mladek wrote:
> Hi Andy,
>
> On Tue 2020-01-28 16:34:25, Andy Shevchenko wrote:
> > Run parse-maintainers.pl and choose VSPRINTF record. Fix it accordingly.
>
> Does this produce any visible error or warning anywhere, please?

Not for the moment.
Nevertheless, there is an initial motivation behind this [1,2].


> checkpatch.pl does not warn about it.
>
> Also the order does not look defined in the file. When I run
> parse-maintainers.pl on the entire MAINTAINERS file:

See [2] for the details.

> MAINTAINERS | 5584 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++-----------------------------------------------------------------
> 1 file changed, 2787 insertions(+), 2797 deletions(-)
>
> The file has 18545 lines. It means that huge amount of entries
> do not follow the order.

Yes, but it's getting better.

[1]: 7683e9e52925 ("Properly alphabetize MAINTAINERS file")
[2]: https://lore.kernel.org/lkml/CA+55aFy3naVgbRubhjfq7k4CcSiFOEdQNkNwHTLDLmepECu9yA@xxxxxxxxxxxxxx/

--
With Best Regards,
Andy Shevchenko