eclipse-iceoryx / eclipse-iceoryx/iceoryx

Remove nullability from all iceoryx constructs

Open
#1,654 0 comments 0 reactions 0 assignees View on GitHub
technical debt
Dominant language
C++
Stars
2.2k
Forks
492
Avg merge
18h 57m
Merged PRs (30d)
1

Description

## Brief feature description

To reduce the likely-hood of bugs we should remove the null ability from constructs like:
- [ ] SmartChunk
- [ ] Sample
- [ ] Request
- [ ] Response
- [ ] SharedChunk

Furthermore, we should explicitly allow the relative and relocatable pointer to be nullable. Those are such low level constructs and on those layers `nullptr` maybe transports some information as well which would lead to unexpected surprises when raw pointer are suddenly no longer nullable.

Contributor guide

Open the contributing guide

Research direction

Start by locating the definitions and tests for SmartChunk, Sample, Request, Response, SharedChunk, and the relative and relocatable pointers. Determine how nullability is currently represented, then verify that the listed constructs reject null while the pointer types explicitly continue to permit it.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
distributed-systems
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.