SCP: Give priority to device names in intial HELP commands

This commit is contained in:
Mark Pizzolato 2019-04-18 01:48:16 -07:00
parent 8d172a4049
commit 590be6b0dd

192
scp.c
View file

@ -2825,6 +2825,11 @@ for (cmdp = cmd_table; cmdp && (cmdp->name != NULL); cmdp++) {
max_cmdname_size = strlen(cmdp->name); max_cmdname_size = strlen(cmdp->name);
} }
} }
fprintf (st, "Help is available for devices\n\n");
fprintf (st, " HELP dev\n");
fprintf (st, " HELP dev SET\n");
fprintf (st, " HELP dev SHOW\n");
fprintf (st, " HELP dev REGISTERS\n\n");
fprintf (st, "Help is available for the following commands:\n\n "); fprintf (st, "Help is available for the following commands:\n\n ");
qsort (hlp_cmdp, cmd_cnt, sizeof(*hlp_cmdp), _cmd_name_compare); qsort (hlp_cmdp, cmd_cnt, sizeof(*hlp_cmdp), _cmd_name_compare);
line_offset = 4; line_offset = 4;
@ -3224,121 +3229,120 @@ t_stat help_cmd (int32 flag, CONST char *cptr)
{ {
char gbuf[CBUFSIZE], gbuf2[CBUFSIZE]; char gbuf[CBUFSIZE], gbuf2[CBUFSIZE];
CTAB *cmdp; CTAB *cmdp;
DEVICE *dptr;
UNIT *uptr;
t_stat r;
t_bool explicit_device = FALSE;
GET_SWITCHES (cptr); /* get switches */ GET_SWITCHES (cptr); /* get switches */
if (sim_switches & SWMASK ('F')) if (sim_switches & SWMASK ('F'))
flag = flag | SCP_HELP_FLAT; flag = flag | SCP_HELP_FLAT;
if (*cptr) { if (*cptr) {
cptr = get_glyph (cptr, gbuf, 0); cptr = get_glyph (cptr, gbuf, 0);
if ((cmdp = find_cmd (gbuf))) {
if (*cptr) {
if ((cmdp->action == &set_cmd) || (cmdp->action == &show_cmd)) {
DEVICE *dptr;
UNIT *uptr;
t_stat r;
cptr = get_glyph (cptr, gbuf, 0); if (0 == strcmp (gbuf, "DEVICE")) {
dptr = find_unit (gbuf, &uptr); explicit_device = TRUE;
if (dptr == NULL) cptr = get_glyph (cptr, gbuf, 0);
dptr = find_dev (gbuf); }
if (dptr != NULL) { dptr = find_unit (gbuf, &uptr);
r = help_dev_help (stdout, dptr, uptr, flag, (cmdp->action == &set_cmd) ? "SET" : "SHOW"); if ((dptr == NULL) &&
if (sim_log) ((dptr = find_dev (gbuf)) == NULL)) {
help_dev_help (sim_log, dptr, uptr, flag | SCP_HELP_FLAT, (cmdp->action == &set_cmd) ? "SET" : "SHOW"); if (explicit_device)
return r; return sim_messagef (SCPE_ARG, "No such device %s\n", gbuf);
} if ((cmdp = find_cmd (gbuf))) {
if (cmdp->action == &set_cmd) { /* HELP SET xxx (not device or unit) */ if (*cptr) {
if ((cmdp = find_ctab (set_glob_tab, gbuf)) && if ((cmdp->action == &set_cmd) || (cmdp->action == &show_cmd)) {
(cmdp->help)) DEVICE *dptr;
return help_cmd_output (flag, cmdp->help, cmdp->help_base); UNIT *uptr;
} t_stat r;
else { /* HELP SHOW xxx (not device or unit) */
SHTAB *shptr = find_shtab (show_glob_tab, gbuf);
if ((shptr == NULL) || (shptr->help == NULL) || (*shptr->help == '\0')) cptr = get_glyph (cptr, gbuf, 0);
return SCPE_ARG; dptr = find_unit (gbuf, &uptr);
return help_cmd_output (flag, shptr->help, NULL); if (dptr == NULL)
dptr = find_dev (gbuf);
if (dptr != NULL) {
r = help_dev_help (stdout, dptr, uptr, flag, (cmdp->action == &set_cmd) ? "SET" : "SHOW");
if (sim_log)
help_dev_help (sim_log, dptr, uptr, flag | SCP_HELP_FLAT, (cmdp->action == &set_cmd) ? "SET" : "SHOW");
return r;
}
if (cmdp->action == &set_cmd) { /* HELP SET xxx (not device or unit) */
if ((cmdp = find_ctab (set_glob_tab, gbuf)) &&
(cmdp->help))
return help_cmd_output (flag, cmdp->help, cmdp->help_base);
}
else { /* HELP SHOW xxx (not device or unit) */
SHTAB *shptr = find_shtab (show_glob_tab, gbuf);
if ((shptr == NULL) || (shptr->help == NULL) || (*shptr->help == '\0'))
return SCPE_ARG;
return help_cmd_output (flag, shptr->help, NULL);
}
return SCPE_ARG;
} }
return SCPE_ARG;
} }
} if (cmdp->help) {
if (cmdp->help) { if (strcmp (cmdp->name, "HELP") == 0) {
if (strcmp (cmdp->name, "HELP") == 0) { DEVICE *dptr;
DEVICE *dptr; int i;
int i;
for (i = 0; (dptr = sim_devices[i]) != NULL; i++) { for (i = 0; (dptr = sim_devices[i]) != NULL; i++) {
if (dptr->help) if (dptr->help)
sim_printf ("h{elp} %-17s display help for device %s\n", dptr->name, dptr->name); sim_printf ("h{elp} %-17s display help for device %s\n", dptr->name, dptr->name);
if (dptr->attach_help || if (dptr->attach_help ||
(DEV_TYPE(dptr) == DEV_MUX) || (DEV_TYPE(dptr) == DEV_MUX) ||
(DEV_TYPE(dptr) == DEV_DISK) || (DEV_TYPE(dptr) == DEV_DISK) ||
(DEV_TYPE(dptr) == DEV_TAPE)) { (DEV_TYPE(dptr) == DEV_TAPE)) {
sim_printf ("h{elp} %s ATTACH\t display help for device %s ATTACH command\n", dptr->name, dptr->name); sim_printf ("h{elp} %s ATTACH\t display help for device %s ATTACH command\n", dptr->name, dptr->name);
} }
if (dptr->registers) { if (dptr->registers) {
if (dptr->registers->name != NULL) if (dptr->registers->name != NULL)
sim_printf ("h{elp} %s REGISTERS\t display help for device %s register variables\n", dptr->name, dptr->name); sim_printf ("h{elp} %s REGISTERS\t display help for device %s register variables\n", dptr->name, dptr->name);
} }
if (dptr->modifiers) { if (dptr->modifiers) {
MTAB *mptr; MTAB *mptr;
for (mptr = dptr->modifiers; mptr->pstring != NULL; mptr++) { for (mptr = dptr->modifiers; mptr->pstring != NULL; mptr++) {
if (mptr->help) { if (mptr->help) {
sim_printf ("h{elp} %s SET\t\t display help for device %s SET commands (modifiers)\n", dptr->name, dptr->name); sim_printf ("h{elp} %s SET\t\t display help for device %s SET commands (modifiers)\n", dptr->name, dptr->name);
break; break;
}
} }
} }
} }
} }
} else {
else { if (((cmdp->action == &exdep_cmd) || (0 == strcmp(cmdp->name, "BOOT"))) &&
if (((cmdp->action == &exdep_cmd) || (0 == strcmp(cmdp->name, "BOOT"))) && sim_dflt_dev->help) {
sim_dflt_dev->help) { sim_dflt_dev->help (stdout, sim_dflt_dev, sim_dflt_dev->units, 0, cmdp->name);
sim_dflt_dev->help (stdout, sim_dflt_dev, sim_dflt_dev->units, 0, cmdp->name); if (sim_log)
if (sim_log) sim_dflt_dev->help (sim_log, sim_dflt_dev, sim_dflt_dev->units, 0, cmdp->name);
sim_dflt_dev->help (sim_log, sim_dflt_dev, sim_dflt_dev->units, 0, cmdp->name); }
} }
strlcpy (gbuf2, cmdp->help, sizeof (gbuf2));
if (*cptr) {
strlcat (gbuf2, " ", sizeof (gbuf2));
strlcat (gbuf2, cptr, sizeof (gbuf2));
}
help_cmd_output (flag, gbuf2, cmdp->help_base);
} }
strlcpy (gbuf2, cmdp->help, sizeof (gbuf2)); else { /* no help so it is likely a command alias */
if (*cptr) { CTAB *cmdpa;
strlcat (gbuf2, " ", sizeof (gbuf2));
strlcat (gbuf2, cptr, sizeof (gbuf2));
}
help_cmd_output (flag, gbuf2, cmdp->help_base);
}
else { /* no help so it is likely a command alias */
CTAB *cmdpa;
for (cmdpa=cmd_table; cmdpa->name != NULL; cmdpa++) for (cmdpa=cmd_table; cmdpa->name != NULL; cmdpa++)
if ((cmdpa->action == cmdp->action) && (cmdpa->help)) { if ((cmdpa->action == cmdp->action) && (cmdpa->help)) {
sim_printf ("%s is an alias for the %s command:\n%s", sim_printf ("%s is an alias for the %s command:\n%s",
cmdp->name, cmdpa->name, cmdpa->help); cmdp->name, cmdpa->name, cmdpa->help);
break; break;
} }
if (cmdpa->name == NULL) /* not found? */ if (cmdpa->name == NULL) /* not found? */
sim_printf ("No help available for the %s command\n", cmdp->name); sim_printf ("No help available for the %s command\n", cmdp->name);
}
} }
} }
else { else {
DEVICE *dptr; if (dptr->flags & DEV_DISABLE)
UNIT *uptr; sim_printf ("Device %s is currently disabled\n", dptr->name);
t_stat r;
t_bool explicit_device = FALSE;
if (0 == strcmp (gbuf, "DEVICE")) {
explicit_device = TRUE;
cptr = get_glyph (cptr, gbuf, 0);
}
dptr = find_unit (gbuf, &uptr);
if (dptr == NULL) {
dptr = find_dev (gbuf);
if (dptr == NULL)
return sim_messagef (SCPE_ARG, "No HELP available for %s%s %s\n",
explicit_device ? "DEVICE " : "", gbuf, cptr);
if (dptr->flags & DEV_DISABLE)
sim_printf ("Device %s is currently disabled\n", dptr->name);
}
r = help_dev_help (stdout, dptr, uptr, flag, cptr); r = help_dev_help (stdout, dptr, uptr, flag, cptr);
if (sim_log) if (sim_log)
help_dev_help (sim_log, dptr, uptr, flag | SCP_HELP_FLAT, cptr); help_dev_help (sim_log, dptr, uptr, flag | SCP_HELP_FLAT, cptr);