llvm/llvm-project

add newline between include and namespace

Open

#42.112 aperta il 25 lug 2019

Vedi su GitHub
 (6 commenti) (1 reazione) (1 assegnatario)C++ (10.782 fork)batch import
bugzillaclang-formatenhancementgood first issue

Metriche repository

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

Descrizione
Bugzilla Link 42767
Version unspecified
OS Windows NT

Extended Description

We have options to remove duplicate white-space lines, however, it would also be good for readability to be able to require at least one empty line:

  • before and after and #include block
  • before and after a namespace
//
// License
//
#include <memory>
#include <string>
namespace {

}

convert to:

//
// License
//
                            <<-- added empty line
#include <memory>
#include <string>
                            <<-- added empty line
namespace Bar {

class Foo {

};

} // Bar

Guida contributor

Direzione di ricerca
Esamina come clang format gestisce le righe vuote tra le direttive include e i blocchi namespace. Esamina le opzioni esistenti come 'MaxEmptyLinesToKeep' e 'SortIncludes' per capire come introdurre nuove opzioni di configurazione che impongano almeno una riga vuota prima e dopo i blocchi include e namespace.
Tech stack
cpp
Dominio
tooling
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
4
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
3-5 giorni
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++GitLLVM development environment
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
40