clang-tidy fix for cppcoreguidelines-pro-bounds-constant-array-index generates invalid code

Offen

#9,237 erstellt am 25.4.2022 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit: 4/5
Geschätzter Aufwand: 3-5 Tage
Anfängerfreundlichkeit: 35/100

Issue-Typ: Bug
Klarheit: Größtenteils klar
Aktivitätsstatus: Veraltet
Tech-Stack: cpp
Bereich: tooling

Rechercherichtung

Beginne damit, die clang-tidy-Konfiguration und das C++-Beispiel aus dem Issue zu reproduzieren, und vergleiche dann den generierten Fix mit den LLVM-Issues 37858 und 55088. Verfolge den Code-Analysis-Fix-Pfad in vscode-cpptools; abgeschlossen bedeutet, dass die gemeldeten Array-Index-Fälle keinen ungültigen Code mehr generieren, einschließlich der Variante mit einem vorangestellten #if/endif.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

bug external Feature: Code Analysis

With 1.10.1 (currently unreleased), using "C_Cpp.codeAnalysis.clangTidy.config": "{ Checks: 'cppcoreguidelines-pro-bounds-constant-array-index', CheckOptions: [{key: cppcoreguidelines-pro-bounds-constant-array-index.GslHeader, value: '<gsl/gsl.h>'}]}"

with code

#include <array>

int func(int i)
{
    std::array<int, 2> vvv;
    return vvv[i];
}

generates the incorrect code return gsl::at(v, i);.

It's already reported at https://github.com/llvm/llvm-project/issues/37858 .

Bug#2: There's another related bug at https://github.com/llvm/llvm-project/issues/55088 , when there's an #if/endif at the top of the file.

Vorherrschende Sprache: TypeScript
Sterne: 6.2k
Forks: 1.7k
Ø Merge: 14 Std. 46 Min.
Gemergte PRs (30 T.): 61

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
Forken Sie das Repository und arbeiten Sie in einem Branch.
Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus microsoft/vscode-cpptools

Extension Signature Verification Failed: C/C++ Extension Pack Offen

Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 35/100

microsoft/vscode-cpptools#14787 · Erstellt am 18.9.2026
[Bug] cpptools fails to start on Ubuntu ARM64 due to missing execute permissions on shared libraries Offen

bug fixed Language Service regression

microsoft/vscode-cpptools#14779 · 1 zugewiesene Person · Erstellt am 17.9.2026
[WSL2] C/C++ IntelliSense triggers ~30s wsl.exe stalls and Remote-WSL disconnects Offen

Language Service more info needed

microsoft/vscode-cpptools#14778 · 1 Kommentar · 1 zugewiesene Person · Erstellt am 17.9.2026
WebAssembly intrinsics aren't detected correctly in a freestanding `--target=wasm32` Offen

Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 64/100

microsoft/vscode-cpptools#14769 · Erstellt am 13.9.2026
When using `"cStandard": "c17"`, errors for calls to functions with missing prototypes are not reported Offen

Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 68/100

microsoft/vscode-cpptools#14768 · 1 Kommentar · Erstellt am 13.9.2026

Alle Issues in microsoft/vscode-cpptools