Skip to content

perf: a bunch of optimization#2417

Open
SimplisticMind wants to merge 2 commits into
ModOrganizer2:masterfrom
SimplisticMind:performance
Open

perf: a bunch of optimization#2417
SimplisticMind wants to merge 2 commits into
ModOrganizer2:masterfrom
SimplisticMind:performance

Conversation

@SimplisticMind

Copy link
Copy Markdown

This is a large one so if you want to just cherrypick - feel free to do so; with that out of the way, a few less obvious ones:
envfs.h - upwards of ~20%+ faster refreshing
moapplication.cpp - Qt elements render up to twice as fast
settings.cpp - most impactful settings got hoisted out, other ones aren't hot enough to really necessitate such hoisting too
fileregister.cpp - sortOrigins() - 4x faster compared to locking each file seperately

@Holt59

Holt59 commented Jun 27, 2026

Copy link
Copy Markdown
Member

Thanks for the PR, it would be easier and more likely to be accepted if you split it in smaller chunk because as it stands, it's complicated to review (and cherry-picking would take a lot of time).

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