Skip to content

Remove MaterialInfo consts#5857

Merged
ab9rf merged 1 commit into
DFHack:developfrom
quietust:matinfo
Jul 23, 2026
Merged

Remove MaterialInfo consts#5857
ab9rf merged 1 commit into
DFHack:developfrom
quietust:matinfo

Conversation

@quietust

Copy link
Copy Markdown
Member

in favor of using df::builtin_mats enum entries

Also adjust some comparisons to use more easily understandable values (e.g. <= CREATURE_200 instead of < HIST_FIG_1)

@quietust

Copy link
Copy Markdown
Member Author

I'm open to leaving some of those constants behind, such as NUM_BUILTIN, since some of them might not be immediately obvious what you should use instead. GROUP_SIZE is something that needs to be removed, though, since while the 3 groups are currently the same size, there's no guarantee that this will continue to be the case in the future.

in favor of using df::builtin_mats enum entries

Also adjust some comparisons to use more easily understandable values
(e.g. <= CREATURE_200 instead of < HIST_FIG_1)
@ab9rf
ab9rf merged commit 9ec079a into DFHack:develop Jul 23, 2026
14 checks passed
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