<div dir="ltr"><div dir="ltr" class="gmail_msg">So, I forwarded the request for highlighting trailing whitespaces to phabricator upstream (<a href="https://secure.phabricator.com/T11879">https://secure.phabricator.com/T11879</a>), and upstream folks suggest we enable the Lint feature in Arcanist (<a href="https://secure.phabricator.com/book/phabricator/article/arcanist_lint/">https://secure.phabricator.com/book/phabricator/article/arcanist_lint/</a>). This will enforce the check when `arc diff` is run (reviewers wouldn't see the warnings though).</div><div dir="ltr" class="gmail_msg"><br></div><div class="gmail_msg">There are two linters we might be interested in enabling:</div><div class="gmail_msg"><div class="gmail_msg">- cpplint (code style checker based on <a href="https://github.com/google/styleguide">cpplint.py</a>)</div><div class="gmail_msg">- cppcheck (C++ linter based on <a href="http://cppcheck.sourceforge.net/">cppcheck</a>)</div></div><div dir="ltr" class="gmail_msg"><br></div><div>Note that cpplint assumes google code style, but I think it can potentially be replaced it with clang-format with configurable code styles. </div><div><br class="gmail_msg"><div class="gmail_quote gmail_msg"><div dir="ltr" class="gmail_msg">On Wed, Nov 16, 2016 at 12:28 PM Eric Liu <<a href="mailto:ioeric@google.com" class="gmail_msg" target="_blank">ioeric@google.com</a>> wrote:<br class="gmail_msg"></div><blockquote class="gmail_quote gmail_msg" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr" class="gmail_msg">I'm not sure how easy it is to get clang-format into Phabricator since it is mostly developed by (phab) upstream. I'll file feature request regarding trailing whitespaces and clang-format to upstream to see what upstream forks think. <div class="gmail_msg"><br class="gmail_msg"></div><div class="gmail_msg">But a bot checking revision sounds great. It might also be useful to have the bot run clang-tidy and post findings/FixHints as comments on the patch. </div></div><div dir="ltr" class="gmail_msg"><br class="gmail_msg"><div class="gmail_quote gmail_msg"><div dir="ltr" class="gmail_msg">On Wed, Nov 16, 2016 at 12:05 PM Johannes Doerfert via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org" class="gmail_msg" target="_blank">llvm-dev@lists.llvm.org</a>> wrote:<br class="gmail_msg"></div><blockquote class="gmail_quote gmail_msg" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 11/14, Mehdi Amini via llvm-dev wrote:<br class="gmail_msg">
> Ideally I’d even really like to have a both checking for revision on<br class="gmail_msg">
> phab, clang-formatting them, and post a comment when there is a<br class="gmail_msg">
> mismatch :)<br class="gmail_msg">
I'd like that!<br class="gmail_msg">
_______________________________________________<br class="gmail_msg">
LLVM Developers mailing list<br class="gmail_msg">
<a href="mailto:llvm-dev@lists.llvm.org" class="gmail_msg" target="_blank">llvm-dev@lists.llvm.org</a><br class="gmail_msg">
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" class="gmail_msg" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev</a><br class="gmail_msg">
</blockquote></div></div></blockquote></div></div></div>