<p>Dear folks</p>
<p>By doing so you are imposing implicit rules. It's bad to have any implicit rule.<br></p>
<p>I suggest amending this rule as:</p>
<p>Line width generally should be within 100 columns, and must not exceed 130 columns.<br>
Exception: An URI longer than 130 columns should be kept on one line so that it can be copied in its entirety.</p>
<p>Yours, Junxiao</p>
<div class="gmail_quote">On Feb 27, 2015 7:21 PM, "Alex Afanasyev" <<a href="mailto:alexander.afanasyev@ucla.edu">alexander.afanasyev@ucla.edu</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I personally, do not want to make changes for the the rule we have.  The current rule states:<br>
<br>
  - Lines should be within a reasonable range. Lines longer than 100 columns should generally be avoided.<br>
<br>
This rule was meant to be guideline and it still holds.  I would like to see lines to be mostly within 100 columns, as it is more convenient.  It doesn't preclude lines to be longer and doesn't require any justification to be put inside the code.  During the code review we can ask why line is longer or suggest to shorten it, but this is not something that worth having conversation for a long time (as we do now).<br>
<br>
---<br>
Alex<br>
<br>
> On Feb 27, 2015, at 4:59 PM, Davide Pesavento <<a href="mailto:davide.pesavento@lip6.fr">davide.pesavento@lip6.fr</a>> wrote:<br>
><br>
> As a hard limit, 130 sounds acceptable to me. Although if you're doing<br>
> side-by-side diffs/merges, or editing in two vertically-split panes,<br>
> 130 is way too much. Therefore I think we should still encourage<br>
> staying below 100 columns (or even 80), unless line wrapping in that<br>
> point makes the code really harder to read.<br>
><br>
> Best,<br>
> Davide<br>
><br>
> On Sat, Feb 28, 2015 at 1:33 AM, Junxiao Shi<br>
> <<a href="mailto:shijunxiao@email.arizona.edu">shijunxiao@email.arizona.edu</a>> wrote:<br>
>> Dear folks<br>
>><br>
>> ndn-cxx code style currently states that line width SHOULD be limited to 100<br>
>> columns.<br>
>> This matter has been disputed in several recent code reviews.<br>
>> Sometimes I'm also struggling to satisfy this requirement.<br>
>><br>
>> I suggest amending the rule to set 130 columns as a hard limit.<br>
>> 132 columns is a common terminal width, but two columns should be reserved<br>
>> for 'diff'.<br>
>><br>
>> What do others on this mailing list think?<br>
>><br>
>> Yours, Junxiao<br>
>><br>
>><br>
>> _______________________________________________<br>
>> Nfd-dev mailing list<br>
>> <a href="mailto:Nfd-dev@lists.cs.ucla.edu">Nfd-dev@lists.cs.ucla.edu</a><br>
>> <a href="http://www.lists.cs.ucla.edu/mailman/listinfo/nfd-dev" target="_blank">http://www.lists.cs.ucla.edu/mailman/listinfo/nfd-dev</a><br>
>><br>
> _______________________________________________<br>
> Nfd-dev mailing list<br>
> <a href="mailto:Nfd-dev@lists.cs.ucla.edu">Nfd-dev@lists.cs.ucla.edu</a><br>
> <a href="http://www.lists.cs.ucla.edu/mailman/listinfo/nfd-dev" target="_blank">http://www.lists.cs.ucla.edu/mailman/listinfo/nfd-dev</a><br>
<br>
</blockquote></div>