Re: [PATCH] docs: dev-tools: checkpatch: Add targets for checkpatch tags

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Diederik de Haas <didi.debian@xxxxxxxxx> writes:

> Make the tags directly linkable by defining targets for them.
>
> Closes: https://lore.kernel.org/r/8090211.0vHzs8tI1a@bagend/
> Signed-off-by: Diederik de Haas <didi.debian@xxxxxxxxx>
> ---
>  Documentation/dev-tools/checkpatch.rst | 216 +++++++++++++++++++++++++
>  1 file changed, 216 insertions(+)

In addition to the other comments, I have to add: you have said what you
are doing but not *why*.  All of these labels clutter up the text; what
use will be made of them to justify that?

Thanks,

jon




[Index of Archives]     [Kernel Newbies]     [Security]     [Netfilter]     [Bugtraq]     [Linux FS]     [Yosemite Forum]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux