Re: [PATCH v2 v2] Documentation: proc: fix section numbering in table of contents

[Randy Dunlap]

On 4/27/26 2:55 AM, Jonathan Corbet wrote:
> Randy Dunlap <rdunlap@xxxxxxxxxxxxx> writes:
> 
>> On 4/24/26 2:06 AM, Baolin Liu wrote:
>>> From: Baolin Liu <liubaolin@xxxxxxxxxx>
>>>
>>> Commit e24ccaaf7ec4 ("block: remove last remaining traces of IDE
>>> documentation") removed the IDE section but left its table of
>>> contents entry behind.
>>> Fix the stale entry and renumber the following sections.
>>>
>>> Fixes: e24ccaaf7ec4 ("block: remove last remaining traces of IDE documentation")
>>> Signed-off-by: Baolin Liu <liubaolin@xxxxxxxxxx>
>>
>> LGTM.
>> Acked-by: Randy Dunlap <rdunlap@xxxxxxxxxxxxx>
>>
>> I note that the html-generated TOC is already correct;
>> e.g., https://docs.kernel.org/filesystems/proc.html#networking-info-in-proc-net
>> is already section 1.3.
> 
> There are advantages to having a computer do a job like that :)
> 
> I do really question the value of these manually maintained TOCs; they
> are always going to be out of date.  But ...
> 
>> AFAICT, this TOC is for users/readers who use an editor or pager
>> to read this file, but the patch is useful in that setting.
> 
> Others seem to disagree, so we keep them.
> 
> I've applied the patch, thanks.

Yes, others do disagree, but my comment was not meant to be an endorsement
for keeping it there. I'm fine without it.

-- 
~Randy