[PATCH] D53339: [clang-tidy] Add the abseil-duration-factory-float check

Hyrum Wright via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Oct 23 08:40:13 PDT 2018


hwright added inline comments.


================
Comment at: clang-tidy/abseil/DurationFactoryFloatCheck.h:19
+
+/// Prefer integer Duration factories when possible.
+///
----------------
JonasToth wrote:
> Please add more to the doc here, like `This check finds ... and transforms these calls into ...` or similar.
I've added some more text.

I'd prefer not to make this duplicate much of what is already in the user-facing documentation, since that should be more complete and canonical (and is already included by reference).


https://reviews.llvm.org/D53339





More information about the cfe-commits mailing list