
<html><head><meta http-equiv="Content-Type" content="text/html charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class="">I see! That’d be nice to have indeed.<div class=""><br class=""></div><div class="">Ideally I’d even really like to have a both checking for revision on phab, clang-formatting them, and post a comment when there is a mismatch :)</div><div class=""><br class=""></div><div class="">— </div><div class="">Mehdi</div><div class=""><br class=""></div><div class=""><br class=""><div><blockquote type="cite" class=""><div class="">On Nov 14, 2016, at 2:43 PM, Rui Ueyama <<a href="mailto:ruiu@google.com" class="">ruiu@google.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div dir="ltr" class="">When I create a patch, I run clang-format-diff, so that's fine. What I want to do is to find trailing whitespaces in others patches.</div><div class="gmail_extra"><br class=""><div class="gmail_quote">On Mon, Nov 14, 2016 at 2:30 PM, Mehdi Amini <span dir="ltr" class=""><<a href="mailto:mehdi.amini@apple.com" target="_blank" class="">mehdi.amini@apple.com</a>></span> wrote:<br class=""><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br class="">

<br class="">

clang-format removes them.<br class="">

If you are (like me) likely to forget to run "git clang-format HEAD~” before pushing, adding a pre-commit (or pre-push) hook is helpful.<br class="">

<br class="">

—<br class="">

Mehdi<br class="">

<span class=""><br class="">

<br class="">

> On Nov 14, 2016, at 2:01 PM, Rui Ueyama via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org" class="">llvm-dev@lists.llvm.org</a>> wrote:<br class="">

><br class="">

> I wonder if Phabricator has an ability to highlight trailing whitespaces, because I often find trailing whitespaces after they are submitted. If they were highlighted, they would have been removed before submitting. Does anyone know about this?<br class="">

</span>> ______________________________<wbr class="">_________________<br class="">

> LLVM Developers mailing list<br class="">

> <a href="mailto:llvm-dev@lists.llvm.org" class="">llvm-dev@lists.llvm.org</a><br class="">

> <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" target="_blank" class="">http://lists.llvm.org/cgi-bin/<wbr class="">mailman/listinfo/llvm-dev</a><br class="">

<br class="">

</blockquote></div><br class=""></div>

</div></blockquote></div><br class=""></div></body></html>