[PATCH] D117205: [clang-tidy] Support custom fix hint for cppcoreguidelines-pro-bounds-constant-array-index

Carlos Galvez via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jan 13 07:39:45 PST 2022


carlosgalvezp marked an inline comment as done.
carlosgalvezp added inline comments.


================
Comment at: clang-tools-extra/docs/ReleaseNotes.rst:156
 
+- Add new option ``cppcoreguidelines-pro-bounds-constant-array-index.FixHint``
+  to allow users to specify a different fix hint than ``gsl::at``.
----------------
Eugene.Zelenko wrote:
> carlosgalvezp wrote:
> > Eugene.Zelenko wrote:
> > > Please sort checks in this section alphabetically.
> > Hmm I'm not sure what you mean, should these bullets be ordered alphabetically? If so, why is "Fixed" showing up after "Updated"?
> Now `cppcoreguidelines-pro-bounds-constant-array-index` is before `cppcoreguidelines-explicit-virtual-functions`.
Ah, of course. Fixed!


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D117205/new/

https://reviews.llvm.org/D117205



More information about the cfe-commits mailing list