Open pbarone-latai opened 2 months ago
@pbarone-latai any updates on your side?
The issue is that there are quite a few places which now potentially need a IOX_DISCARD_RESULT
or an actual check for the return value
Yeah, unfortunately no update on my end. It was a bit more involved than I initially thought. I don't anticipate having time to do this for a little while, though I do really think it would be worthwhile, both for consumers as well as for iceoryx in general. I'm fine closing the ticket if you don't want to leave the open ticket around
@pbarone-latai no problem. Keep your time. It is not necessary to close the PR if you intend to work on it later :)
Notes for Reviewer
This PR adds
[[nodiscard]]
on any iox container APIs which may fail and returnfalse
if a container is at capacityPre-Review Checklist for the PR Author
iox-123-this-is-a-branch
)iox-#123 commit text
)task-list-completed
)Checklist for the PR Reviewer
iceoryx_hoofs
have been added to./clang-tidy-diff-scans.txt
Post-review Checklist for the PR Author
References