Re: gEDA-dev: Another autosave crash....

[Ales Hvezda <ahvezda-at-NOSPAMseul_dot_org>]

[snip]
>I think we also need:
>
>  toplevel->auto_save_timeout = 0;
>
>in s_toplevel_new(). I put mine after toplevel->auto_save_interval = 0;

	Ah, now I understand why valgrind wasn't happy.
auto_save_interval and auto_save_timeout look similar if you aren't
paying attention (d'oh).  Yes, this is required.  Thanks.


>
>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.

	In theory, however, the timer was being setup for the preview
toplevel and that was probably a bad idea.

							-Ales



_______________________________________________
geda-dev mailing list
geda-dev@moria.seul.org
http://www.seul.org/cgi-bin/mailman/listinfo/geda-dev