[Lxde-list] --background-color to lxterminal

[Juha H. <jh...@tu...>]

it would be nice if bgcolor/fgcolor given in lxterminal.conf could be
overridden by command like option --background-color/--foreground-color.

i tried to experimentally add that to lxterminal.c after settings are
loaded from config file:

       /* load from config file/
	if (!g_file_test(path, G_FILE_TEST_EXISTS)) {
d		setting = load_setting_from_file(PACKAGE_DATA_DIR "/lxterminal/lxterminal.conf");
		/* save to user's directory */
		setting_save(setting);
	} else {
		setting = load_setting_from_file(path);
	}
	g_free(path);

	/* override from command line */
	for (i=1;i<argc;i++) {
	    if (strncmp(argv[i],"--background-color=", 19)==0) {
		setting->bgcolor=g_strdup(argv[i]+19);
	    }
	    if (strncmp(argv[i],"--foreground-color=", 19)==0) {
		setting->fgcolor=g_strdup(argv[i]+19);
	    }
	}

but it didn't have any effect. i also tried to add debug fprints() to
the code, but nothing came to the file.

could someone give a hint why the above does not work?  obviously i
don't understand the design.

-- juha