[PATCH] Add "set print finish"

Andrew Burgess andrew.burgess@embecosm.com
Thu May 16 21:46:00 GMT 2019


* Tom Tromey <tromey@adacore.com> [2019-05-16 12:08:39 -0600]:

> >>>>> "Andrew" == Andrew Burgess <andrew.burgess@embecosm.com> writes:
> 
> >> +  add_setshow_boolean_cmd ("finish", class_support,
> >> +			   &user_print_options.finish_print, _("\
> >> +Set whether `finish' prints the return value."), _("\
> >> +Show whether `finish' prints the return value."), NULL,
> >> +			   NULL,
> >> +			   show_print_finish,
> >> +			   &setprintlist, &showprintlist);
> 
> 
> >> +  /* Whether "finish" should print the value.  */
> >> +  int finish_print;
> >> };
> 
> Andrew> Should new flags not be bool ?
> 
> Ordinarily, but in this case the address of one of these is passed to
> add_setshow_boolean_cmd, which still takes an "int *" -- see the code I
> quoted above.

Of course.  Sorry for the noise :)

Thanks,
Andrew


> 
> Tom



More information about the Gdb-patches mailing list