-
Notifications
You must be signed in to change notification settings - Fork 6
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
Dynamic Groups - Applied optimisations based HEMTT suggestions #212
base: master
Are you sure you want to change the base?
Conversation
@OverlordZorn man... this source file
Not fun to check the changes. And those aren't really changing the functionality, right? |
Does this effort make sense, considering HEMTT's optimizer will do most of these tweaks by itself anyway? |
I have Dynamic Groups integrated into our unit mod. everytime i build and test something, hemtt threw these "warnings" at me at how much i can improve it. I added these changes on my side, so hemtt shuts up. Since i did the work already, i thought "might as well provide it to the OG repo" |
i run these changes on our units mod and there have been no issues being noted on our side. |
Co-authored-by: Salbei <[email protected]>
Co-authored-by: Salbei <[email protected]>
No description provided.