openPMD/openPMD-api

Namespace Per Dir

Aperta

#82 aperta il 19 feb 2018

Vedi su GitHubGuida contributor
 (0 commenti) (0 reazioni) (0 assegnatari)C++ (57 fork)auto 404
api: breakinggood first issuehelp wanted

Metriche repository

Star
 (160 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

Follow-up to #81.

Each directory inside include/openPMD/ should define everything in a namespace with the same name as the dir.

  • backend/ (openPMD::backend)
  • IO/ (openPMD::IO)
    • ADIOS/ (openPMD::IO::ADIOS)
    • HDF5/ (openPMD::IO::HDF5)

Guida contributor

Direzione di ricerca
Ispeziona la struttura delle directory sotto include/openPMD/, quindi aggiungi dichiarazioni di namespace corrispondenti al nome di ciascuna cartella a tutti i file header/sorgente e aggiorna il codice che fa riferimento a queste classi/funzioni per utilizzare i nuovi namespace.
Tech stack
cpp
Dominio
backend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
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++Git
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75