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

Respect env RUBY_DBUS_ENDIANNESS=B (or =l) for outgoing messages. #130

Merged
merged 3 commits into from
Apr 4, 2023

Conversation

mvidner
Copy link
Owner

@mvidner mvidner commented Apr 4, 2023

also: Message#marshall speedup: don't marshall the body twice.

@coveralls
Copy link

coveralls commented Apr 4, 2023

Coverage Status

Coverage: 95.324% (+0.1%) from 95.177% when pulling ab88b12 on other-endianness into d7f6bf2 on master.

Since v0.2.1 there was a first marshalling to find out the body size for
the header and then we threw the result away and redid it. Reuse the result.
A global switch of the endianness of the sent messages.

This is probably useful only to tease peer implementations: dbus-daemon
will forward our messages to them with the original endianness.
@mvidner mvidner merged commit dae55db into master Apr 4, 2023
@mvidner mvidner deleted the other-endianness branch April 4, 2023 14:45
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.

2 participants