Skip to content

Conversation

@awillenbuecher-xq-tec
Copy link
Contributor

This PR makes some modifications to the ABI Compatible Data Types, as discussed during the Architecture Community F2F Workshop on 2025-11-05. Primarily, these are:

  • explicit support for UTF-8 encoded strings
  • more precise specification of required primitive types
  • remove ambiguities concerning memory layout and constraints on generics
  • minor rework of container types
  • clarify distinction between the concepts of type description and type structure metadata

@github-actions
Copy link

⚠️ Docs-as-Code version mismatch detected
Please check the CI build logs for details and align the documentation version with the Bazel dependency.

@github-actions
Copy link

The created documentation from the pull request is available at: docu-html

@github-actions
Copy link

This PR is stale because it has been open for 30 days with no activity. It will be closed in 10 days if no further activity occurs. #magic___^_^___line

@github-actions github-actions bot added the Stale label Dec 14, 2025
@awillenbuecher-xq-tec
Copy link
Contributor Author

@FScholPer @antonkri @johannes-esr @qor-lb Could either of you review and merge this PR? It would be unfortunate if these improvements got lost.

@github-actions github-actions bot removed the Stale label Dec 16, 2025
Strings
"""""""

Strings have the same memory layout as ``AbiVec<u8>``, but additionally guarantee that their content is valid UTF-8.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wondering how we're going to guarantee UTF-8 validity

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

UTF-8 validity of str is automatically guaranteed in Rust. In C++, custom wrapper types for UTF-8 strings would enforce validity, in the same way as the generated types for tagged unions would enforce valid memory patterns.

@4og 4og merged commit 78bc939 into eclipse-score:main Jan 21, 2026
6 checks passed
@github-project-automation github-project-automation bot moved this from In review to Done in Base Libs FT Jan 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants