Quote the path for the windows "run at startup" registry key.

Tue, 26 Oct 2010 18:52:56 +0000

author
Daniel Atallah <datallah@pidgin.im>
date
Tue, 26 Oct 2010 18:52:56 +0000
changeset 30876
cd041b5b7ceb
parent 30875
37e57d4ba633
child 30877
e0c4797434bf
child 31083
41dbf303d766

Quote the path for the windows "run at startup" registry key.
The only reason it ever worked like it was is that MS apparently has some wacky
fallbacks that makes it find stuff in the Program Files folder when it sees a
directory called "Program".
Thanks to Christian Lange for noticing and tracking down the problem.
Fixes #12781

pidgin/plugins/win32/winprefs/winprefs.c file | annotate | diff | comparison | revisions
--- a/pidgin/plugins/win32/winprefs/winprefs.c	Mon Oct 25 22:43:30 2010 +0000
+++ b/pidgin/plugins/win32/winprefs/winprefs.c	Tue Oct 26 18:52:56 2010 +0000
@@ -193,7 +193,7 @@
 	char *runval = NULL;
 
 	if(gtk_toggle_button_get_active(GTK_TOGGLE_BUTTON(w)))
-		runval = g_strdup_printf("%s" G_DIR_SEPARATOR_S "pidgin.exe", wpurple_install_dir());
+		runval = g_strdup_printf("\"%s" G_DIR_SEPARATOR_S "pidgin.exe\"", wpurple_install_dir());
 
 	if(!wpurple_write_reg_string(HKEY_CURRENT_USER, RUNKEY, "Pidgin", runval)
 		/* For Win98 */
@@ -267,15 +267,16 @@
 }
 
 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);

mercurial