<div dir="ltr"><div>So it looks like we're pretty much all in agreement with 1-line and >=3 line comments. My only real concern is if "--ignore BLOCK_COMMENT_STYLE" in the patch removes the check for the long comments, though I confess to being ignorant of how checkpatch works.</div><div><br></div><div>To chip in my own $0.02 on the 2-line comment discussion: IMO they should use the same form as 1-line comments so that we have exactly one form for short comments one form for long comments.</div><div><br></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Aug 26, 2016 at 2:40 PM, Nico Huber <span dir="ltr"><<a href="mailto:nico.h@gmx.de" target="_blank">nico.h@gmx.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span>On <a href="tel:26.08.2016%2017" value="+12608201617" target="_blank">26.08.2016 17</a>:56, Vadim Bendebury wrote:<br>
> I actually tend to agree with Julius that it does not make sense to waste 4<br>
> lines for a two line comment.  So, ideally the tool should be enforcing the<br>
> verbose style for comments longer than say 2 lines.<br>
<br>
</span>Well, I too prefer the concise style for shorter comments. But I<br>
wouldn't enforce a number of lines. Maybe just say it's allowed for<br>
single-paragraph comments?<br>
<br>
And thanks, Julius, for bringing up the readability. This is what we<br>
should focus on. Actually I think a verbose-style comment shouldn't be<br>
allowed between code lines. That's something for longer explanations<br>
like a function description.<br>
<br>
But I'm really with Linus when it comes to the leading, single asterisk.<br>
It looks totally weird, IMHO<br>
<br>
  /* A small, concise comment<br>
     that doesn't fit a single line */<br>
<br>
is easier for the eye than<br>
<br>
  /* A small, concise comment<br>
   * that doesn't fit a single line */<br>
<br>
where your eyes somehow stick on the asterisk first and then have to<br>
search for the real start of the line.<br>
<br>
I wouldn't go as far as changing current code, but enforcing a style<br>
(we can agree on) for new code might prevent future discussions.<br>
<span class="m_-5813710102049437497HOEnZb"><font color="#888888"><br>
Nico<br>
</font></span><div class="m_-5813710102049437497HOEnZb"><div class="m_-5813710102049437497h5"><br>
--<br>
coreboot mailing list: <a href="mailto:coreboot@coreboot.org" target="_blank">coreboot@coreboot.org</a><br>
<a href="https://www.coreboot.org/mailman/listinfo/coreboot" rel="noreferrer" target="_blank">https://www.coreboot.org/mailm<wbr>an/listinfo/coreboot</a><br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="m_-5813710102049437497gmail_signature" data-smartmail="gmail_signature">David Hendricks (dhendrix)<br>Systems Software Engineer, Google Inc.</div>
</div></div>