https://bugzilla.xfce.org/show_bug.cgi?id=15873

--- Comment #8 from Igor <f2...@yandex.ru> ---
Hi Ruben, my bad, I overlooked that the check actually looks for a failed
condition. :(

Since xfce_message_dialog_new() does allow NULL parent, I think your function
should allow it, too.

Another question is what value should be returned in the failure scenario:
g_return_val_if_fail (parent == NULL || GTK_IS_WINDOW (parent), 0);
Currently, 0 is returned; however, GtkResponseType enum doesn't define any
response as 0 - "GTK+ leaves values of 0 or greater for application-defined
response ids." (see
https://developer.gnome.org/gtk3/stable/GtkDialog.html#GtkResponseType)

-- 
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Xfce-bugs mailing list
Xfce-bugs@xfce.org
https://mail.xfce.org/mailman/listinfo/xfce-bugs

Reply via email to