llvm/llvm-project

We should probably warn at how 'static inline' variables are odd at namespace scope.

Open

#48.977 aperta il 18 mar 2021

Vedi su GitHub
 (10 commenti) (0 reazioni) (1 assegnatario)C++ (10.782 fork)batch import
bugzillac++17clang:diagnosticsconfirmedgood first issue

Metriche repository

Star
 (26.378 star)
Metriche merge PR
 (Merge medio 1g 2h) (1000 PR mergiate in 30 g)

Descrizione
Bugzilla Link 49633
Version unspecified
OS All
CC @serberoth,@zygoloid,@TH3CHARLie

Extended Description

" static inline int A; "

seems a little silly, as 'inline' means 'make this definition mergeable', and static means 'this is a unique definition'. It seems that some sort of warning that questions why someone would do this is valuable.

Guida contributor

Direzione di ricerca
Cerca diagnostiche esistenti simili in Clang, come gli avvisi per funzioni static inline. Esamina i file Sema o DiagnoseDeclSpec per aggiungere un avviso per variabili static inline nell'ambito del namespace.
Tech stack
cpp
Dominio
compilers
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C++LLVM development
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60