[clang-tidy] Add check to replace operator[] with at() Enforce SL.con.3#95220

Merged

vbvictor merged 101 commits intollvm:mainfrom

paulhdk:prefer-at-over-subscript-operator

Aug 24, 2025

Commits on Aug 4, 2025

Enforce SL.con.3: Add check to replace operator[] with at() on std containers
paulhdk
committed
EugeneZelenko's comments

sebwolf-de
authored and
paulhdk
committed
Refactoring, but not finished yet

sebwolf-de
authored and
paulhdk
committed
Rename AvoidBoundsErrorsCheck to PreferAtOverSubscriptOperatorCheck

paulhdk
and
leunam99
committed
Add a user-customizable mechanism for excluding classes from the analysis

paulhdk
and
leunam99
committed
Update the documentation

paulhdk
and
leunam99
committed
Add remaining tests requested by @PiotrZSL

paulhdk
and
leunam99
committed
Remove unused and rename variables

paulhdk
and
leunam99
committed
Update types in PreferAtOverSubscriptOperatorCheck::check()

paulhdk
and
leunam99
committed
Fix length of underline in prefer-at-over-subscript-operator.rst

paulhdk
and
leunam99
committed
Enforce 80 char column width in prefer-at-over-subscript-operator.rst

paulhdk
and
leunam99
committed
Synchronise prefer-at-over-subscript-operator.rst with ReleaseNotes.rst

paulhdk
and
leunam99
committed
Move RUN lines to the top of prefer-at-over-subscript-operator.cpp

paulhdk
and
leunam99
committed
Add an empty lien after each `code-block` in prefer-at-over-subscript-operator.rst

paulhdk
and
leunam99
committed
Use ofClass() instead of hasParent() in PreferAtOverSubscriptOperatorCheck.cpp

paulhdk
and
leunam99
committed
Use matchesAnyListedName() in matcher instead of explicit std::find()

paulhdk
and
leunam99
committed
Match source range of expression in warnings emitted by PreferAtOverSubscriptOperatorCheck

paulhdk
and
leunam99
committed
PreferAtOverSubscriptOpterator ->

paulhdk
and
leunam99
committed
Remove extra comment

paulhdk
and
leunam99
committed
Mention bounds safety profile explicitly

paulhdk
and
leunam99
committed
Move paragraph about which rule/profile is being implemented above

paulhdk
and
leunam99
committed
Rename variables related to class exclusion mechanism

paulhdk
and
leunam99
committed
Remove redundant nullptr cast

paulhdk
and
leunam99
committed
Refactor the check's matcher as per 5chmidti's suggestion

paulhdk
and
leunam99
committed
Check that alternative is accessible in findAlternative() as per @5chmidti's and @PiotrZSL's suggestions

paulhdk
and
leunam99
committed
Use upticks when diagnostic messages refer to code

paulhdk
and
leunam99
committed
Make findAlternative() static

paulhdk
and
leunam99
committed
Move MatchedExpr definition closer to where it's first used in

paulhdk
and
leunam99
committed
Synchronise comment in header file with ReleaseNotes and check

paulhdk
and
leunam99
committed
Reword reasoning re: class exclusion in pro-bounds-avoid-unchecked-container-accesses.rst

paulhdk
and
leunam99
committed
Replace array with container inpro-bounds-avoid-unchecked-container-accesses.rst

paulhdk
and
leunam99
committed
Update unique_ptr example in pro-bounds-avoid-unchecked-container-accesses.rst

paulhdk
and
leunam99
committed
Remove outdated comment in ProBoundsAvoidUncheckedContainerAccess.cpp

paulhdk
and
leunam99
committed
Use TK_IgnoreUnlessSpelledInSource as traversal kind

paulhdk
and
leunam99
committed
Revert "Use TK_IgnoreUnlessSpelledInSource as traversal kind"

paulhdk
and
leunam99
committed
Add template test with std::map as an argument

