PointCloudLibrary/pcl

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

Open

#3.245 aberto em 18 de jul. de 2019

Ver no GitHub
 (5 comments) (0 reactions) (0 assignees)C++ (4.506 forks)batch import
good first issuekind: todo

Métricas do repositório

Stars
 (9.023 stars)
Métricas de merge de PR
 (Mesclagem média 8d 7h) (3 fundiu PRs em 30d)

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

Guia do colaborador

Direção de pesquisa
Identifique todos os arquivos .hpp no código fonte do PCL que não estão incluídos no arquivo .h correspondente com a guarda #ifdef PCL NO PRECOMPILE e adicione a linha de inclusão. Por exemplo, pcl/surface/impl/grid projection.hpp é incluído apenas em .cpp e pcl/outofcore/impl/octree base.hpp requer inclusão manual pelo usuário. Garanta que esses arquivos sejam incluídos em seu arquivo .h correspondente.
Pilha de tecnologia
cpp
Domain
build system
Tipo Issue
Refatoração
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Meio dia
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
C++Git
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60