Thread (19 messages) 19 messages, 10 authors, 2016-07-18

Re: [patch] crypto: sha256-mb - cleanup a || vs | typo

From: Tim Chen <hidden>
Date: 2016-07-18 22:12:38
Also in: kernel-janitors, lkml

On Mon, 2016-07-18 at 10:59 +0200, Pavel Machek wrote:
On Fri 2016-07-08 10:19:26, Linus Torvalds wrote:
quoted
[ rare comment rant. I think I'll do this once, and then ignore the discussion ]

On Fri, Jul 8, 2016 at 9:45 AM, Herbert Xu [off-list ref] wrote:
quoted

Nack.  As I said the commenting style in the crypto API is the
same as the network stack.  So unless we decide to change both
please stick to the current style.
Can we please get rid of the brain-damaged stupid networking comment
syntax style, PLEASE?
Please? :-). Having different comment styles between networking and
the rest is confusing.

And yes, this uglyness is documented for net/ and drivers/net/, but
not for crypto/, so at the very least Documentation/CodingStyle should
be updated.

									Pavel
The cleanup patch has already been merged.  Thanks.

Tim
--
To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help