Peter,
in align.cpp and transformation.cpp, in delete signal handlers, you wrote
static gboolean sp_transformation_dialog_delete(GtkObject *, GdkEvent *, gpointer data) { if ( data != static_cast<gpointer>(dlg) ) { g_warning("possible bug: dlg differs from passed data"); /* E.g. maybe we shouldn't update the global x,y,w,h, or maybe we should use dlg instead of dead_dlg below. */ } GtkWindow &dead_dlg = *static_cast<GtkWindow *>(data); gtk_window_get_position(&dead_dlg, &x, &y); gtk_window_get_size(&dead_dlg, &w, &h);
instead of the previous code which simply used dlg, ignoring data. I don't quite understand the reason for this change. Besides:
- Why only these two files? The same code is used in all dialogs.
- Attempting to close Inkscape with any of these two dialogs open will crash (bug 917122).
So for now I restored the old code to fix the crash, but I'm just curious why you deemed this change necessary.
_________________________________________________________________ Add photos to your messages with MSN Premium. Get 2 months FREE* http://join.msn.com/?pgmarket=en-ca&page=byoa/prem&xAPID=1994&DI...