extra variable?
Closed this issue · 1 comments
plakapenka commented
Expected and current behavior
I think in CModelInfo it is worth removing NUM_MODEL_INFOS and using TOTAL_DFF_MODEL_IDS (it is possible to rename) instead to be able to control the number of models in one place for the future.
I'm too far from code ideology to make this decision myself
Additional information
No response
Steps to reproduce
....
Screenshots/Videos
No response
Pirulax commented
Hey,
Thanks for taking the time to report the issue.
constexpr static auto NUM_MODEL_INFOS = TOTAL_DFF_MODEL_IDS
and the "problem" is solved.
I'd keep the extra isolation between the 2.