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
Performance Enhancements #1229
Performance Enhancements #1229
Conversation
puu.sh seems to be having troubles right now, and I'd love to see the performance graphs! :) |
Sorry for them not being available. uploaded them to github. |
@@ -72,7 +72,6 @@ private void UpdateRepo(object sender, DoWorkEventArgs e) | |||
private void PostUpdateRepo(object sender, RunWorkerCompletedEventArgs e) | |||
{ | |||
SetDescription("Scanning for manually installed mods"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't this line also be removed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ah, I see the reason :)
We only need to know if any mod provides the specified mod, not which mods provides it.
this improves the launch time a bit
the call is already done before the refresh, and we don't need to do it twice
this removes unneeded calls to Availablemodule.Latest(..)
rebased the pr. |
Profiling was started before clicking on Refresh, and stopped after.
Before my changes:
After my changes: