<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><br>
<br>
Personally, I find this:<br>
<br>
if (a_good_variable_name-><wbr>another_good_name &&<br>
do_something_to_it(a_good_<wbr>variable_name->another_good_<wbr>name)) {<br>
    // Do things<br>
}<br></blockquote><div><br></div><div>I prefer this to the alternative as well.</div><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">
<br>
if (a_good_variable_name-><wbr>another_good_name<br>
    && do_something_to_it(a_good_<wbr>variable_name->another_good_<wbr>name)) {<br>
    // Do things<br>
}<br>
<br><snip><br>
<br>
Questions than:<br>
(1) Should there even be a line length rule?<br></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>(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_initial_unsolicited)</div><div><br></div><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>(3) Should we simply advocate for readability, with examples of what</div><div>is readable and what is not?</div><div><br></div><div>Readability should be first and foremost, and some examples would probably help with that.</div></div></div><div><br></div>-- <br><div class="gmail_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>
</div></div>