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

Add const to some functions. #505

Merged
merged 1 commit into from
Feb 11, 2024
Merged

Add const to some functions. #505

merged 1 commit into from
Feb 11, 2024

Conversation

poweron0102
Copy link
Contributor

It just make some ::new() function to be const.

allowing it to be call on const definitions like this:

// Old
const SIZE: TilemapSize = TilemapSize{ x: 256, y: 256 };

// New
const SIZE: TilemapSize = TilemapSize::new(256, 256);

And some compile time optimizations.

@StarArawn
Copy link
Owner

I fixed the clippy issue elsewhere. I'll merge this. 👍

@StarArawn StarArawn merged commit 86549e9 into StarArawn:main Feb 11, 2024
7 of 8 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