PointCloudLibrary/pcl

Ensure no `*.hpp` files are needed to be included by the user, only the `*.h` should be required

Open

#3245 aperta il 18 lug 2019

Vedi su GitHub
 (5 commenti) (0 reazioni) (0 assegnatari)C++ (4506 fork)batch import
good first issuekind: todo

Metriche repository

Star
 (9023 star)
Metriche merge PR
 (Merge medio 8g 7h) (3 PR mergiate in 30 g)

Descrizione

Your Environment

  • Operating System and version: Win10
  • Compiler:
  • PCL Version: 1.9.1

Context

In general, we can find PCL .hpp are included at end of header file

#ifdef PCL_NO_PRECOMPILE
#include <pcl/....hpp>
#endif

But why some .hpp are not. For example: pcl/surface/impl/grid_projection.hpp: it is included in .cpp pcl/outofcore/impl/octree_base.hpp: it need to be included by user self

Expected Behavior

Current Behavior

Code to Reproduce

Possible Solution

Guida contributor

Direzione di ricerca
Identifica tutti i file .hpp nel sorgente PCL che non sono inclusi nel corrispondente file .h con la guardia #ifdef PCL NO PRECOMPILE, e aggiungi la riga di inclusione. Ad esempio, pcl/surface/impl/grid projection.hpp è incluso solo in .cpp e pcl/outofcore/impl/octree base.hpp richiede l'inclusione manuale da parte dell'utente. Assicurati che tali file siano inclusi nel loro counterpart .h.
Tech stack
cpp
Dominio
build system
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.
60