<div dir="ltr"><div><br class="">On Sun, May 19, 2013 at 5:35 PM, Manuel Klimek <span dir="ltr"><<a href="mailto:klimek@google.com" target="_blank">klimek@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<div dir="ltr">Also, I'd suggest to not submit what phabricator produces - those messages contain way too much noise for the signal.</div></blockquote><div><br></div><div style>In this particular case log message generated by phabricator's arc utility is: </div>
</div><div><br></div><blockquote style="margin:0px 0px 0px 40px;border:none;padding:0px"><div style>Clang-format: allow -style="{yaml/json}" on command line</div><div style><br></div><div style>Summary: + improved handling of default style and predefined styles.</div>
<div style><br></div><div style>Reviewers: djasper, klimek</div><div style><br></div><div style>Reviewed By: klimek</div><div style><br></div><div style>CC: cfe-commits</div><div style><br></div><div style>Differential Revision: <a href="http://llvm-reviews.chandlerc.com/D813">http://llvm-reviews.chandlerc.com/D813</a></div>
</blockquote><div><br></div><div class="gmail_extra">The first two non-empty lines here are human-written (except for the "Summary:" word). I would also categorize everything else in this message except for the "CC: cfe-commits" part as useful information. So my estimate says the SNR here is quite high. Or am I missing something? Do you have suggestions on how to improve this message?<br>
<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div class="gmail_extra"><br>
<div class="gmail_quote"><div><div class="h5">On Sun, May 19, 2013 at 1:08 PM, Alexander Kornienko <span dir="ltr"><<a href="mailto:alexfh@google.com" target="_blank">alexfh@google.com</a>></span> wrote:<br>
</div></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div><div class="h5"><div dir="ltr">I thought it should be rather clear. But if you think otherwise, I'll try to explain:<div>
This commit adds support for specifying style configuration in yaml/json format directly as a value of -style command-line option, e.g. -style="{BasedOnStyle: llvm, IndentWidth: 8}". In addition to that it improves getPredefinedStyle method and code handling default style (when no -style specified on command line, or when an invalid style specified, or when BasedOnStyle parameter is not specified in the used .clang-format file, or when BasedOnStyle contains invalid style name). "Improved" here means that there's now only one place where default style name is specified, and fewer places where it's used. </div>


<div><br></div><div>Is this detailed enough?</div></div><div class="gmail_extra"><div><div><br><br><div class="gmail_quote">On Sun, May 19, 2013 at 6:58 AM, Chandler Carruth <span dir="ltr"><<a href="mailto:chandlerc@google.com" target="_blank">chandlerc@google.com</a>></span> wrote:<br>


<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div><br><div class="gmail_quote">
On Sun, May 19, 2013 at 12:53 AM, Alexander Kornienko <span dir="ltr"><<a href="mailto:alexfh@google.com" target="_blank">alexfh@google.com</a>></span> wrote:<br>


<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div style="overflow:hidden">Clang-format: allow -style="{yaml/json}" on command line<br>

<br>
Summary: + improved handling of default style and predefined styles.</div></blockquote></div><br></div>This description doesn't make a whole lot of sense... Can you clarify what is in this commit? (And watch out for this in the future, or fix whatever tool if it is a tool that messes this up?)</div>



</div>
</blockquote></div><br><br clear="all"><div><br></div></div></div><span><font color="#888888">-- <br><div><div><font color="#666666"><span style="border-width:2px 0px 0px;border-style:solid;border-color:rgb(213,15,37);padding-top:2px;margin-top:2px">Alexander Kornienko |</span><span style="border-width:2px 0px 0px;border-style:solid;border-color:rgb(51,105,232);padding-top:2px;margin-top:2px"> Software Engineer |</span></font><span style="border-width:2px 0px 0px;border-style:solid;border-color:rgb(0,153,57);padding-top:2px;margin-top:2px"><font color="#666666"> </font><a href="mailto:alexfh@google.com" style="color:rgb(17,85,204)" target="_blank">alexfh@google.com</a> |</span><span style="border-width:2px 0px 0px;border-style:solid;border-color:rgb(238,178,17);padding-top:2px;margin-top:2px"> <a value="+35315435283" style="color:rgb(17,85,204)">+49 151 221 77 957</a></span></div>


</div><div><font color="#666666"><span style="font-family:Arial,Verdana,sans-serif">Google Germany GmbH | </span><span style="font-family:Arial,Verdana,sans-serif">Dienerstr. 12 | </span><span style="font-family:Arial,Verdana,sans-serif">80331 München</span></font></div>



</font></span></div>
<br></div></div><div class="im">_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu" target="_blank">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
<br></div></blockquote></div><br></div>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div><div><font color="#666666"><span style="border-width:2px 0px 0px;border-style:solid;border-color:rgb(213,15,37);padding-top:2px;margin-top:2px">Alexander Kornienko |</span><span style="border-width:2px 0px 0px;border-style:solid;border-color:rgb(51,105,232);padding-top:2px;margin-top:2px"> Software Engineer |</span></font><span style="border-width:2px 0px 0px;border-style:solid;border-color:rgb(0,153,57);padding-top:2px;margin-top:2px"><font color="#666666"> </font><a href="mailto:alexfh@google.com" style="color:rgb(17,85,204)" target="_blank">alexfh@google.com</a> |</span><span style="border-width:2px 0px 0px;border-style:solid;border-color:rgb(238,178,17);padding-top:2px;margin-top:2px"> <a value="+35315435283" style="color:rgb(17,85,204)">+49 151 221 77 957</a></span></div>
</div><div><font color="#666666"><span style="background-color:rgb(255,255,255);font-family:Arial,Verdana,sans-serif">Google Germany GmbH | </span><span style="background-color:rgb(255,255,255);font-family:Arial,Verdana,sans-serif">Dienerstr. 12 | </span><span style="background-color:rgb(255,255,255);font-family:Arial,Verdana,sans-serif">80331 München</span></font></div>

</div></div>