Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enforce little endian encoding #437

Merged
merged 12 commits into from
Jun 24, 2024

Conversation

jean-roland
Copy link
Contributor

@jean-roland jean-roland commented Jun 21, 2024

Until know, zenoh-pico assumed the system was little endian which will cause issues if this assumption is wrong.

This PR adds a layer to take into account system endianness and enforce little endian encoding which is the Zenoh encoding. It will solve #420.

  • Add an endianness module.
  • Use this module on the api/slice side.
  • Use the module on the protocole / wire encoding side.

@jean-roland jean-roland marked this pull request as ready for review June 24, 2024 09:48
@milyin milyin merged commit d67ea84 into eclipse-zenoh:dev/1.0.0 Jun 24, 2024
50 checks passed
@jean-roland jean-roland deleted the ft_endianness branch July 3, 2024 08:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants