AuroraMiddleware/gtk2
1
0
Fork 0
forked from AuroraMiddleware/gtk
Code Issues Pull Requests Projects Releases Wiki Activity

open-with: don't override the class signal handler, as it's RUN_LAST

Browse Source 
And explain why in a comment; basically we want our handler to always
run before client code.
This commit is contained in:
Cosimo Cecchi 2010-11-16 15:12:17 +01:00
parent c8ea415873
commit 1d67b4ac26
1 changed files with 11 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
gtk/gtkopenwithdialog.c
View File

@ -293,7 +293,8 @@ emit_application_selected (GtkOpenWithDialog *self,
static void
gtk_open_with_dialog_response (GtkDialog *dialog,
gint response_id)
gint response_id,
gpointer user_data)
{
GAppInfo *application;
GtkOpenWithDialog *self = GTK_OPEN_WITH_DIALOG (dialog);
@ -346,6 +347,9 @@ gtk_open_with_dialog_response (GtkDialog *dialog,
}
}
/* don't forward this signal to other clients in this case */
g_signal_stop_emission_by_name (self, "response");
break;
default :
break;
@ -1058,7 +1062,6 @@ static void
gtk_open_with_dialog_class_init (GtkOpenWithDialogClass *klass)
{
GObjectClass *gobject_class;
GtkDialogClass *dialog_class;
gobject_class = G_OBJECT_CLASS (klass);
gobject_class->finalize = gtk_open_with_dialog_finalize;
@ -1066,9 +1069,6 @@ gtk_open_with_dialog_class_init (GtkOpenWithDialogClass *klass)
gobject_class->get_property = gtk_open_with_dialog_get_property;
gobject_class->constructed = gtk_open_with_dialog_constructed;
dialog_class = GTK_DIALOG_CLASS (klass);
dialog_class->response = gtk_open_with_dialog_response;
properties[PROP_GFILE] =
g_param_spec_object ("gfile",
P_("A GFile object"),
@ -1249,6 +1249,12 @@ gtk_open_with_dialog_init (GtkOpenWithDialog *self)
gtk_dialog_set_default_response (GTK_DIALOG (self),
GTK_RESPONSE_OK);
/* we can't override the class signal handler here, as it's a RUN_LAST;
* we want our signal handler instead to be executed before any user code.
*/
g_signal_connect (self, "response",
G_CALLBACK (gtk_open_with_dialog_response), NULL);
}
static void