microsoft / microsoft/vscode-cpptools

Linter provides no warnings for invalid C code

Aperta
#828 20 commenti 35 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Feature Request Language Service postponed
Lingua principale
TypeScript
Stelle
6.2k
Fork
1.7k
Merge medio
14h 46m
PR unite (30g)
61

Descrizione

See title, create a fresh test.c file with the contents:

#include <stdio.h>

int main()
{
    asdfd;
    return 0;
}

(If) Linting is working properly, asdfd will be underlined with squigglies as you would expect. What you would not expect is that turning it into a function call makes the squigglies go away:

#include <stdio.h>

int main()
{
    asdfd();
    return 0;
}

Furthermore, hovering over it produces the popup int asdfd(), as if it was defined to return int somewhere. This is just silly.

Related to #746

Guida per i contributori

Apri la guida per i contributori

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Direzione di ricerca

Riproduci il comportamento in un test.c nuovo usando i due snippet, confrontando l’identificatore non valido autonomo con la forma di chiamata di funzione. Leggi l’issue correlata #746 e traccia il comportamento del linting e dell’hover C/C++ a partire da questa riproduzione. È completato quando entrambe le forme non valide vengono diagnosticate correttamente e la chiamata non riceve più un int asdfd() inventato nell’hover.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
c
Ambito
devtools
Tipo di issue
Bug
Difficoltà
3/5
Tempo stimato
1-2 giorni
Stato di attività
Tranquilla
Chiarezza
Abbastanza chiara
Idoneità per principianti
48/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.