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

Bad UI performance with many modules loaded #405

Open
informatorius opened this issue Jan 31, 2025 · 0 comments
Open

Bad UI performance with many modules loaded #405

informatorius opened this issue Jan 31, 2025 · 0 comments
Labels
bug Something isn't working

Comments

@informatorius
Copy link

informatorius commented Jan 31, 2025

dnSpyEx version

6.5.1

Describe the Bug

I got bad UI performance with many modules loaded. Callstack of dnspy shows it updates Command.CanExecute in UI thread on any mouse click. But it does not return just a true or false value. But sync calls into DocumentTreeView and updates all nodes.

Expected behavior: Update DocumentTreeView async and send event to UI when finished to update Command.CanExecute from a bool value.

Image

Also it seems this DocumentTreeView update happens too often or is uncached.

And using parallel.foreach or Linq.AsParallel() may distribute the node updates and search to more CPU cores for faster results.

Image

How To Reproduce

load many modules and search for a string or debug

Expected Behavior

see above

Actual Behavior

see above

Additional Context

Should I create a pull request if feasible?

@informatorius informatorius added the bug Something isn't working label Jan 31, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant