From 112e8700a6fd2fed65ca70132c9cbed4132e8bd4 Mon Sep 17 00:00:00 2001 From: Simon Marchi Date: Thu, 22 Dec 2016 16:17:31 -0500 Subject: Class-ify ui_out This patch finalizes the C++ conversion of the ui-out subsystem, by turning the ui_out and ui_out_impl structures into a single class hierarchy. ui_out functions are turned into virtual methods of that new class, so as a result there are a lot of call sites to update. In the previous version of the patchset, there were separate ui_out and ui_out_impl classes, but it wasn't really useful and added boilerplate. In this version there is simply an ui_out base class that is extended for CLI, TUI and MI. It's a bit hard to maintain a ChangeLog for such a big patch, I did my best but I'm sure there are some missing or outdated info in there... gdb/ChangeLog: * ui-out.h (ui_out_begin, ui_out_end, ui_out_table_header, ui_out_table_body, ui_out_field_int, ui_out_field_fmt_int, ui_out_field_core_addr, ui_out_field_string, ui_out_field_stream, ui_out_field_fmt, ui_out_field_skip, ui_out_spaces, ui_out_text, ui_out_message, ui_out_wrap_hint, ui_out_flush, ui_out_test_flags, ui_out_query_field, ui_out_is_mi_like_p, ui_out_redirect): Remove, replace with a method in class ui_out. (table_begin_ftype): Remove, replace with pure virtual method in class ui_out. (table_body_ftype): Likewise. (table_end_ftype): Likewise. (table_header_ftype): Likewise. (ui_out_begin_ftype): Likewise. (ui_out_end_ftype): Likewise. (field_int_ftype): Likewise. (field_skip_ftype): Likewise. (field_string_ftype): Likewise. (field_fmt_ftype): Likewise. (spaces_ftype): Likewise. (text_ftype): Likewise. (message_ftype): Likewise. (wrap_hint_ftype): Likewise. (flush_ftype): Likewise. (redirect_ftype): Likewise. (data_destroy_ftype): Likewise. (struct ui_out_impl): Remove, replace with class ui_out. (ui_out_new): Remove. (class ui_out): New class. * ui-out.c (struct ui_out): Remove, replaced with class ui_out. (current_level): Remove, replace with ui_out method. (push_level): Likewise. (pop_level): Likewise. (uo_table_begin, uo_table_body, uo_table_end, uo_table_header, uo_begin, uo_end, uo_field_int, uo_field_skip, uo_field_fmt, uo_spaces, uo_text, uo_message, uo_wrap_hint, uo_flush, uo_redirect, uo_field_string): Remove. (ui_out_table_begin): Replace with ... (ui_out::table_begin): ... this. (ui_out_table_body): Replace with ... (ui_out::table_body): ... this. (ui_out_table_end): Replace with ... (ui_out::table_end): ... this. (ui_out_table_header): Replace with ... (ui_out::table_header): ... this. (ui_out_begin): Replace with ... (ui_out::begin): ... this. (ui_out_end): Replace with ... (ui_out::end): ... this. (ui_out_field_int): Replace with ... (ui_out::field_int): ... this. (ui_out_field_fmt_int): Replace with ... (ui_out::field_fmt_int): ... this. (ui_out_field_core_addr): Replace with ... (ui_out::field_core_addr): ... this. (ui_out_field_stream): Replace with ... (ui_out::field_stream): ... this. (ui_out_field_skip): Replace with ... (ui_out::field_skip): ... this. (ui_out_field_string): Replace with ... (ui_out::field_string): ... this. (ui_out_field_fmt): Replace with ... (ui_out::field_fmt): ... this. (ui_out_spaces): Replace with ... (ui_out::spaces): ... this. (ui_out_text): Replace with ... (ui_out::text): ... this. (ui_out_message): Replace with ... (ui_out::message): ... this. (ui_out_wrap_hint): Replace with ... (ui_out::wrap_hint): ... this. (ui_out_flush): Replace with ... (ui_out::flush): ... this. (ui_out_redirect): Replace with ... (ui_out::redirect): ... this. (ui_out_test_flags): Replace with ... (ui_out::test_flags): ... this. (ui_out_is_mi_like_p): Replace with ... (ui_out::is_mi_like_p): ... this. (verify_field): Replace with ... (ui_out::verify_field): ... this. (ui_out_query_field): Replace with ... (ui_out::query_table_field): ... this. (ui_out_data): Remove. (ui_out_new): Remove, replace with ... (ui_out::ui_out): ... this constructor. (do_cleanup_table_end, make_cleanup_ui_out_tuple_begin_end, do_cleanup_end, make_cleanup_ui_out_tuple_begin_end, make_cleanup_ui_out_list_begin_end): Update fallouts of struct ui_out -> class ui_out change. * cli-out.c (cli_out_data): Remove. (cli_uiout_dtor): Remove. (cli_table_begin): Replace with ... (cli_ui_out::do_table_begin): ... this new method. (cli_table_body): Replace with ... (cli_ui_out::do_table_body): ... this new method. (cli_table_end): Replace with ... (cli_ui_out::do_table_end): ... this new method. (cli_table_header): Replace with ... (cli_ui_out::do_table_header): ... this new method. (cli_begin): Replace with ... (cli_ui_out::do_begin): ... this new method. (cli_end): Replace with ... (cli_ui_out::do_end): ... this new method. (cli_field_int): Replace with ... (cli_ui_out::do_field_int): ... this new method. (cli_field_skip): Replace with ... (cli_ui_out::do_field_skip): ... this new method. (cli_field_string): Replace with ... (cli_ui_out::do_field_string): ... this new method. (cli_field_fmt): Replace with ... (cli_ui_out::do_field_fmt): ... this new method. (cli_spaces): Replace with ... (cli_ui_out::do_spaces): ... this new method. (cli_text): Replace with ... (cli_ui_out::do_text): ... this new method. (cli_message): Replace with ... (cli_ui_out::do_message): ... this new method. (cli_wrap_hint): Replace with ... (cli_ui_out::do_wrap_hint): ... this new method. (cli_flush): Replace with ... (cli_ui_out::do_flush): ... this new method. (cli_redirect): Replace with ... (cli_ui_out::do_redirect): ... this new method. (out_field_fmt): Replace with ... (cli_ui_out::out_field_fmt): ... this new method. (field_separator): Replace with ... (cli_ui_out::field_separator): ... this new method. (cli_out_set_stream): Replace with ... (cli_ui_out::set_stream): ... this new method. (cli_ui_out_impl): Remove. (cli_out_data_ctor): Remove. (cli_ui_out_impl::cli_ui_out_impl): New constructor. (cli_ui_out_impl::~cli_ui_out_impl): New destructor. (cli_out_new): Change return type to cli_ui_out *, instantiate a cli_ui_out. * cli-out.h (cli_ui_out_data): Remove, replace with class cli_ui_out. (class cli_ui_out): New class. (cli_ui_out_impl): Remove. (cli_out_data_ctor): Remove. (cli_out_new): Change return type to cli_ui_out*. (cli_out_set_stream): Remove. * cli/cli-interp.c (struct cli_interp) : Change type to cli_ui_out*. (cli_interpreter_resume): Adapt. (cli_interpreter_exec): Adapt. * mi/mi-out.c (mi_ui_out_data, mi_out_data): Remove. (mi_ui_out_impl): Remove. (mi_table_begin): Replace with ... (mi_ui_out::do_table_begin): ... this. (mi_table_body): Replace with ... (mi_ui_out::do_table_body): ... this. (mi_table_end): Replace with ... (mi_ui_out::do_table_end): ... this. (mi_table_header): Replace with ... (mi_ui_out::do_table_header): ... this. (mi_begin): Replace with ... (mi_ui_out::do_begin): ... this. (mi_end): Replace with ... (mi_ui_out::do_end): ... this. (mi_field_int): Replace with ... (mi_ui_out::do_field_int): ... this. (mi_field_skip): Replace with ... (mi_ui_out::do_field_skip): ... this. (mi_field_string): Replace with ... (mi_ui_out::do_field_string): ... this. (mi_field_fmt): Replace with ... (mi_ui_out::do_field_fmt): ... this. (mi_spaces): Replace with ... (mi_ui_out::do_spaces): ... this. (mi_text): Replace with ... (mi_ui_out::do_text): ... this. (mi_message): Replace with ... (mi_ui_out::do_message): ... this. (mi_wrap_hint): Replace with ... (mi_ui_out::do_wrap_hint): ... this. (mi_flush): Replace with ... (mi_ui_out::do_flush): ... this. (mi_redirect): Replace with ... (mi_ui_out::do_redirect): (field_separator): Replace with ... (mi_ui_out::field_separator): (mi_open): Replace with ... (mi_ui_out::open): ... this. (mi_close): Replace with ... (mi_ui_out::close): ... this. (mi_out_rewind): Replace with ... (mi_ui_out::rewind): ... this. (mi_out_put): Replace with ... (mi_ui_out::put): ... this. (mi_version): Replace with ... (mi_ui_out::version): ... this. (mi_out_data_ctor): Replace with ... (mi_ui_out::mi_ui_out): ... this. (mi_out_data_dtor): Replace with ... (mi_ui_out::~mi_ui_out): ... this. (mi_out_new): Change return type to mi_ui_out*, instantiate an mi_ui_out object. (as_mi_ui_out): New function. (mi_version): Update fallouts of struct ui_out to class ui_out transition. (mi_out_put): Likewise. (mi_out_rewind): Likewise. * mi/mi-out.h (mi_out_new): Change return type to mi_ui_out*. * tui/tui-out.c (tui_ui_out_data, tui_out_data, tui_ui_out_impl): Remove. (tui_field_int): Replace with ... (tui_ui_out::do_field_int): ... this. (tui_field_string): Replace with ... (tui_ui_out::do_field_string): ... this. (tui_field_fmt): Replace with ... (tui_ui_out::do_field_fmt): ... this. (tui_text): Replace with ... (tui_ui_out::do_text): ... this. (tui_out_new): Change return type to tui_ui_out*, instantiate tui_ui_out object. (tui_ui_out::tui_ui_out): New. * tui/tui-out.h: New file. * tui/tui.h (tui_out_new): Move declaration to tui/tui-out.h. * tui/tui-io.c: Include tui/tui-out.h. (tui_old_uiout): Change type to cli_ui_out*. (tui_setup_io): Use dynamic_cast. * tui/tui-io.h (tui_old_uiout): Change type to cli_ui_out*. * tui/tui-interp.c (tui_resume): Adapt. * ada-lang.c (print_it_exception): Update fallouts of struct ui_out to class ui_out transition. (print_one_exception): Likewise. (print_mention_exception): Likewise. * ada-tasks.c (print_ada_task_info): Likewise. (info_task): Likewise. (task_command): Likewise. * auto-load.c (print_script): Likewise. (auto_load_info_scripts): Likewise. (info_auto_load_cmd): Likewise. * break-catch-sig.c (signal_catchpoint_print_one): Likewise. * break-catch-syscall.c (print_it_catch_syscall): Likewise. (print_one_catch_syscall): Likewise. * break-catch-throw.c (print_it_exception_catchpoint): Likewise. (print_one_exception_catchpoint): Likewise. (print_one_detail_exception_catchpoint): Likewise. (print_mention_exception_catchpoint): Likewise. * breakpoint.c (maybe_print_thread_hit_breakpoint): Likewise. (print_solib_event): Likewise. (watchpoint_check): Likewise. (wrap_indent_at_field): Likewise. (print_breakpoint_location): Likewise. (output_thread_groups): Likewise. (print_one_breakpoint_location): Likewise. (breakpoint_1): Likewise. (default_collect_info): Likewise. (watchpoints_info): Likewise. (print_it_catch_fork): Likewise. (print_one_catch_fork): Likewise. (print_it_catch_vfork): Likewise. (print_one_catch_vfork): Likewise. (print_it_catch_solib): Likewise. (print_one_catch_solib): Likewise. (print_it_catch_exec): Likewise. (print_one_catch_exec): Likewise. (mention): Likewise. (print_it_ranged_breakpoint): Likewise. (print_one_ranged_breakpoint): Likewise. (print_one_detail_ranged_breakpoint): Likewise. (print_mention_ranged_breakpoint): Likewise. (print_it_watchpoint): Likewise. (print_mention_watchpoint): Likewise. (print_it_masked_watchpoint): Likewise. (print_one_detail_masked_watchpoint): Likewise. (print_mention_masked_watchpoint): Likewise. (bkpt_print_it): Likewise. (tracepoint_print_one_detail): Likewise. (tracepoint_print_mention): Likewise. (update_static_tracepoint): Likewise. (tracepoints_info): Likewise. (save_breakpoints): Likewise. * cli/cli-cmds.c (complete_command): Likewise. * cli/cli-logging.c (set_logging_redirect): Likewise. (pop_output_files): Likewise. (handle_redirections): Likewise. * cli/cli-script.c (print_command_lines): Likewise. * cli/cli-setshow.c (do_show_command): Likewise. (cmd_show_list): Likewise. * cp-abi.c (list_cp_abis): Likewise. (show_cp_abi_cmd): Likewise. * darwin-nat-info.c (darwin_debug_regions_recurse): Likewise. * disasm.c (gdb_pretty_print_insn): Likewise. (do_mixed_source_and_assembly_deprecated): Likewise. (do_mixed_source_and_assembly): Likewise. * gdb_bfd.c (print_one_bfd): Likewise. (maintenance_info_bfds): Likewise. * guile/scm-breakpoint.c (gdbscm_breakpoint_commands): Likewise. * guile/scm-ports.c (ioscm_with_output_to_port_worker): Likewise. * i386-linux-tdep.c (i386_linux_handle_segmentation_fault): Likewise. * i386-tdep.c (i386_mpx_print_bounds): Likewise. * infcmd.c (run_command_1): Likewise. (print_return_value_1): Likewise. * inferior.c (print_selected_inferior): Likewise. (print_inferior): Likewise. * infrun.c (print_end_stepping_range_reason): Likewise. (print_signal_exited_reason): Likewise. (print_exited_reason): Likewise. (print_signal_received_reason): Likewise. (print_no_history_reason): Likewise. * interps.c (interp_set): Likewise. * linespec.c (decode_line_full): Likewise. * linux-thread-db.c (info_auto_load_libthread_db): Likewise. * mi/mi-cmd-env.c (mi_cmd_env_pwd): Likewise. (mi_cmd_env_path): Likewise. (mi_cmd_env_dir): Likewise. (mi_cmd_inferior_tty_show): Likewise. * mi/mi-cmd-file.c (mi_cmd_file_list_exec_source_file): Likewise. (print_partial_file_name): Likewise. (mi_cmd_file_list_exec_source_files): Likewise. * mi/mi-cmd-info.c (mi_cmd_info_ada_exceptions): Likewise. (mi_cmd_info_gdb_mi_command): Likewise. * mi/mi-cmd-stack.c (mi_cmd_stack_info_depth): Likewise. (mi_cmd_stack_list_args): Likewise. (list_arg_or_local): Likewise. * mi/mi-cmd-var.c (print_varobj): Likewise. (mi_cmd_var_create): Likewise. (mi_cmd_var_delete): Likewise. (mi_cmd_var_set_format): Likewise. (mi_cmd_var_show_format): Likewise. (mi_cmd_var_info_num_children): Likewise. (mi_cmd_var_list_children): Likewise. (mi_cmd_var_info_type): Likewise. (mi_cmd_var_info_path_expression): Likewise. (mi_cmd_var_info_expression): Likewise. (mi_cmd_var_show_attributes): Likewise. (mi_cmd_var_evaluate_expression): Likewise. (mi_cmd_var_assign): Likewise. (varobj_update_one): Likewise. * mi/mi-interp.c (as_mi_interp): Likewise. (mi_on_normal_stop_1): Likewise. (mi_tsv_modified): Likewise. (mi_breakpoint_created): Likewise. (mi_breakpoint_modified): Likewise. (mi_solib_loaded): Likewise. (mi_solib_unloaded): Likewise. (mi_command_param_changed): Likewise. (mi_memory_changed): Likewise. (mi_user_selected_context_changed): Likewise. * mi/mi-main.c (print_one_inferior): Likewise. (output_cores): Likewise. (list_available_thread_groups): Likewise. (mi_cmd_data_list_register_names): Likewise. (mi_cmd_data_list_changed_registers): Likewise. (output_register): Likewise. (mi_cmd_data_evaluate_expression): Likewise. (mi_cmd_data_read_memory): Likewise. (mi_cmd_data_read_memory_bytes): Likewise. (mi_cmd_list_features): Likewise. (mi_cmd_list_target_features): Likewise. (mi_cmd_add_inferior): Likewise. (mi_execute_command): Likewise. (mi_load_progress): Likewise. (print_variable_or_computed): Likewise. (mi_cmd_trace_frame_collected): Likewise. * mi/mi-symbol-cmds.c (mi_cmd_symbol_list_lines): Likewise. * osdata.c (info_osdata_command): Likewise. * probe.c (gen_ui_out_table_header_info): Likewise. (print_ui_out_not_applicables): Likewise. (print_ui_out_info): Likewise. (info_probes_for_ops): Likewise. (enable_probes_command): Likewise. (disable_probes_command): Likewise. * progspace.c (print_program_space): Likewise. * python/py-breakpoint.c (bppy_get_commands): Likewise. * python/py-framefilter.c (py_print_type): Likewise. (py_print_value): Likewise. (py_print_single_arg): Likewise. (enumerate_args): Likewise. (enumerate_locals): Likewise. (py_print_args): Likewise. (py_print_frame): Likewise. * record-btrace.c (btrace_ui_out_decode_error): Likewise. (btrace_call_history_insn_range): Likewise. (btrace_call_history_src_line): Likewise. (btrace_call_history): Likewise. * remote.c (show_remote_cmd): Likewise. * skip.c (skip_info): Likewise. * solib.c (info_sharedlibrary_command): Likewise. * source.c (print_source_lines_base): Likewise. * spu-tdep.c (info_spu_event_command): Likewise. (info_spu_signal_command): Likewise. (info_spu_mailbox_list): Likewise. (info_spu_dma_cmdlist): Likewise. (info_spu_dma_command): Likewise. (info_spu_proxydma_command): Likewise. * stack.c (print_stack_frame): Likewise. (print_frame_arg): Likewise. (read_frame_arg): Likewise. (print_frame_args): Likewise. (print_frame_info): Likewise. (print_frame): Likewise. * symfile.c (load_progress): Likewise. (generic_load): Likewise. (print_transfer_performance): Likewise. * thread.c (do_captured_list_thread_ids): Likewise. (print_thread_info_1): Likewise. (restore_selected_frame): Likewise. (do_captured_thread_select): Likewise. (print_selected_thread_frame): Likewise. * top.c (execute_command_to_string): Likewise. * tracepoint.c (tvariables_info_1): Likewise. (trace_status_mi): Likewise. (tfind_1): Likewise. (print_one_static_tracepoint_marker): Likewise. (info_static_tracepoint_markers_command): Likewise. * utils.c (do_ui_out_redirect_pop): Likewise. (fputs_maybe_filtered): Likewise. --- gdb/ui-out.c | 503 ++++++++++++++++------------------------------------------- 1 file changed, 133 insertions(+), 370 deletions(-) (limited to 'gdb/ui-out.c') diff --git a/gdb/ui-out.c b/gdb/ui-out.c index 774be2cb715..9fe72ddf3bf 100644 --- a/gdb/ui-out.c +++ b/gdb/ui-out.c @@ -28,6 +28,7 @@ #include #include #include +#include /* A header of a ui_out_table. */ @@ -306,172 +307,113 @@ int ui_out_table::entry_level () const return m_entry_level; } -/* The ui_out structure */ - -struct ui_out - { - int flags; - /* Specific implementation of ui-out. */ - const struct ui_out_impl *impl; - void *data; - - /* Vector to store and track the ui-out levels. */ - std::vector> levels; - - int level () const - { - return this->levels.size (); - } - - /* A table, if any. At present only a single table is supported. */ - std::unique_ptr table; - }; +int +ui_out::level () const +{ + return m_levels.size (); +} /* The current (inner most) level. */ -static ui_out_level * -current_level (struct ui_out *uiout) + +ui_out_level * +ui_out::current_level () const { - return uiout->levels.back ().get (); + return m_levels.back ().get (); } /* Create a new level, of TYPE. */ -static void -push_level (struct ui_out *uiout, - enum ui_out_type type) +void +ui_out::push_level (ui_out_type type) { std::unique_ptr level (new ui_out_level (type)); - uiout->levels.push_back (std::move (level)); + m_levels.push_back (std::move (level)); } /* Discard the current level. TYPE is the type of the level being discarded. */ -static void -pop_level (struct ui_out *uiout, - enum ui_out_type type) +void +ui_out::pop_level (ui_out_type type) { /* We had better not underflow the buffer. */ - gdb_assert (uiout->level () > 0); - gdb_assert (current_level (uiout)->type () == type); - - uiout->levels.pop_back (); -} - -/* These are the interfaces to implementation functions. */ - -static void uo_table_begin (struct ui_out *uiout, int nbrofcols, - int nr_rows, const char *tblid); -static void uo_table_body (struct ui_out *uiout); -static void uo_table_end (struct ui_out *uiout); -static void uo_table_header (struct ui_out *uiout, int width, - enum ui_align align, - const std::string &col_name, - const std::string &col_hdr); -static void uo_begin (struct ui_out *uiout, - enum ui_out_type type, - const char *id); -static void uo_end (struct ui_out *uiout, - enum ui_out_type type); -static void uo_field_int (struct ui_out *uiout, int fldno, int width, - enum ui_align align, const char *fldname, int value); -static void uo_field_skip (struct ui_out *uiout, int fldno, int width, - enum ui_align align, const char *fldname); -static void uo_field_fmt (struct ui_out *uiout, int fldno, int width, - enum ui_align align, const char *fldname, - const char *format, va_list args) - ATTRIBUTE_PRINTF (6, 0); -static void uo_spaces (struct ui_out *uiout, int numspaces); -static void uo_text (struct ui_out *uiout, const char *string); -static void uo_message (struct ui_out *uiout, - const char *format, va_list args) - ATTRIBUTE_PRINTF (2, 0); -static void uo_wrap_hint (struct ui_out *uiout, const char *identstring); -static void uo_flush (struct ui_out *uiout); -static int uo_redirect (struct ui_out *uiout, struct ui_file *outstream); - -/* Prototypes for local functions */ - -static void verify_field (struct ui_out *uiout, int *fldno, int *width, - enum ui_align *align); - -/* exported functions (ui_out API) */ + gdb_assert (m_levels.size () > 0); + gdb_assert (current_level ()->type () == type); + + m_levels.pop_back (); +} /* Mark beginning of a table. */ -static void -ui_out_table_begin (struct ui_out *uiout, int nr_cols, - int nr_rows, const std::string &tblid) +void +ui_out::table_begin (int nr_cols, int nr_rows, const std::string &tblid) { - if (uiout->table != nullptr) + if (m_table_up != nullptr) internal_error (__FILE__, __LINE__, _("tables cannot be nested; table_begin found before \ previous table_end.")); - uiout->table.reset ( - new ui_out_table (uiout->level () + 1, nr_cols, tblid)); + m_table_up.reset (new ui_out_table (level () + 1, nr_cols, tblid)); - uo_table_begin (uiout, nr_cols, nr_rows, tblid.c_str ()); + do_table_begin (nr_cols, nr_rows, tblid.c_str ()); } void -ui_out_table_body (struct ui_out *uiout) +ui_out::table_header (int width, ui_align alignment, + const std::string &col_name, const std::string &col_hdr) { - if (uiout->table == nullptr) + if (m_table_up == nullptr) internal_error (__FILE__, __LINE__, - _("table_body outside a table is not valid; it must be \ -after a table_begin and before a table_end.")); + _("table_header outside a table is not valid; it must be \ +after a table_begin and before a table_body.")); - uiout->table->start_body (); + m_table_up->append_header (width, alignment, col_name, col_hdr); - uo_table_body (uiout); + do_table_header (width, alignment, col_name, col_hdr); } -static void -ui_out_table_end (struct ui_out *uiout) +void +ui_out::table_body () { - if (uiout->table == nullptr) + if (m_table_up == nullptr) internal_error (__FILE__, __LINE__, - _("misplaced table_end or missing table_begin.")); + _("table_body outside a table is not valid; it must be " + "after a table_begin and before a table_end.")); - uo_table_end (uiout); + m_table_up->start_body (); - uiout->table = nullptr; + do_table_body (); } void -ui_out_table_header (struct ui_out *uiout, int width, enum ui_align alignment, - const std::string &col_name, const std::string &col_hdr) +ui_out::table_end () { - if (uiout->table == nullptr) + if (m_table_up == nullptr) internal_error (__FILE__, __LINE__, - _("table_header outside a table is not valid; it must be " - "after a table_begin and before a table_body.")); + _("misplaced table_end or missing table_begin.")); - uiout->table->append_header (width, alignment, col_name, col_hdr); + do_table_end (); - uo_table_header (uiout, width, alignment, col_name, col_hdr); + m_table_up = nullptr; } static void do_cleanup_table_end (void *data) { - struct ui_out *ui_out = (struct ui_out *) data; + ui_out *uiout = (ui_out *) data; - ui_out_table_end (ui_out); + uiout->table_end (); } struct cleanup * -make_cleanup_ui_out_table_begin_end (struct ui_out *ui_out, int nr_cols, - int nr_rows, const char *tblid) +make_cleanup_ui_out_table_begin_end (ui_out *uiout, int nr_cols, int nr_rows, + const char *tblid) { - ui_out_table_begin (ui_out, nr_cols, nr_rows, tblid); - return make_cleanup (do_cleanup_table_end, ui_out); + uiout->table_begin (nr_cols, nr_rows, tblid); + return make_cleanup (do_cleanup_table_end, uiout); } void -ui_out_begin (struct ui_out *uiout, - enum ui_out_type type, - const char *id) +ui_out::begin (ui_out_type type, const char *id) { /* Be careful to verify the ``field'' before the new tuple/list is pushed onto the stack. That way the containing list/table/row is @@ -483,30 +425,29 @@ ui_out_begin (struct ui_out *uiout, { int fldno; int width; - enum ui_align align; + ui_align align; - verify_field (uiout, &fldno, &width, &align); + verify_field (&fldno, &width, &align); } - push_level (uiout, type); + push_level (type); /* If the push puts us at the same level as a table row entry, we've got a new table row. Put the header pointer back to the start. */ - if (uiout->table != nullptr - && uiout->table->current_state () == ui_out_table::state::BODY - && uiout->table->entry_level () == uiout->level ()) - uiout->table->start_row (); + if (m_table_up != nullptr + && m_table_up->current_state () == ui_out_table::state::BODY + && m_table_up->entry_level () == level ()) + m_table_up->start_row (); - uo_begin (uiout, type, id); + do_begin (type, id); } void -ui_out_end (struct ui_out *uiout, - enum ui_out_type type) +ui_out::end (ui_out_type type) { - pop_level (uiout, type); + pop_level (type); - uo_end (uiout, type); + do_end (type); } struct ui_out_end_cleanup_data @@ -521,7 +462,7 @@ do_cleanup_end (void *data) struct ui_out_end_cleanup_data *end_cleanup_data = (struct ui_out_end_cleanup_data *) data; - ui_out_end (end_cleanup_data->uiout, end_cleanup_data->type); + end_cleanup_data->uiout->end (end_cleanup_data->type); xfree (end_cleanup_data); } @@ -541,7 +482,7 @@ struct cleanup * make_cleanup_ui_out_tuple_begin_end (struct ui_out *uiout, const char *id) { - ui_out_begin (uiout, ui_out_type_tuple, id); + uiout->begin (ui_out_type_tuple, id); return make_cleanup_ui_out_end (uiout, ui_out_type_tuple); } @@ -549,331 +490,166 @@ struct cleanup * make_cleanup_ui_out_list_begin_end (struct ui_out *uiout, const char *id) { - ui_out_begin (uiout, ui_out_type_list, id); + uiout->begin (ui_out_type_list, id); return make_cleanup_ui_out_end (uiout, ui_out_type_list); } void -ui_out_field_int (struct ui_out *uiout, - const char *fldname, - int value) +ui_out::field_int (const char *fldname, int value) { int fldno; int width; - enum ui_align align; + ui_align align; - verify_field (uiout, &fldno, &width, &align); + verify_field (&fldno, &width, &align); - uo_field_int (uiout, fldno, width, align, fldname, value); + do_field_int (fldno, width, align, fldname, value); } void -ui_out_field_fmt_int (struct ui_out *uiout, - int input_width, - enum ui_align input_align, - const char *fldname, - int value) +ui_out::field_fmt_int (int input_width, ui_align input_align, + const char *fldname, int value) { int fldno; int width; - enum ui_align align; + ui_align align; - verify_field (uiout, &fldno, &width, &align); + verify_field (&fldno, &width, &align); - uo_field_int (uiout, fldno, input_width, input_align, fldname, value); + do_field_int (fldno, input_width, input_align, fldname, value); } /* Documented in ui-out.h. */ void -ui_out_field_core_addr (struct ui_out *uiout, - const char *fldname, - struct gdbarch *gdbarch, - CORE_ADDR address) +ui_out::field_core_addr (const char *fldname, struct gdbarch *gdbarch, + CORE_ADDR address) { - ui_out_field_string (uiout, fldname, - print_core_address (gdbarch, address)); + field_string (fldname, print_core_address (gdbarch, address)); } void -ui_out_field_stream (struct ui_out *uiout, - const char *fldname, - struct ui_file *stream) +ui_out::field_stream (const char *fldname, ui_file *stream) { std::string buffer = ui_file_as_string (stream); if (!buffer.empty ()) - ui_out_field_string (uiout, fldname, buffer.c_str ()); + field_string (fldname, buffer.c_str ()); else - ui_out_field_skip (uiout, fldname); + field_skip (fldname); ui_file_rewind (stream); } /* Used to omit a field. */ void -ui_out_field_skip (struct ui_out *uiout, - const char *fldname) +ui_out::field_skip (const char *fldname) { int fldno; int width; - enum ui_align align; + ui_align align; - verify_field (uiout, &fldno, &width, &align); + verify_field (&fldno, &width, &align); - uo_field_skip (uiout, fldno, width, align, fldname); + do_field_skip (fldno, width, align, fldname); } void -ui_out_field_string (struct ui_out *uiout, - const char *fldname, - const char *string) +ui_out::field_string (const char *fldname, const char *string) { int fldno; int width; - enum ui_align align; + ui_align align; - verify_field (uiout, &fldno, &width, &align); + verify_field (&fldno, &width, &align); - uo_field_string (uiout, fldno, width, align, fldname, string); + do_field_string (fldno, width, align, fldname, string); } /* VARARGS */ void -ui_out_field_fmt (struct ui_out *uiout, - const char *fldname, - const char *format, ...) +ui_out::field_fmt (const char *fldname, const char *format, ...) { va_list args; int fldno; int width; - enum ui_align align; + ui_align align; /* Will not align, but has to call anyway. */ - verify_field (uiout, &fldno, &width, &align); + verify_field (&fldno, &width, &align); va_start (args, format); - uo_field_fmt (uiout, fldno, width, align, fldname, format, args); + do_field_fmt (fldno, width, align, fldname, format, args); va_end (args); } void -ui_out_spaces (struct ui_out *uiout, int numspaces) +ui_out::spaces (int numspaces) { - uo_spaces (uiout, numspaces); + do_spaces (numspaces); } void -ui_out_text (struct ui_out *uiout, - const char *string) +ui_out::text (const char *string) { - uo_text (uiout, string); + do_text (string); } void -ui_out_message (struct ui_out *uiout, const char *format, ...) +ui_out::message (const char *format, ...) { va_list args; va_start (args, format); - uo_message (uiout, format, args); + do_message (format, args); va_end (args); } void -ui_out_wrap_hint (struct ui_out *uiout, const char *identstring) +ui_out::wrap_hint (const char *identstring) { - uo_wrap_hint (uiout, identstring); + do_wrap_hint (identstring); } void -ui_out_flush (struct ui_out *uiout) +ui_out::flush () { - uo_flush (uiout); + do_flush (); } int -ui_out_redirect (struct ui_out *uiout, struct ui_file *outstream) +ui_out::redirect (ui_file *outstream) { - return uo_redirect (uiout, outstream); + return do_redirect (outstream); } /* Test the flags against the mask given. */ -int -ui_out_test_flags (struct ui_out *uiout, ui_out_flags mask) -{ - return (uiout->flags & mask); -} - -int -ui_out_is_mi_like_p (struct ui_out *uiout) -{ - return uiout->impl->is_mi_like_p; -} - -/* Interface to the implementation functions. */ - -void -uo_table_begin (struct ui_out *uiout, int nbrofcols, - int nr_rows, - const char *tblid) -{ - if (!uiout->impl->table_begin) - return; - uiout->impl->table_begin (uiout, nbrofcols, nr_rows, tblid); -} - -void -uo_table_body (struct ui_out *uiout) -{ - if (!uiout->impl->table_body) - return; - uiout->impl->table_body (uiout); -} - -void -uo_table_end (struct ui_out *uiout) -{ - if (!uiout->impl->table_end) - return; - uiout->impl->table_end (uiout); -} - -void -uo_table_header (struct ui_out *uiout, int width, enum ui_align align, - const std::string &col_name, const std::string &col_hdr) -{ - if (!uiout->impl->table_header) - return; - uiout->impl->table_header (uiout, width, align, col_name, col_hdr); -} - -void -uo_begin (struct ui_out *uiout, - enum ui_out_type type, - const char *id) -{ - if (uiout->impl->begin == NULL) - return; - uiout->impl->begin (uiout, type, id); -} - -void -uo_end (struct ui_out *uiout, - enum ui_out_type type) -{ - if (uiout->impl->end == NULL) - return; - uiout->impl->end (uiout, type); -} - -void -uo_field_int (struct ui_out *uiout, int fldno, int width, enum ui_align align, - const char *fldname, - int value) -{ - if (!uiout->impl->field_int) - return; - uiout->impl->field_int (uiout, fldno, width, align, fldname, value); -} - -void -uo_field_skip (struct ui_out *uiout, int fldno, int width, enum ui_align align, - const char *fldname) -{ - if (!uiout->impl->field_skip) - return; - uiout->impl->field_skip (uiout, fldno, width, align, fldname); -} - -void -uo_field_string (struct ui_out *uiout, int fldno, int width, - enum ui_align align, - const char *fldname, - const char *string) +ui_out_flags +ui_out::test_flags (ui_out_flags mask) { - if (!uiout->impl->field_string) - return; - uiout->impl->field_string (uiout, fldno, width, align, fldname, string); + return m_flags & mask; } -void -uo_field_fmt (struct ui_out *uiout, int fldno, int width, enum ui_align align, - const char *fldname, - const char *format, - va_list args) -{ - if (!uiout->impl->field_fmt) - return; - uiout->impl->field_fmt (uiout, fldno, width, align, fldname, format, args); -} - -void -uo_spaces (struct ui_out *uiout, int numspaces) -{ - if (!uiout->impl->spaces) - return; - uiout->impl->spaces (uiout, numspaces); -} - -void -uo_text (struct ui_out *uiout, - const char *string) -{ - if (!uiout->impl->text) - return; - uiout->impl->text (uiout, string); -} - -void -uo_message (struct ui_out *uiout, - const char *format, - va_list args) +bool +ui_out::is_mi_like_p () { - if (!uiout->impl->message) - return; - uiout->impl->message (uiout, format, args); -} - -void -uo_wrap_hint (struct ui_out *uiout, const char *identstring) -{ - if (!uiout->impl->wrap_hint) - return; - uiout->impl->wrap_hint (uiout, identstring); -} - -void -uo_flush (struct ui_out *uiout) -{ - if (!uiout->impl->flush) - return; - uiout->impl->flush (uiout); -} - -int -uo_redirect (struct ui_out *uiout, struct ui_file *outstream) -{ - if (!uiout->impl->redirect) - return -1; - return uiout->impl->redirect (uiout, outstream); + return do_is_mi_like_p (); } /* Verify that the field/tuple/list is correctly positioned. Return the field number and corresponding alignment (if available/applicable). */ -static void -verify_field (struct ui_out *uiout, int *fldno, int *width, - enum ui_align *align) +void +ui_out::verify_field (int *fldno, int *width, ui_align *align) { - ui_out_level *current = current_level (uiout); + ui_out_level *current = current_level (); const char *text; - if (uiout->table != nullptr - && uiout->table->current_state () != ui_out_table::state::BODY) + if (m_table_up != nullptr + && m_table_up->current_state () != ui_out_table::state::BODY) { internal_error (__FILE__, __LINE__, _("table_body missing; table fields must be \ @@ -882,10 +658,10 @@ specified after table_body and inside a list.")); current->inc_field_count (); - if (uiout->table != nullptr - && uiout->table->current_state () == ui_out_table::state::BODY - && uiout->table->entry_level () == uiout->level () - && uiout->table->get_next_header (fldno, width, align, &text)) + if (m_table_up != nullptr + && m_table_up->current_state () == ui_out_table::state::BODY + && m_table_up->entry_level () == level () + && m_table_up->get_next_header (fldno, width, align, &text)) { if (*fldno != current->field_count ()) internal_error (__FILE__, __LINE__, @@ -899,40 +675,27 @@ specified after table_body and inside a list.")); } } - -/* Access to ui-out members data. */ - -void * -ui_out_data (struct ui_out *uiout) -{ - return uiout->data; -} - /* Access table field parameters. */ -int -ui_out_query_field (struct ui_out *uiout, int colno, - int *width, int *alignment, const char **col_name) + +bool +ui_out::query_table_field (int colno, int *width, int *alignment, + const char **col_name) { - if (uiout->table == nullptr) - return 0; + if (m_table_up == nullptr) + return false; - return uiout->table->query_field (colno, width, alignment, col_name); + return m_table_up->query_field (colno, width, alignment, col_name); } -/* Initialize private members at startup. */ +/* The constructor. */ -struct ui_out * -ui_out_new (const struct ui_out_impl *impl, void *data, - ui_out_flags flags) +ui_out::ui_out (ui_out_flags flags) +: m_flags (flags) { - struct ui_out *uiout = new ui_out (); - - uiout->data = data; - uiout->impl = impl; - uiout->flags = flags; - /* Create the ui-out level #1, the default level. */ - push_level (uiout, ui_out_type_tuple); + push_level (ui_out_type_tuple); +} - return uiout; +ui_out::~ui_out () +{ } -- cgit v1.2.1