<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Mar 16, 2017 at 2:35 PM, Kevin Harwell <span dir="ltr"><<a href="mailto:kharwell@digium.com" target="_blank">kharwell@digium.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, Mar 16, 2017 at 2:54 PM, Matthew Jordan <span dir="ltr"><<a href="mailto:mjordan@digium.com" target="_blank">mjordan@digium.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><snip><span class=""><br>
<br>
Personally, I find this:<br>
<br>
if (a_good_variable_name->another<wbr>_good_name &&<br>
do_something_to_it(a_good_vari<wbr>able_name->another_good_name)) {<br>
    // Do things<br>
}<br></span></blockquote><div><br></div><div>I prefer this to the alternative as well.</div></div></div></div></blockquote><div><br></div><div>So don't indent wrapped predicates or is that just email formatting?</div><div> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="">
<br>
if (a_good_variable_name->another<wbr>_good_name<br>
    && do_something_to_it(a_good_vari<wbr>able_name->another_good_name)) {<br>
    // Do things<br>
}<br></span></blockquote></div></div></div></blockquote><div><br></div><div>I actually prefer this one with the operators at the front but I'm fine either way.</div><div> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="">
<br></span><snip><span class=""><br>
<br>
Questions than:<br>
(1) Should there even be a line length rule?<br></span></blockquote><div><br></div><div><div>I lean toward yes just to keep things sane (e.g. keeping someone extending a function definition to infinity and beyond). Also sometimes I split my editor's screen vertically and have multiple code views up. The vertical split reduces my column length and I either have to start scrolling or (in my current setup) the line gets wrapped. Both hurt readability for me.</div></div></div></div></div></blockquote><div><br></div><div>This is where choice of editor or IDE comes into play.  I use Eclipse so my left pane is a navigator, the center pane is the editor and the right pane is search/outline/bookmarks.  Even with that, I have my column marker set at 100 which works well (for me).  Still though, there's code, mostly function prototypes or definitions, that make me nuts because they run well beyond 100 characters.  Even more annoying are header files like res_pjsip_session.h where both declarations and comments go out to 200 columns.  </div><div> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div><span class=""><div> </div><div>(2) If there is a line length, what is a reasonable length given some</div><div>of our function names? (Looking at things like </div><div>ast_sip_get_mwi_disable_<wbr>initial_unsolicited)</div><div><br></div></span><div>I personally think the current length of 90 is reasonable. I'll add though that going a bit above this (say up to 100) in some cases should not prevent code submission. For instance if a variable name is going to extend an 'if' statement 4 more characters there is no reason to break that up.</div></div></div></div></div></blockquote><div><br></div><div>My vote is 100.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div><span class=""><div> </div><div>(3) Should we simply advocate for readability, with examples of what</div><div>is readable and what is not?</div><div><br></div></span><div>Readability should be first and foremost, and some examples would probably help with that.</div></div></div></div></div></blockquote><div><br></div><div>+1000.  I can adapt and so can my editor but having real guidelines is key.  </div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><span class="HOEnZb"><font color="#888888"><div><br></div>-- <br><div class="m_-7168618871965579779gmail_signature"><div dir="ltr"><pre style="padding:2px;border:1px solid rgb(114,99,77);background-color:rgb(238,238,238);color:rgb(0,0,0);overflow:auto">Kevin Harwell
Digium, Inc. | Software Developer
445 Jan Davis Drive NW - Huntsville, AL 35806 - USA
Check us out at: <a href="http://digium.com" target="_blank">http://digium.com</a> & <a href="http://asterisk.org" target="_blank">http://asterisk.org</a></pre></div></div>
</font></span></div></div>
<br>--<br>
______________________________<wbr>______________________________<wbr>_________<br>
-- Bandwidth and Colocation Provided by <a href="http://www.api-digital.com" rel="noreferrer" target="_blank">http://www.api-digital.com</a> --<br>
<br>
asterisk-dev mailing list<br>
To UNSUBSCRIBE or update options visit:<br>
   <a href="http://lists.digium.com/mailman/listinfo/asterisk-dev" rel="noreferrer" target="_blank">http://lists.digium.com/<wbr>mailman/listinfo/asterisk-dev</a><br></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><span style="font-size:12.8px">George Joseph</span><br style="font-size:12.8px"><span style="font-size:12.8px">Digium, Inc. | Software Developer</span><span style="font-size:12.8px"><br>445 Jan Davis Drive NW - Huntsville, AL 35806 - US<br></span><span style="font-size:12.8px">Check us out at: </span><a href="http://www.digium.com/" rel="noreferrer" style="color:rgb(17,85,204);font-size:12.8px" target="_blank">www.digium.com</a><span style="font-size:12.8px"> & </span><a href="http://www.asterisk.org/" rel="noreferrer" style="color:rgb(17,85,204);font-size:12.8px" target="_blank">www.asterisk.org</a><br><div><br></div></div></div>
</div></div>