[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