pidgin: aa3b9562: Quote the path for the windows "run at s...

Daniel Atallah daniel.atallah at gmail.com
Wed Oct 27 09:46:15 EDT 2010


On Wed, Oct 27, 2010 at 00:14, Kevin Stange <kstange at pidgin.im> wrote:
> On 10/26/2010 01:56 PM, datallah at pidgin.im wrote:
>>  static GtkWidget* get_config_frame(PurplePlugin *plugin) {
>> -     GtkWidget *ret;
>> -     GtkWidget *vbox;
>> -     GtkWidget *button;
>> +     GtkWidget *ret, *vbox, *button, *language_sel;
>>       char *run_key_val;
>>       char *tmp;
>>
>>       ret = gtk_vbox_new(FALSE, 18);
>>       gtk_container_set_border_width(GTK_CONTAINER(ret), 12);
>>
>> +     vbox = pidgin_make_frame(ret, _("Language"));
>> +     language_sel = gtk_option_menu_new();
>> +
>>       /* Autostart */
>>       vbox = pidgin_make_frame(ret, _("Startup"));
>>       tmp = g_strdup_printf(_("_Start %s on Windows startup"), PIDGIN_NAME);
>
> Was this change intentional?  It seems to add a Language frame, and
> create a widget that's never used.

Oops.   That wasn't supposed to be part of the commit.  I'll back it out.

-D




More information about the Devel mailing list