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

[Suggestion] XGameRuntimeFeature total length enum field #44

Closed
apache-hb opened this issue Oct 26, 2023 · 1 comment
Closed

[Suggestion] XGameRuntimeFeature total length enum field #44

apache-hb opened this issue Oct 26, 2023 · 1 comment

Comments

@apache-hb
Copy link

XGameRuntimeFeature currently has no enum case to indicate the number of fields, this requires hardcoding a 22 into an array length which is suboptimal when updating if new features are added to this enum.

@bbowman
Copy link
Member

bbowman commented May 5, 2024

Hey @apache-hb sorry for the long time to respond to this issue but I'm curious on your use case for this.

The intended use case for XGameRuntimeFeature is that a consumer would call xgameruntimeisfeatureavailable to check that a specific feature is available for use at runtime. Adding an enum value for max or count as you appear to be suggesting would seem to imply dynamically querying for all available features which may actually be more error prone if a feature is added that the original code was not ready to handle.

I'm going to close the issue for now as I don't believe we plan to change this. Thank you for the suggestion though and feel free to provide more context on how this would be useful for you if you believe we should reconsider.

@bbowman bbowman closed this as completed May 5, 2024
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

2 participants