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

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));
    } 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. */
    }
}

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

|

<
|
|
|

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));
    } else if (event == EVENT_VALCHANGE) {

	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;