<div dir="ltr">r198694, thanks!<br></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Jan 7, 2014 at 12:02 AM, Daniel Jasper <span dir="ltr"><<a href="mailto:djasper@google.com" target="_blank">djasper@google.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><p dir="ltr">Sounds like a good idea to me! </p><div class="HOEnZb"><div class="h5">
<div class="gmail_quote">On Jan 7, 2014 6:55 AM, "Nico Weber" <<a href="mailto:thakis@chromium.org" target="_blank">thakis@chromium.org</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div dir="ltr">Hi,<br><br>`clang-format --version` currently uses llvm's default version formatter which looks like this:<br><br>  $ clang-format  --version<br>  LLVM (<a href="http://llvm.org/" target="_blank">http://llvm.org/</a>):<br>


    LLVM version 3.5svn<br>
    Optimized build with assertions.<br>    Built Jan  3 2014 (14:28:46).<br>    Default target: x86_64-apple-darwin13.0.0<br>    Host CPU: core-avx-i<br><br>This includes all kinds of stuff that's not interesting for a code formatter, and it lacks the revision clang-format was built at. In comparison, `clang --version` prints<br>



<br>  $ bin/clang --version<br>  clang version 3.5 (198452)<br>  Target: x86_64-apple-darwin13.0.0<br>  Thread model: posix<br><br>which I think is much better (but Target: and arguably Thread model: aren't interesting for a formatter either).<br>



<br>The attached patch changes the output of `clang-format --version` to be just<br><br>  $ bin/clang-format --version<br>  clang-format version 3.5 (198452)<br><div><br></div><div>The part I care about most is including the revision number.</div>


<div><br></div><div>Ok?</div><div><br></div><div>Nico</div></div>
</blockquote></div>
</div></div></blockquote></div><br></div>