paulhdk
and
leunam99
committed
Use getNameInfo().getSourceRange() instead of getSourceRange() when

paulhdk
and
leunam99
committed
Replace use of Method->getNameInfo().getAsString() with Method->getName()

paulhdk
and
leunam99
committed
Add TODO comment re: findAlternative()'s capabilities in class

paulhdk
and
leunam99
committed
Reduce size of SourceRange when printing warnings

paulhdk
and
leunam99
committed
Add customizable fixit suggestions

paulhdk
and
leunam99
committed
Document customizable fixit suggestions

paulhdk
and
leunam99
committed
Test customizable fixit suggestions

paulhdk
and
leunam99
committed
Update one-sentence description of `cppcoreguidelines-pro-bounds-avoid-unchecked-container-accesses` check

paulhdk
and
leunam99
committed
Format description of the `cppcoreguidelines-pro-bounds-avoid-unchecked-container-accesses` in ReleaseNotes.rst

paulhdk
and
leunam99
committed
Generalise description of excluded classes in `pro-bounds-avoid-unchecked-container-accesses.rst`

paulhdk
and
leunam99
committed
s/bound-safe/bounds-safe

paulhdk
and
leunam99
committed
docs: remove whitespace

paulhdk
and
leunam99
committed
Update clang-tools-extra/docs/ReleaseNotes.rst

paulhdk
and
EugeneZelenko
committed
Update clang-tools-extra/docs/clang-tidy/checks/cppcoreguidelines/pro-bounds-avoid-unchecked-container-accesses.rst

paulhdk
and
EugeneZelenko
committed
Update clang-tools-extra/docs/clang-tidy/checks/cppcoreguidelines/pro-bounds-avoid-unchecked-container-accesses.rst

paulhdk
and
EugeneZelenko
committed
Update clang-tools-extra/docs/clang-tidy/checks/cppcoreguidelines/pro-bounds-avoid-unchecked-container-accesses.rst

paulhdk
and
EugeneZelenko
committed
SubscriptExcludeClasses -> ExcludeClasses

paulhdk
and
leunam99
committed
SubscriptFixMode -> FixMode

paulhdk
and
leunam99
committed
SubscriptFixFunction -> FixFunction

paulhdk
and
leunam99
committed
Remove "Subscript" prefix for variable names

paulhdk
and
leunam99
committed
Update clang-tools-extra/docs/clang-tidy/checks/cppcoreguidelines/pro-bounds-avoid-unchecked-container-accesses.rst

paulhdk
and
EugeneZelenko
committed
Update clang-tools-extra/docs/clang-tidy/checks/cppcoreguidelines/pro-bounds-avoid-unchecked-container-accesses.rst

paulhdk
and
EugeneZelenko
committed
Update clang-tools-extra/clang-tidy/cppcoreguidelines/ProBoundsAvoidUncheckedContainerAccesses.cpp

paulhdk
and
EugeneZelenko
committed
Remove contradicting "but" in cppcoreguidelines/pro-bounds-avoid-unchecked-container-accesses.rst

paulhdk
and
leunam99
committed
Only match subscript operators with a parameter list of size 0 or 1

paulhdk
and
leunam99
committed
Simplify how user-specified excluded classes are stored

paulhdk
and
leunam99
committed
Remove unused import

paulhdk
and
leunam99
committed
Make FixMode "none" lowercase

paulhdk
and
leunam99
committed
Separate C++23 tests

paulhdk
and
leunam99
committed
Change semantics around excluded classes

paulhdk
and
leunam99
committed
Add tests for new "ExcludedClasses" semantics and split tests into

paulhdk
and
leunam99
committed
Update clang-tools-extra/clang-tidy/cppcoreguidelines/ProBoundsAvoidUncheckedContainerAccesses.cpp

paulhdk
and
vbvictor
committed
Update clang-tools-extra/clang-tidy/cppcoreguidelines/ProBoundsAvoidUncheckedContainerAccesses.cpp

