PointCloudLibrary/pcl

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

Open

#3 245 ouverte le 18 juil. 2019

Voir sur GitHub
 (5 commentaires) (0 réactions) (0 assignés)C++ (4 506 forks)batch import
good first issuekind: todo

Métriques du dépôt

Stars
 (9 023 stars)
Métriques de merge PR
 (Merge moyen 8j 7h) (3 PRs mergées en 30 j)

Description

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

Guide contributeur

Direction de recherche
Identifiez tous les fichiers .hpp dans le source PCL qui ne sont pas inclus dans le fichier .h correspondant avec la garde #ifdef PCL NO PRECOMPILE, et ajoutez la ligne d'inclusion. Par exemple, pcl/surface/impl/grid projection.hpp n'est inclus que dans .cpp et pcl/outofcore/impl/octree base.hpp nécessite une inclusion manuelle par l'utilisateur. Assurez vous que ces fichiers sont inclus dans leur homologue .h.
Stack technique
cpp
Domaine
build system
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C++Git
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60