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

assorted canonicalizations #720

Merged
merged 14 commits into from
Mar 3, 2024
Merged

assorted canonicalizations #720

merged 14 commits into from
Mar 3, 2024

Conversation

ab9rf
Copy link
Member

@ab9rf ab9rf commented Feb 10, 2024

this PR contains an assortment of changes to structures to either use bay12 names for types and fields, or note what the bay12 names are

this PR seeks to avoid any changes that would conflict with existing code; changes that require coordination will be proposed separately

Copy link
Contributor

github-actions bot commented Feb 15, 2024

The sizes of the following types have changed in this pull request. Please verify that this is intentional. Note that renamed types are not currently detected and will also be listed here.

Type Platform Previous size New size (this PR) Change
dungeon_contextst linux64 0 1656 +1656
dungeon_contextst windows64 0 1656 +1656
entity_position_profile_claimst linux64 0 4 +4
entity_position_profile_claimst windows64 0 4 +4
knowledge_profilest linux64 0 368 +368
knowledge_profilest windows64 0 368 +368
meeting_context linux64 32 0 -32
meeting_context windows64 32 0 -32
process_unit_aux linux64 1664 0 -1664
process_unit_aux windows64 1664 0 -1664

This is an automated comment. Please leave a reply if you think that the above information is incorrect.

@ab9rf ab9rf changed the title canonicalize entity position & entity position profile assorted canonicalizations Mar 1, 2024
@myk002 myk002 merged commit ca2668f into DFHack:master Mar 3, 2024
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants