PointCloudLibrary/pcl

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

Open

#4212 aperta il 23 giu 2020

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)C++ (4506 fork)batch import
effort: mediumgood first issuekind: todoneeds: pr mergepriority: gsoc

Metriche repository

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

Descrizione

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

Guida contributor

Direzione di ricerca
Esamina il codice in cui appare il cast e rimuovilo, assicurando la compatibilità con index t.
Tech stack
cpp
Dominio
backend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
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++
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70