Re: [PATCH 2/9] gdb: Refactor 'stack.c:print_frame'.

[Tom Tromey <tom@tromey.com>]

>>>>> "Christina" == Christina Schimpe <christina.schimpe@intel.com> writes:

Christina> Refactor the function 'stack.c:print_frame' by introducing several
Christina> small functions.
Christina> The functions will also be used in a later patch "gdb: Implement
Christina> 'bt shadow' to print the shadow stack backtrace."

Thanks.

Christina> +static void
Christina> +print_funname (ui_out *uiout,
Christina> +	       gdb::unique_xmalloc_ptr<char> const &funname)
Christina> +{
Christina> +  annotate_frame_function_name ();
Christina> +  string_file stb;
Christina> +  gdb_puts (funname ? funname.get () : "??", &stb);
Christina> +  uiout->field_stream ("func", stb, function_name_style.style ());

Not really your problem, but using string_file here seems like overkill.
This would be better as plain field_string using funname directly.

anyway,
Approved-By: Tom Tromey <tom@tromey.com>

Tom