paulhdk
and
vbvictor
committed
Update clang-tools-extra/clang-tidy/cppcoreguidelines/ProBoundsAvoidUncheckedContainerAccesses.cpp

paulhdk
and
vbvictor
committed
Update clang-tools-extra/docs/clang-tidy/checks/cppcoreguidelines/pro-bounds-avoid-unchecked-container-accesses.rst

paulhdk
and
vbvictor
committed
s/flags/finds in docs

paulhdk
and
leunam99
committed
Mention default option at end of description in docs

paulhdk
and
leunam99
committed
Refactor `ExcludedClassesStr`

paulhdk
and
leunam99
committed
Formatting

paulhdk
and
leunam99
committed
Fix tests
paulhdk
committed
Initialise ProBoundsAvoidUncheckedContainerAccesses members in
paulhdk
committed
docs: outline examples of safe alterantives to subscript operator
paulhdk
committed
Reorder `Option.store`s and remove empty line
paulhdk
committed
Add FixFunctionEmptyArgs option
paulhdk
committed
Update clang-tools-extra/clang-tidy/cppcoreguidelines/ProBoundsAvoidUncheckedContainerAccesses.cpp

paulhdk
and
EugeneZelenko
committed
Update clang-tools-extra/clang-tidy/cppcoreguidelines/ProBoundsAvoidUncheckedContainerAccesses.cpp

paulhdk
and
EugeneZelenko
committed
Format
paulhdk
committed
Use `%` formatting for diagnostics
paulhdk
committed
docs: point out default values of options at the end of their
paulhdk
committed
Fix format strings
paulhdk
committed
Excplitly check for `FixFunctionEmptyArgs` option being empty
paulhdk
committed
Use CHECK-MESSAGES-NOT in `FixFunctionEmptyArgs` tests
paulhdk
committed
Print warning when FixFunctionEmptyArgs is explicitly set to "", but
paulhdk
committed
docs: reword paragraph about the STL containers which are excluded by default
paulhdk
committed
docs: use :option:... syntax in FixFunction documentation
paulhdk
committed
docs: fix typo
paulhdk
committed
s/accesses/access in check name
paulhdk
committed
Update clang-tools-extra/clang-tidy/cppcoreguidelines/ProBoundsAvoidUncheckedContainerAccess.cpp

paulhdk
and
vbvictor
committed
Fix header comment alignment
paulhdk
committed
docs: reword paragraph about excluded STL containers
paulhdk
committed
Update ProBoundsAvoidUncheckedContainerAccess.h

paulhdk
and
EugeneZelenko
committed
docs: add missing backticks
paulhdk
committed
Update clang-tools-extra/docs/clang-tidy/checks/cppcoreguidelines/pro-bounds-avoid-unchecked-container-access.rst

paulhdk
and
EugeneZelenko
authored
Update clang-tools-extra/docs/clang-tidy/checks/cppcoreguidelines/pro-bounds-avoid-unchecked-container-access.rst

paulhdk
and
vbvictor
authored

Commits on Aug 8, 2025

Update clang-tools-extra/docs/clang-tidy/checks/cppcoreguidelines/pro-bounds-avoid-unchecked-container-access.rst

paulhdk
and
EugeneZelenko
authored

[clang-tidy] Add check to replace operator[] with at() Enforce SL.con.3#95220vbvictor merged 101 commits intollvm:mainllvm/llvm-project:mainfrom paulhdk:prefer-at-over-subscript-operatorpaulhdk/llvm-project:prefer-at-over-subscript-operatorCopy head branch name to clipboard

Commits

Commits on Aug 4, 2025

Commits on Aug 8, 2025

[clang-tidy] Add check to replace operator[] with at() Enforce SL.con.3#95220
vbvictor merged 101 commits intollvm:mainfrom
paulhdk:prefer-at-over-subscript-operator