[PATCH] D56661: [clang-tidy] Fix incorrect array name generation in cppcoreguidelines-pro-bounds-constant-array-index
Jonas Toth via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Sep 12 03:03:00 PDT 2019
JonasToth added a comment.
Sorry for the long delay!
I have one question about a slighty different code constellation.
std::optional<std::array<int, 4>> my_array;
for(int i = 0; i < 42; ++i)
(*my_array)[i] = 42;
Will this be fixed properly?
for(int i = 0; i < 42; ++i)
gsl::at(*my_array, i) = 42;
If the braces in `(*my_array)` would be moved into the function that is ok as well.
This is just of general interest and should not block this patch in my opinion, as its a different bug.
Repository:
rCTE Clang Tools Extra
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D56661/new/
https://reviews.llvm.org/D56661
More information about the cfe-commits
mailing list