Differences From Artifact [aae26ebe06]:

• File config.c — part of check-in [78529746fb] at 2011-10-01 12:38:59 on branch trunk — Change the semantics of 'FontSpec' so that it's a dynamically allocated type. The main reason for this is to stop it from taking up a fixed large amount of space in every 'struct value' subunion in conf.c, although that makes little difference so far because Filename is still doing the same thing (and is therefore next on my list). However, the removal of its arbitrary length limit is not to be sneezed at. (user: simon size: 81314)

To Artifact [a0746e7fd5]:

• File config.c — part of check-in [7756dc109c] at 2011-10-02 06:01:57 on branch trunk — Turn 'Filename' into a dynamically allocated type with no arbitrary length limit, just as I did to FontSpec yesterday. (user: simon size: 81279)

127
128
129
130
131
132
133
134

135
136
137
138
139



140
141
142
143
144
145
146

127
128
129
130
131
132
133

134
135




136
137
138
139
140
141
142
143
144
145

-
+

-
-
-
-
+
+
+

void conf_filesel_handler(union control *ctrl, void *dlg,
			  void *data, int event)
{
    int key = ctrl->fileselect.context.i;
    Conf *conf = (Conf *)data;

    if (event == EVENT_REFRESH) {
	dlg_filesel_set(ctrl, dlg, *conf_get_filename(conf, key));
	dlg_filesel_set(ctrl, dlg, conf_get_filename(conf, key));
    } else if (event == EVENT_VALCHANGE) {
	Filename filename;
	dlg_filesel_get(ctrl, dlg, &filename);
	conf_set_filename(conf, key, &filename);
	/* If Filenames ever become dynamic, free this one. */
	Filename *filename = dlg_filesel_get(ctrl, dlg);
	conf_set_filename(conf, key, filename);
        filename_free(filename);
    }
}

void conf_fontsel_handler(union control *ctrl, void *dlg,
			  void *data, int event)
{
    int key = ctrl->fontselect.context.i;