diff options
| author | Carlos Galvez <carlosgalvezp@gmail.com> | 2022-01-21 07:59:27 +0000 |
|---|---|---|
| committer | Carlos Galvez <carlosgalvezp@gmail.com> | 2022-01-23 15:52:42 +0000 |
| commit | eb3f20e8fa4b76e0103f15623a2fc3b27fb03f85 (patch) | |
| tree | 6e707ec23f33f61f4194ad1eab7de19463dc709a /lldb/source/Commands/CommandObjectThread.cpp | |
| parent | d4ed3eff9f9c4d00689e34712db8ac0ca65ddb26 (diff) | |
| download | llvm-eb3f20e8fa4b76e0103f15623a2fc3b27fb03f85.zip llvm-eb3f20e8fa4b76e0103f15623a2fc3b27fb03f85.tar.gz llvm-eb3f20e8fa4b76e0103f15623a2fc3b27fb03f85.tar.bz2 | |
[clang-tidy] Remove gsl::at suggestion from cppcoreguidelines-pro-bounds-constant-array-index
Currently the fix hint is hardcoded to gsl::at(). This poses
a problem for people who, for a number of reasons, don't want
or cannot use the GSL library (introducing a new third-party
dependency into a project is not a minor task).
In these situations, the fix hint does more harm than good
as it creates confusion as to what the fix should be. People
can even misinterpret the fix "gsl::at" as e.g. "std::array::at",
which can lead to even more trouble (e.g. when having guidelines
that disallow exceptions).
Furthermore, this is not a requirement from the C++ Core Guidelines.
simply that array indexing needs to be safe. Each project should
be able to decide upon a strategy for safe indexing.
The fix-it is kept for people who want to use the GSL library.
Differential Revision: https://reviews.llvm.org/D117857
Diffstat (limited to 'lldb/source/Commands/CommandObjectThread.cpp')
0 files changed, 0 insertions, 0 deletions