PointCloudLibrary/pcl

Review casts in PCL

Open

#4,230 opened on 2020年6月26日

GitHub で見る
 (5 comments) (0 reactions) (0 assignees)C++ (4,506 forks)batch import
effort: higheffort: lowgood first issuehelp wantedkind: todo

Repository metrics

Stars
 (9,023 stars)
PR merge metrics
 (平均マージ 8d 7h) (30d で 3 merged PRs)

説明

Context

With the migration to index_t, PCL will soon have some unneeded casts which should be removed

Describe the solution you'd like

Each cast needs to be reviewed and removed if it's unneeded. A non-exhaustive list of such situations:

  • assigning index_t to width
  • assigning size_t to Indices

Additional context

This will require significant help from others since the scope is wide, but the issue is not hard at all. To get an idea of the scale, please check the occurrence of static_cast via:

grep '\s*static_cast' * -nrH | grep -v '<float>' | grep -v 'cast<\S*\*\s*>' | wc -l

Note: this doesn't count C style casts

(High effort for total change, low for effort needed for piece-meal contributions)

コントリビューターガイド