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

More configurable defines #6

Open
boozook opened this issue Jul 5, 2024 · 0 comments
Open

More configurable defines #6

boozook opened this issue Jul 5, 2024 · 0 comments

Comments

@boozook
Copy link

boozook commented Jul 5, 2024

Following #3,

May be could be better to make same for HE_COLLISION_PAIR_MAX, HE_GRID_RESULT_MAX and ENTITY_COUNT?

Also may be rename ENTITY_COUNT to ENTITY[_NUM]_MAX? 🤔

Thank you so much for awesome work!


Also I'm sure there is needed some validation for numeric defines. Something like compile-time assert that check that value is numeric and is power of two, or just even number if needed. (I've found some solutions: 1, 2, 3, 4)

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

No branches or pull requests

1 participant