Hi Pedro. The enum auto_boolean and add_setshow_auto_boolean_cmd is cool. Thanks for you remind me. I had changed it. Do you think it's OK? Thanks, Hui On Fri, Mar 13, 2009 at 08:20, Pedro Alves wrote: >> --- a/stack.c >> +++ b/stack.c >> >> +/* Enum strings for "set|show disassemble-next-line".  */ >> + >> +static const char disassemble_next_line_auto[] = "auto"; >> +static const char disassemble_next_line_on[] = "on"; >> +static const char disassemble_next_line_off[] = "off"; >> +static const char *disassemble_next_line_enum[] = >> +{ >> +  disassemble_next_line_auto, >> +  disassemble_next_line_on, >> +  disassemble_next_line_off, >> +  NULL, >> +}; > > Please make this an `enum auto_boolean' / AUTO_BOOLEAN_AUTO, and > register the "disassemble-next-line" command with > add_setshow_auto_boolean_cmd instead. > > -- > Pedro Alves >