This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [RFA 5/5] Use ui_out_emit_list
- From: Pedro Alves <palves at redhat dot com>
- To: Tom Tromey <tom at tromey dot com>, gdb-patches at sourceware dot org
- Date: Tue, 18 Apr 2017 19:44:35 +0100
- Subject: Re: [RFA 5/5] Use ui_out_emit_list
- Authentication-results: sourceware.org; auth=none
- Authentication-results: ext-mx06.extmail.prod.ext.phx2.redhat.com; dmarc=none (p=none dis=none) header.from=redhat.com
- Authentication-results: ext-mx06.extmail.prod.ext.phx2.redhat.com; spf=pass smtp.mailfrom=palves at redhat dot com
- Dkim-filter: OpenDKIM Filter v2.11.0 mx1.redhat.com 7223C3DBCC
- Dmarc-filter: OpenDMARC Filter v1.3.2 mx1.redhat.com 7223C3DBCC
- References: <20170414022337.28368-1-tom@tromey.com> <20170414022337.28368-6-tom@tromey.com>
On 04/14/2017 03:23 AM, Tom Tromey wrote:
> This changes some spots to use ui_out_emit_list. This only touches
> "easy" cases, where the cleanup was used in a block-structured way.
> There's also one more use of ui_out_emit_tuple in here.
>
> ChangeLog
> 2017-04-13 Tom Tromey <tom@tromey.com>
>
> * mi/mi-cmd-file.c (mi_cmd_file_list_shared_libraries): Use
> ui_out_emit_list.
> * stack.c (print_frame): Use ui_out_emit_list.
> * mi/mi-symbol-cmds.c (mi_cmd_symbol_list_lines): Use
> ui_out_emit_list.
> * mi/mi-main.c (print_one_inferior)
> (mi_cmd_data_list_register_names)
> (mi_cmd_data_list_register_values, mi_cmd_list_features)
> (mi_cmd_list_target_features, mi_cmd_trace_frame_collected): Use
> ui_out_emit_list.
> * mi/mi-interp.c (mi_on_normal_stop_1): Use ui_out_emit_list.
> (mi_output_solib_attribs): Use ui_out_emit_list,
> ui_out_emit_tuple.
> * mi/mi-cmd-var.c (varobj_update_one): Use ui_out_emit_list.
> * mi/mi-cmd-stack.c (mi_cmd_stack_list_frames)
> (mi_cmd_stack_list_args, list_args_or_locals): Use
> ui_out_emit_list.
> * disasm.c (do_assembly_only): Use ui_out_emit_list.
> * breakpoint.c (print_solib_event, output_thread_groups): Use
> ui_out_emit_list.
LGTM.
Thanks,
Pedro Alves