dfhack icon indicating copy to clipboard operation
dfhack copied to clipboard

inception denied: opening gui/launcher from within gui/launcher hangs

Open ab9rf opened this issue 3 years ago • 3 comments

reported on dfhack discord

ab9rf avatar Jan 28 '23 03:01 ab9rf

The intended behavior is to close the launcher: https://github.com/DFHack/scripts/blob/ff462de7c4394b9a2df008391ac7ec2878d7021f/gui/launcher.lua#L774-L781

@ab9rf reports on Discord that the hang happens here, because screen == screen->child: https://github.com/DFHack/dfhack/blob/bf95520949edc67fc6965d026861e6be58d4b167/library/Core.cpp#L1832-L1833

lethosor avatar Jan 28 '23 04:01 lethosor

This appears to have been introduced after 0.47.05-r8 - in that version, running gui/launcher from gui/launcher just dismisses the launcher.

lethosor avatar Jan 28 '23 04:01 lethosor

This probably happens because the current screen is in limbo in the hideGuard while the new instance is instantiated. I wonder if quickcmd is also affected, since it uses the same technique.

myk002 avatar Jan 28 '23 05:01 myk002