<div dir="ltr">[resending, this time to the list as well]<div><br></div><div><span class="gmail-im" style="font-size:12.8px">On Tue, Aug 30, 2016 at 6:56 AM, Pierre Fenoll <span dir="ltr"><<a href="mailto:pierrefenoll@gmail.com" target="_blank">pierrefenoll@gmail.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">erl_tidy or erl_prettypr:format is missing a lot of options. For example:<br><br>A long spec might get broken down into multiple lines (and that's fine). However I find it surprising (and hard to grep) that newlines are inserted right after the -spec attribute.<br><br></blockquote><div><br></div></span><div style="font-size:12.8px">I think producing formatted output that matches with what people are likely to grep for is very important.<br></div><div style="font-size:12.8px"><br></div><div style="font-size:12.8px">The commit says getting the "new abstract tree nodes and their associated functions" right was the main priority and that contributions that fix formatting "are very welcome".  In my limited experience (one PR), the Erlang team is very responsive to patches.</div><span class="gmail-im" style="font-size:12.8px"><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br>Later on we could agree on which options we want to have as standard.<br><br></blockquote><div><br></div></span><div style="font-size:12.8px">BTW, I found an informative slide show [1] on the history of gofmt.  This slide describes how gofmt became the standard:<a href="https://talks.golang.org/2015/gofmt-en.slide#26" target="_blank">https://talks.golang.org/2015/<wbr>gofmt-en.slide#26</a></div><div style="font-size:12.8px"><br></div><div style="font-size:12.8px">Some other bullets from the slideshow that I found interesting:</div><div style="font-size:12.8px"><br></div><div style="font-size:12.8px">    * No knobs!<br></div><div style="font-size:12.8px"><p>    * Missing insight: "good enough" uniform formatting style is better than having lots of different formats.<br></p><p>    * Value of style guide: Uniformity, not perfection.</p><p>    * Don't fret over line length control.</p><p>    * Instead, respect user: Consider line breaks in original source.</p><p></p><p>    * Single biggest mistake: comments not attached to AST nodes.</p></div><div style="font-size:12.8px">Mark</div><div style="font-size:12.8px"><br></div><div style="font-size:12.8px">[1] <a href="https://talks.golang.org/2015/gofmt-en.slide#1" target="_blank">https://talks.golang.org/<wbr>2015/gofmt-en.slide#1</a></div></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Aug 30, 2016 at 6:56 AM, Pierre Fenoll <span dir="ltr"><<a href="mailto:pierrefenoll@gmail.com" target="_blank">pierrefenoll@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">> "indentation only" or "full rewrite"<br>
<br>
</span>erl_tidy or erl_prettypr:format is missing a lot of options. For example:<br>
<br>
A long spec might get broken down into multiple lines (and that's fine). However I find it surprising (and hard to grep) that newlines are inserted right after the -spec attribute.<br>
<br>
Options to toggle such behavior are I think needed in erl_tidy.<br>
<br>
Later on we could agree on which options we want to have as standard.<br>
<span class="im HOEnZb"><br>
> On 30 Aug 2016, at 09:54, Vlad Dumitrescu <<a href="mailto:vladdu55@gmail.com">vladdu55@gmail.com</a>> wrote:<br>
><br>
> "indentation only" or "full rewrite"<br>
</span><div class="HOEnZb"><div class="h5">______________________________<wbr>_________________<br>
erlang-questions mailing list<br>
<a href="mailto:erlang-questions@erlang.org">erlang-questions@erlang.org</a><br>
<a href="http://erlang.org/mailman/listinfo/erlang-questions" rel="noreferrer" target="_blank">http://erlang.org/mailman/<wbr>listinfo/erlang-questions</a><br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><span style="font-size:12.8px">Blogging at <a href="http://markbucciarelli.com" target="_blank">markbucciarelli.com</a></span><br></div><div>Tweeting @mbucc</div></div></div></div></div>
</div>