linux-kernel - Re: [RFC PATCH 3/5] kobject: Fix kernel-doc comment first line

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Thu, 2 May 2019 10:39:22 +0200
From:   Johan Hovold <johan@...nel.org>
To:     "Tobin C. Harding" <me@...in.cc>
Cc:     Johan Hovold <johan@...nel.org>,
        "Tobin C. Harding" <tobin@...nel.org>,
        Josh Poimboeuf <jpoimboe@...hat.com>,
        Jiri Kosina <jikos@...nel.org>,
        Miroslav Benes <mbenes@...e.cz>,
        Petr Mladek <pmladek@...e.com>,
        Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
        "Rafael J. Wysocki" <rafael@...nel.org>,
        Joe Lawrence <joe.lawrence@...hat.com>,
        Jonathan Corbet <corbet@....net>,
        live-patching@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [RFC PATCH 3/5] kobject: Fix kernel-doc comment first line

On Thu, May 02, 2019 at 06:25:39PM +1000, Tobin C. Harding wrote: > Adding Jon to CC
> 
> On Thu, May 02, 2019 at 09:38:23AM +0200, Johan Hovold wrote:
> > On Thu, May 02, 2019 at 12:31:40PM +1000, Tobin C. Harding wrote:
> > > kernel-doc comments have a prescribed format.  This includes parenthesis
> > > on the function name.  To be _particularly_ correct we should also
> > > capitalise the brief description and terminate it with a period.
> > 
> > Why do think capitalisation and full stop is required for the function
> > description?
> > 
> > Sure, the example in the current doc happen to use that, but I'm not
> > sure that's intended as a prescription.
> > 
> > The old kernel-doc nano-HOWTO specifically did not use this:
> > 
> > 	https://www.kernel.org/doc/Documentation/kernel-doc-nano-HOWTO.txt
> > 
> 
> Oh?  I was basing this on Documentation/doc-guide/kernel-doc.rst
> 
> 	Function documentation
> 	----------------------
> 
> 	The general format of a function and function-like macro kernel-doc comment is::
> 
> 	  /**
> 	   * function_name() - Brief description of function.
> 	   * @arg1: Describe the first argument.
> 	   * @arg2: Describe the second argument.
> 	   *        One can provide multiple line descriptions
> 	   *        for arguments.
> 
> I figured that was the canonical way to do kernel-doc function
> comments.  I have however refrained from capitalising and adding the
> period to argument strings to reduce code churn.  I figured if I'm
> touching the line to add parenthesis then I might as well make it
> perfect (if such a thing exists).

I think you may have read too much into that example. Many of the
current function and parameter descriptions aren't even full sentences,
so sentence case and full stop doesn't really make any sense.

Looks like we discussed this last fall as well:

	https://lkml.kernel.org/r/20180912093116.GC1089@localhost

Johan