[#394] sample mut uninit

[elfenpiff]

Notes for Reviewer

• Fixes the problem that uninitialized samples can be sent by introducing a SampleMutUninit struct that does not offer a send method
• Renames the method send_sample into send since it was weird to read send_sample(std::move(sample)), this is much cleaner send(std::move(sample)).
• The introduction of the new class needed an adjustment on the C API level - here we always use uninitialized samples
  • SampleMut has no C binding, the C++ implementation is done directly via the C API, otherwise it would be much more code on the FFI layer

Pre-Review Checklist for the PR Author

1. Add sensible notes for the reviewer
2. PR title is short, expressive and meaningful
3. Relevant issues are linked in the References section
4. Every source code file has a copyright header with SPDX-License-Identifier: Apache-2.0 OR MIT
5. Branch follows the naming format (iox2-123-introduce-posix-ipc-example)
6. Commits messages are according to this guideline
   • Commit messages have the issue ID ([#123] Add posix ipc example)
   • Commit author matches Eclipse Contributor Agreement (and ECA is signed)
7. Tests follow the best practice for testing
8. Changelog updated in the unreleased section including API breaking changes
9. Assign PR to reviewer
10. All checks have passed (except task-list-completed)

Checklist for the PR Reviewer

• Commits are properly organized and messages are according to the guideline
• Unit tests have been written for new behavior
• Public API is documented
• PR title describes the changes

Post-review Checklist for the PR Author

1. All open points are addressed and tracked via issues

References

Closes #394

[orecham]

I guess this is based on #393

[elfenpiff]

Yes it is ... I forget to mention this.

But it was an accident

[orecham]

@elfenpiff Looks good. Ping me for approval after rebasing on main or merging #393.

[codecov]

Codecov Report

Attention: Patch coverage is 78.82353% with 18 lines in your changes missing coverage. Please review.

Project coverage is 79.00%. Comparing base (88d7507) to head (dbff454).
Report is 10 commits behind head on main.

Files with missing lines	Patch %	Lines
iceoryx2/src/sample_mut_uninit.rs	74.64%	18 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #395      +/-   ##
==========================================
- Coverage   79.01%   79.00%   -0.02%     
==========================================
  Files         194      194              
  Lines       22730    22730              
==========================================
- Hits        17961    17958       -3     
- Misses       4769     4772       +3     

Files with missing lines	Coverage Δ
iceoryx2/src/port/publisher.rs	84.99% <100.00%> (ø)
iceoryx2/src/sample_mut.rs	56.25% <ø> (ø)
iceoryx2/src/sample_mut_uninit.rs	74.64% <74.64%> (ø)

... and 5 files with indirect coverage changes

[elBoberido]

The request changes is just to make you aware of my comments. If you dismiss them, I'm fine with merging.

Well, obviously I did not find time to look at it, so no reason to block the PR.