PointCloudLibrary/pcl

In preparation for `index_t`, the cast needs to be removed

Open

#4.212 aberto em 23 de jun. de 2020

Ver no GitHub
 (0 comments) (0 reactions) (0 assignees)C++ (4.506 forks)batch import
effort: mediumgood first issuekind: todoneeds: pr mergepriority: gsoc

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

In preparation for index_t, the cast either needs to be removed or it needs to be set explicitly to index_t. I vote for removal unless it's unsupported.

Originally posted by @SergioRAgostinho in https://github.com/_render_node/MDI0OlB1bGxSZXF1ZXN0UmV2aWV3Q29tbWVudDQ0NDE1MTE1NQ==/comments/review_comment

Needs CI run on the PR resolving this issue post #4198 merge

Guia do colaborador

Direção de pesquisa
Examine o código onde o cast aparece e remova o, garantindo compatibilidade com index t.
Pilha de tecnologia
cpp
Domain
backend
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.
1
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Menos de 1 hora
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++
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70