linux-kernel - Re: [PATCH v3] staging: greybus: checkpatch: Change parameter type unsigned to unsigned int

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, 12 Jan 2017 10:05:10 +0300
From:   Dan Carpenter <dan.carpenter@...cle.com>
To:     Roman Sommer <roman.sommer@....de>
Cc:     devel@...verdev.osuosl.org, greybus-dev@...ts.linaro.org,
        Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
        linux-kernel@...cs.fau.de, Johan Hovold <johan@...nel.org>,
        linux-kernel@...r.kernel.org,
        Christian Bewermeyer <christian.bewermeyer@....de>
Subject: Re: [PATCH v3] staging: greybus: checkpatch: Change parameter type
 unsigned to unsigned int

On Wed, Jan 11, 2017 at 04:00:35PM +0100, Roman Sommer wrote:
> Note that this patch does not fix all checkpatch warnings for the
> affected files.
> 

A lot of ways that display git commits put the changelog and the summary
far apart.  The changelog is pretty useless by itself in this case.

https://lkml.org/lkml/2017/1/11/308

See how it puts the date in between?  marc.info is even worse but google
hasn't indexed it yet and I'm too lazy to look it up.  So people read
the summary or the changelog but not necessarily both.

regards,
dan carpenter