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

Abierto
#9,237 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
4/5
Tiempo estimado
3-5 días
Aptitud para principiantes
35/100
Tipo de issue
Error
Claridad
Bastante claro
Estado de actividad
Estancado
Stack tecnológico
cpp
Área
tooling

Línea de trabajo

Empieza reproduciendo la configuración de clang-tidy y el ejemplo de C++ del issue, y después compara el fix generado con los issues 37858 y 55088 de LLVM. Sigue la ruta del fix de análisis de código en vscode-cpptools; hecho significa que los casos de índices de arrays reportados ya no generan código no válido, incluida la variante con un #if/endif inicial.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

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.

Lenguaje dominante
TypeScript
Estrellas
6.2k
Forks
1.7k
Merge medio
14 h 46 min
PR fusionados (30 d)
61

Guía de contribución

Abrir la guía de contribución

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de microsoft/vscode-cpptools

Todos los issues de microsoft/vscode-cpptools

Issues similares

Más issues de TypeScript

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.