Skip to content

Conversation

@BsAtHome
Copy link
Contributor

This fixes #3204.

No formats or sizes change. Just the way the value of the enum is addressed. The enum was already defined as an enum class which has int as the default underlying type. This has not changed but the underlying type is made explicit because the union absolutely needs both fields as ints to match size (and alignment). See also added code comments.

@andypugh
Copy link
Collaborator

andypugh commented Jan 6, 2025

Thanks for your work on this. (and everything else)

@andypugh andypugh merged commit c903ccd into LinuxCNC:master Jan 6, 2025
10 checks passed
@BsAtHome BsAtHome deleted the fix_typepunned-deref branch January 8, 2025 10:14
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.

Type punned dereference in libnml

2 participants