Skip to content

Remove deprecated code before release 1.16 - #6472

Open
theoniko wants to merge 1 commit into
PointCloudLibrary:masterfrom
theoniko:remove-deprecated-code-before-release1.16
Open

Remove deprecated code before release 1.16#6472
theoniko wants to merge 1 commit into
PointCloudLibrary:masterfrom
theoniko:remove-deprecated-code-before-release1.16

Conversation

@theoniko

Copy link
Copy Markdown
Contributor

Fixes #6470

@larshg

larshg commented Aug 30, 2026

Copy link
Copy Markdown
Contributor

You need tp remove the boost files from where they are used in cmake also.
Ie.

"include/pcl/${SUBSYS_NAME}/boost.h"

@theoniko
theoniko force-pushed the remove-deprecated-code-before-release1.16 branch 2 times, most recently from 1d57cd4 to 726827d Compare August 30, 2026 19:09
@theoniko

Copy link
Copy Markdown
Contributor Author

You need tp remove the boost files from where they are used in cmake also. Ie.

"include/pcl/${SUBSYS_NAME}/boost.h"

Done

@@ -1,184 +0,0 @@
/*

@larshg larshg Aug 30, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This file shouldn't be removed.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

* returned (true) or inside (false).
* \param limit_negative the limit_negative flag
*/
PCL_DEPRECATED(1, 16, "use bool getFilterLimitsNegative() instead")

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Only this

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

@@ -1,202 +0,0 @@
/*

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This file shouldn't be removed

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

@theoniko
theoniko force-pushed the remove-deprecated-code-before-release1.16 branch from 726827d to 719011f Compare August 31, 2026 04:40

@mvieth mvieth left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! The failing macOS 15 check is unrelated and can be ignored.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove everything marked deprecated before the PCL 1.16 release

3 participants