gEDA-dev: Another autosave crash....

[Peter Clifton]

> Could people would have experienced this bug please try adding the
> following lines to s_toplevel_delete(), recompile and let us know if
> it solved the issue or not?
> 
>   if (toplevel->auto_save_timeout != 0) {
>     gboolean ret;
>     ret = g_source_remove (toplevel->auto_save_timeout);
>     g_assert (ret);
>   }

I've added this to try, but expect it will take a very long time to be
_sure_ it has cured the problem. I'm confident it will have though.

I think we also need:

  toplevel->auto_save_timeout = 0;

in s_toplevel_new(). I put mine after toplevel->auto_save_interval = 0;

This way, if s_toplevel_delete() is called before the timer is setup, we
won't have a problem. I've not checked, but I think gattrib might have
trouble if this isn't added. If the auto_save_interval is zero, the
timer is never setup either.

Peter