Commit bd7cd9bc authored by Albert Gräf's avatar Albert Gräf Committed by Albert Gräf
Browse files

Fix a spurious segfault on Windows.

This can be seen, e.g., when running regression_tests.pd gui-less in
Windows, as our CI script does. The easy fix is to just bypass the gui
finalization code in glob_quit() when running gui-less (sys_nogui).
parent 9921397d
Pipeline #2720 failed with stage
in 229 minutes and 4 seconds
......@@ -1775,6 +1775,16 @@ extern int do_not_redraw;
void glob_quit(void *dummy, t_floatarg status)
{
if (sys_nogui)
{
// ag: Take the quick way out. Specifically, we do *not* want to clean
// up a non-existent gui here (which also causes spurious segfaults in
// gui-less operation on Windows).
canvas_suspend_dsp();
sys_bail(status);
// sys_bail shouldn't return, but just in case:
return;
}
/* If we're going to try to cleanly close everything here, we should
do the same for all open patches and that is currently not the case,
so for the time being, let's just leave OS to deal with freeing of all
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment