Skip to content

streamline default derives #144

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Jun 20, 2023
Merged

streamline default derives #144

merged 2 commits into from
Jun 20, 2023

Conversation

phip1611
Copy link
Member

@phip1611 phip1611 commented Jun 20, 2023 •
edited
Loading

  • streamline default derives
  • removed packed where it is not needed (because the default C representation is already packed)

phip1611 added 2 commits June 20, 2023 15:45
@phip1611
tree-wide: streamline default derives and repr attributes
d54cddc
@phip1611
tree-wide: remove align(8) for tags
ab6bb32 
This breaks the sizeof() property for many tags that I verify. Furthermore,
the builder structs already take care of any alignments when they construct
the MBI/MB header.
@phip1611 phip1611 changed the title Dev3 streamline default derives Jun 20, 2023
@phip1611 phip1611 merged commit ef79b8b into main Jun 20, 2023
@phip1611 phip1611 deleted the dev3 branch June 20, 2023 13:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@phip1611