microsoft / microsoft/vscode-cpptools

Add missing include should prioritize the header with the class definition over headers with only a forward declaration (VS:1906590)

Aperta
#11,703 0 commenti 1 reazione 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

bug Feature: Add '#include' Language Service
Lingua principale
TypeScript
Stelle
6.2k
Fork
1.7k
Merge medio
14h 46m
PR unite (30g)
61

Descrizione

Set up a project thusly:

Header.h
class Foo; // just forward declaration

Header2.h
class Foo {}; // the definition

Source.cpp
Foo foo;

Actual:
In the current product, Foo in cpp file should squiggle with "undefined" error and hovering over it offers both headers as suggestions.

Expected:
The suggested includes should be only Header2.h (to bring the full definition)

This should be addressed in VS and VS Code

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 caso con Header.h, Header2.h e Source.cpp, dove Header.h dichiara solo in avanti Foo e Header2.h lo definisce. Verifica i suggerimenti per gli include prodotti per Source.cpp; il lavoro è completato quando viene suggerito solo Header2.h, in modo coerente, in VS e VS Code.

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

Valutazione

Stack tecnologico
cpp
Ambito
devtools
Tipo di issue
Bug
Difficoltà
4/5
Tempo stimato
3-5 giorni
Stato di attività
Ferma
Chiarezza
Abbastanza chiara
Idoneità per principianti
42/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.