[PATCH] D54246: [clang-tidy] Add the abseil-duration-factory-scale check

Eugene Zelenko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Nov 8 10:12:19 PST 2018


Eugene.Zelenko added inline comments.


================
Comment at: docs/ReleaseNotes.rst:88
+  Checks for cases where arguments to ``absl::Duration`` factory functions are
+  scaled internally and could be changed to a different factory function.  This
+  check also looks for arguements with a zero value and suggests using
----------------
hwright wrote:
> Eugene.Zelenko wrote:
> > Second statement belongs to documentation.
> It already is in the user-facing documentation, are saying it should be removed from here?
Release Notes are for brief descriptions. Documentation is for details.


https://reviews.llvm.org/D54246





More information about the cfe-commits mailing list