Fast-DDS
Fast-DDS copied to clipboard
Published
20 hours ago •
eProsima
eProsima
[20568] Remove all redundant headers from include/fastrtps
Description
This PR removes all redundant headers from include/fastrtps. It makes UnitsParser private It depends on:
- #4545
Discovery Server related PR:
- eProsima/Discovery-Server#73
Contributor Checklist
- [x] Commit messages follow the project guidelines.
- [x] The code follows the style guidelines of this project.
- N/A: Tests that thoroughly check the new feature have been added/Regression tests checking the bug and its fix have been added; the added tests pass locally
- N/A: Any new/modified methods have been properly documented using Doxygen.
- ❌ Changes are ABI compatible.
- ❌ Changes are API compatible.
- [x] New feature has been added to the
versions.mdfile (if applicable). - [x] New feature has been documented/Current behavior is correctly described in the documentation. Related documentation PR: eProsima/Fast-DDS-docs#737
- N/A: Applicable backports have been included in the description.
Reviewer Checklist
- [x] The PR has a milestone assigned.
- [x] The title and description correctly express the PR's purpose.
- [x] Check contributor checklist is correct.
- [ ] Check CI results: changes do not issue any warning.
- [ ] Check CI results: failing tests are unrelated with the changes.
We should create a Shapes-Demo branch with these changes, in particular the build failed because it couldn't find fastrtps/rtps/common/Guid.h