{"Mark", SER_PAR_MARK},
{"Space", SER_PAR_SPACE},
};
- int i;
- Config *cfg = (Config *)data;
+ int mask = ctrl->listbox.context.i;
+ int i, j;
+ Conf *conf = (Conf *)data;
if (event == EVENT_REFRESH) {
+ /* Fetching this once at the start of the function ensures we
+ * remember what the right value is supposed to be when
+ * operations below cause reentrant calls to this function. */
+ int oldparity = conf_get_int(conf, CONF_serparity);
+
dlg_update_start(ctrl, dlg);
dlg_listbox_clear(ctrl, dlg);
- for (i = 0; i < lenof(parities); i++)
- dlg_listbox_addwithid(ctrl, dlg, parities[i].name,
- parities[i].val);
- for (i = 0; i < lenof(parities); i++)
- if (cfg->serparity == parities[i].val)
- dlg_listbox_select(ctrl, dlg, i);
+ for (i = 0; i < lenof(parities); i++) {
+ if (mask & (1 << i))
+ dlg_listbox_addwithid(ctrl, dlg, parities[i].name,
+ parities[i].val);
+ }
+ for (i = j = 0; i < lenof(parities); i++) {
+ if (mask & (1 << i)) {
+ if (oldparity == parities[i].val) {
+ dlg_listbox_select(ctrl, dlg, j);
+ break;
+ }
+ j++;
+ }
+ }
+ if (i == lenof(parities)) { /* an unsupported setting was chosen */
+ dlg_listbox_select(ctrl, dlg, 0);
+ oldparity = SER_PAR_NONE;
+ }
dlg_update_done(ctrl, dlg);
+ conf_set_int(conf, CONF_serparity, oldparity); /* restore */
} else if (event == EVENT_SELCHANGE) {
int i = dlg_listbox_index(ctrl, dlg);
if (i < 0)
i = SER_PAR_NONE;
else
i = dlg_listbox_getid(ctrl, dlg, i);
- cfg->serparity = i;
+ conf_set_int(conf, CONF_serparity, i);
}
}
{"RTS/CTS", SER_FLOW_RTSCTS},
{"DSR/DTR", SER_FLOW_DSRDTR},
};
- int i;
- Config *cfg = (Config *)data;
+ int mask = ctrl->listbox.context.i;
+ int i, j;
+ Conf *conf = (Conf *)data;
if (event == EVENT_REFRESH) {
+ /* Fetching this once at the start of the function ensures we
+ * remember what the right value is supposed to be when
+ * operations below cause reentrant calls to this function. */
+ int oldflow = conf_get_int(conf, CONF_serflow);
+
dlg_update_start(ctrl, dlg);
dlg_listbox_clear(ctrl, dlg);
- for (i = 0; i < lenof(flows); i++)
- dlg_listbox_addwithid(ctrl, dlg, flows[i].name,
- flows[i].val);
- for (i = 0; i < lenof(flows); i++)
- if (cfg->serflow == flows[i].val)
- dlg_listbox_select(ctrl, dlg, i);
+ for (i = 0; i < lenof(flows); i++) {
+ if (mask & (1 << i))
+ dlg_listbox_addwithid(ctrl, dlg, flows[i].name, flows[i].val);
+ }
+ for (i = j = 0; i < lenof(flows); i++) {
+ if (mask & (1 << i)) {
+ if (oldflow == flows[i].val) {
+ dlg_listbox_select(ctrl, dlg, j);
+ break;
+ }
+ j++;
+ }
+ }
+ if (i == lenof(flows)) { /* an unsupported setting was chosen */
+ dlg_listbox_select(ctrl, dlg, 0);
+ oldflow = SER_FLOW_NONE;
+ }
dlg_update_done(ctrl, dlg);
+ conf_set_int(conf, CONF_serflow, oldflow);/* restore */
} else if (event == EVENT_SELCHANGE) {
int i = dlg_listbox_index(ctrl, dlg);
if (i < 0)
- i = SER_PAR_NONE;
+ i = SER_FLOW_NONE;
else
i = dlg_listbox_getid(ctrl, dlg, i);
- cfg->serflow = i;
+ conf_set_int(conf, CONF_serflow, i);
}
}
-void ser_setup_config_box(struct controlbox *b, int midsession)
+void ser_setup_config_box(struct controlbox *b, int midsession,
+ int parity_mask, int flow_mask)
{
struct controlset *s;
union control *c;
- /*
- * Add the serial back end to the protocols list at the top of
- * the config box.
- */
- s = ctrl_getset(b, "Session", "hostport",
- "Specify your connection by host name or IP address");
- {
+ if (!midsession) {
int i;
extern void config_protocolbuttons_handler(union control *, void *,
void *, int);
+
+ /*
+ * Add the serial back end to the protocols list at the
+ * top of the config box.
+ */
+ s = ctrl_getset(b, "Session", "hostport",
+ "Specify the destination you want to connect to");
+
for (i = 0; i < s->ncontrols; i++) {
c = s->ctrls[i];
if (c->generic.type == CTRL_RADIO &&
if (c->radio.shortcuts) {
c->radio.shortcuts =
sresize(c->radio.shortcuts, c->radio.nbuttons, char);
- c->radio.shortcuts[c->radio.nbuttons-1] = NO_SHORTCUT;
+ c->radio.shortcuts[c->radio.nbuttons-1] = 'r';
}
}
}
"Select a serial line");
ctrl_editbox(s, "Serial line to connect to", 'l', 40,
HELPCTX(serial_line),
- dlg_stdeditbox_handler, I(offsetof(Config,serline)),
- I(sizeof(((Config *)0)->serline)));
+ conf_editbox_handler, I(CONF_serline), I(1));
}
s = ctrl_getset(b, "Connection/Serial", "sercfg", "Configure the serial line");
ctrl_editbox(s, "Speed (baud)", 's', 40,
HELPCTX(serial_speed),
- dlg_stdeditbox_handler, I(offsetof(Config,serspeed)), I(-1));
+ conf_editbox_handler, I(CONF_serspeed), I(-1));
ctrl_editbox(s, "Data bits", 'b', 40,
HELPCTX(serial_databits),
- dlg_stdeditbox_handler,I(offsetof(Config,serdatabits)),I(-1));
+ conf_editbox_handler, I(CONF_serdatabits), I(-1));
/*
* Stop bits come in units of one half.
*/
ctrl_editbox(s, "Stop bits", 't', 40,
HELPCTX(serial_stopbits),
- dlg_stdeditbox_handler,I(offsetof(Config,serstopbits)),I(-2));
+ conf_editbox_handler, I(CONF_serstopbits), I(-2));
ctrl_droplist(s, "Parity", 'p', 40,
HELPCTX(serial_parity),
- serial_parity_handler, I(0));
+ serial_parity_handler, I(parity_mask));
ctrl_droplist(s, "Flow control", 'f', 40,
HELPCTX(serial_flow),
- serial_flow_handler, I(0));
+ serial_flow_handler, I(flow_mask));
}