slajerek / RetroDebugger

Retro Debugger is a multiplatform debugger APIs host for retro computers: C64 (Vice), Atari800 and NES (NestopiaUE).
175 stars 20 forks source link

Shows up "Assertion failed!" dialog upon start #16

Open maciejmalecki opened 1 year ago

maciejmalecki commented 1 year ago

Describe the bug After I have rearanged the layout of the app (I moved C64 CPU status to the very top of the screen), I no longer can run RetroDebugger without this error (see screen shot). It is an assert fail on node->TabBar == 0 (imgui.cpp Line 16271). When I start RetroDebugger normally (by clicking exe file from Win Explorer, assertion dialog shows, but I can ignore it and debugger then starts normally). When I use RetroDebugger from KickAssembler VSC plugin (by Paul Hocker), no matter what I press, debugger gets closed after I click anything on that assert dialog.

To Reproduce Steps to reproduce the behavior:

  1. Run Retrodebugger (either by clicking on exe or by Ctrl+F6 from VSC)
  2. See error

The error is most likely caused by my local configuration, but I cannot find the place where it is stored (I don't see it in Users/). Probably cleaning that out would help, but sadly I don't know how to do it.

Expected behavior I don't see a reason for this assert dialog to show up. App should work no matter how I rearrange the layout.

Screenshots image

Desktop (please complete the following information):

Additional context Add any other context about the problem here.

slajerek commented 1 year ago

This seems to be a bug in ImGui that I hit once before. Cleaning layouts is enough. Data is stored in default program configuration, depends on Windows version, usually in ProgramData/RetroDebugger Please send me layouts.dat file before clearing that. There's also a command line option that when run with exe will clear settings.

maciejmalecki commented 1 year ago

here it is: https://drive.google.com/file/d/1BINnurpavlLZd__gYYmv0EqbBTv1Ugp-/view?usp=sharing

slajerek commented 9 months ago

This has been partially fixed in a69c371 although this was a crude fix by removing assert in ImGui. Interestingly this works well without that assert. I will post this issue to ImGui forums with more details.