[PATCH] D50389: [clang-tidy] new check for Abseil

Jonas Toth via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Aug 10 08:29:55 PDT 2018


JonasToth added a comment.

LGTM with only the formatting question.

But don't commit before any of the reviewers accepts (@alexfh @aaron.ballman usually have the last word)



================
Comment at: clang-tidy/abseil/DurationDivisionCheck.cpp:50
+                 *result.SourceManager, result.Context->getLangOpts()),
+             ")");
+}
----------------
This line looks odd, does it come from clang-format?


https://reviews.llvm.org/D50389





More information about the cfe-commits mailing list