eclipse-iceoryx / eclipse-iceoryx/iceoryx

Replace 'iox::string' with semantically appropriate types like `FilePath`

Open
#2,018 3 comments 0 reactions 1 assignee Claimed by @varuntumbe View on GitHub
enhancement good first issue technical debt
Dominant language
C++
Stars
2.2k
Forks
492
Avg merge
18h 57m
Merged PRs (30d)
1

Description

## Brief feature description

In many places an `iox::string` is used instead of a semantically more appropriate type like `FilePath`. In order to prevent superfluous checks and easy to make mistakes the semantically appropriate type shall be used.

See also https://github.com/eclipse-iceoryx/iceoryx/pull/2016#discussion_r1315024006

## Detailed information

Tasks

- [ ] Check the usage of `iox::string` in the API
- [ ] Replace `iox::string` with `FileName`, `UserName`, `GroupName`, `FilePath` and `Path`
- [ ] Use `FileName` for `RuntineName_t`, which is an `iox::string` alias
- [ ] Use `FileName` for `detail::PosixSharedMemory::Name_t`, which is an `iox::string` alias

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.