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

Right panel shows old info in case all is deleted in the left-hand tree #3293

Closed
greenozon opened this issue Sep 27, 2024 · 7 comments · Fixed by #3297 or #3298
Closed

Right panel shows old info in case all is deleted in the left-hand tree #3293

greenozon opened this issue Sep 27, 2024 · 7 comments · Fixed by #3297 or #3298
Labels

Comments

@greenozon
Copy link

Steps to reproduce

  1. Delete all assemblies
  2. rihgt panel show old info

Error message shown

eg:

image

Copy and paste any error messages / relevant text and/or screenshot the incorrect behaviour.

Details

  • ILSpy version 9.0.0.7785-preview2+c1a42ccf7205f41d9ba3988926162feb6e57ca98
    .NET version 8.0.8+08338fcaa5c9b9a8190abb99222fed12aaba956c
@greenozon greenozon added the Bug label Sep 27, 2024
@siegfriedpammer
Copy link
Member

The problem seems to be that the last tab is closed. This should not be the case. Previously the last tab was uncloseable.

@christophwille
Copy link
Member

By previously you mean pre- #3274?

@greenozon
Copy link
Author

added video show

vid-.2024-09-29.10-57-52-579.mp4

@siegfriedpammer
Copy link
Member

By previously you mean pre- #3274?

Yes, our latest official release shows the following, if the assembly list is empty:

image

There is a tab "New tab" with no content. From @greenozon's first screenshot, we can see that in the current master, there can be zero open tabs, which is not desired, because ILSpy is essentially an SDI application, with the possibility to view multiple documents. if necessary.

@christophwille
Copy link
Member

@tom-englert

@siegfriedpammer
Copy link
Member

@tom-englert I can still reproduce this using the latest master (ILSpy version 9.0.0.7802-preview2+7f46aab34493d01edf9801df6b86fc14d5c8ae62):

  1. create/open non-empty assembly list
  2. ctrl+a all tree nodes
  3. hit delete
  4. observe that the decompilation view is reloading/still showing old info

I also noticed that switching the current assembly list while a decompilation is in progress, it's not cancelled. When switching tree nodes the old decompilation is cancelled. I would expect the same to happen, when switching the assembly list mid-progress.

I will try to look into this, but ever since the refactoring I am no longer sure how the code is supposed to work/be organized, so actually fixing stuff is a lot harder. The code is organized better, but I still have to get used to it.

@tom-englert
Copy link
Contributor

@siegfriedpammer I'll have a look, too

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jan 5, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants