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

Rework ConvertEndian method to support fixed buffers #55

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Taezen
Copy link
Contributor

@Taezen Taezen commented Oct 3, 2022

This changes the following:

  • Converts endianness of fixed buffers properly.
  • Gets the right size for managed primitive types (especially bool and char)
  • Gets the managed field offset from the CoreCLR FieldDesc class through the FieldHandle

The changes are necessary, because the MemoryMarshaller uses the managed sizes and offsets.

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.

1 participant