# HG changeset patch # User John W. Eaton # Date 1711165893 14400 # Node ID 775dde0cb3e570897704ad74e8ef5c6145ba6a76 # Parent ddd3da3f1a7015df278be8fcbc6dfb76c57c2c21 use qOverload(fptr) instead of QOverload::of(fptr) Now that we are using C++17 we can use qOverload (function_pointer) instead of QOverload::of (function_pointer) Affected files: ButtonGroup.cc, Container.cc, Figure.cc, PopupMenuControl.cc, command-widget.cc, documentation-bookmarks.cc, file-editor-tab.cc, file-editor.cc, main-window.cc, octave-qobject.cc, octave-qobject.h, set-path-dialog.cc, settings-dialog.cc, terminal-dock-widget.cc, and variable-editor.cc. diff -r ddd3da3f1a70 -r 775dde0cb3e5 libgui/graphics/ButtonGroup.cc --- a/libgui/graphics/ButtonGroup.cc Fri Mar 22 17:31:33 2024 -0700 +++ b/libgui/graphics/ButtonGroup.cc Fri Mar 22 23:51:33 2024 -0400 @@ -179,7 +179,7 @@ frame->hide (); connect (m_buttongroup, - QOverload::of (&QButtonGroup::buttonClicked), + qOverload (&QButtonGroup::buttonClicked), this, &ButtonGroup::buttonClicked); } diff -r ddd3da3f1a70 -r 775dde0cb3e5 libgui/graphics/Container.cc --- a/libgui/graphics/Container.cc Fri Mar 22 17:31:33 2024 -0700 +++ b/libgui/graphics/Container.cc Fri Mar 22 23:51:33 2024 -0400 @@ -69,11 +69,11 @@ m_canvas = Canvas::create (m_interpreter, gh, this, fig.get ("renderer").string_value ()); - connect (m_canvas, QOverload::of (&Canvas::interpreter_event), - this, QOverload::of (&Container::interpreter_event)); + connect (m_canvas, qOverload (&Canvas::interpreter_event), + this, qOverload (&Container::interpreter_event)); - connect (m_canvas, QOverload::of (&Canvas::interpreter_event), - this, QOverload::of (&Container::interpreter_event)); + connect (m_canvas, qOverload (&Canvas::interpreter_event), + this, qOverload (&Container::interpreter_event)); connect (m_canvas, SIGNAL (gh_callback_event (const graphics_handle&, diff -r ddd3da3f1a70 -r 775dde0cb3e5 libgui/graphics/Figure.cc --- a/libgui/graphics/Figure.cc Fri Mar 22 17:31:33 2024 -0700 +++ b/libgui/graphics/Figure.cc Fri Mar 22 23:51:33 2024 -0400 @@ -122,11 +122,11 @@ m_container = new Container (win, interp); win->setCentralWidget (m_container); - connect (m_container, QOverload::of (&Container::interpreter_event), - this, QOverload::of (&Figure::interpreter_event)); + connect (m_container, qOverload (&Container::interpreter_event), + this, qOverload (&Figure::interpreter_event)); - connect (m_container, QOverload::of (&Container::interpreter_event), - this, QOverload::of (&Figure::interpreter_event)); + connect (m_container, qOverload (&Container::interpreter_event), + this, qOverload (&Figure::interpreter_event)); figure::properties& fp = properties
(); diff -r ddd3da3f1a70 -r 775dde0cb3e5 libgui/graphics/PopupMenuControl.cc --- a/libgui/graphics/PopupMenuControl.cc Fri Mar 22 17:31:33 2024 -0700 +++ b/libgui/graphics/PopupMenuControl.cc Fri Mar 22 23:51:33 2024 -0400 @@ -64,7 +64,7 @@ update (uicontrol::properties::ID_VALUE); - connect (box, QOverload::of (&QComboBox::activated), + connect (box, qOverload (&QComboBox::activated), this, &PopupMenuControl::currentIndexChanged); } diff -r ddd3da3f1a70 -r 775dde0cb3e5 libgui/src/command-widget.cc --- a/libgui/src/command-widget.cc Fri Mar 22 17:31:33 2024 -0700 +++ b/libgui/src/command-widget.cc Fri Mar 22 23:51:33 2024 -0400 @@ -90,11 +90,11 @@ connect (this, &command_widget::new_command_line_signal, m_console, &console::new_command_line); - connect (m_console, QOverload::of (&console::interpreter_event), - this, QOverload::of (&command_widget::interpreter_event)); + connect (m_console, qOverload (&console::interpreter_event), + this, qOverload (&command_widget::interpreter_event)); - connect (m_console, QOverload::of (&console::interpreter_event), - this, QOverload::of (&command_widget::interpreter_event)); + connect (m_console, qOverload (&console::interpreter_event), + this, qOverload (&command_widget::interpreter_event)); insert_interpreter_output ("\n\n Welcome to Octave\n\n"); diff -r ddd3da3f1a70 -r 775dde0cb3e5 libgui/src/documentation-bookmarks.cc --- a/libgui/src/documentation-bookmarks.cc Fri Mar 22 17:31:33 2024 -0700 +++ b/libgui/src/documentation-bookmarks.cc Fri Mar 22 23:51:33 2024 -0400 @@ -340,7 +340,7 @@ } menu.addAction (tr ("&Add Folder"), this, - QOverload::of (&documentation_bookmarks::add_folder)); + qOverload (&documentation_bookmarks::add_folder)); menu.addSeparator (); diff -r ddd3da3f1a70 -r 775dde0cb3e5 libgui/src/m-editor/file-editor-tab.cc --- a/libgui/src/m-editor/file-editor-tab.cc Fri Mar 22 17:31:33 2024 -0700 +++ b/libgui/src/m-editor/file-editor-tab.cc Fri Mar 22 23:51:33 2024 -0400 @@ -232,11 +232,11 @@ // Any interpreter_event signal from a file_editor_tab_widget is // handled the same as for the parent main_window object. - connect (m_edit_area, QOverload::of (&octave_qscintilla::interpreter_event), - this, QOverload::of (&file_editor_tab::interpreter_event)); - - connect (m_edit_area, QOverload::of (&octave_qscintilla::interpreter_event), - this, QOverload::of (&file_editor_tab::interpreter_event)); + connect (m_edit_area, qOverload (&octave_qscintilla::interpreter_event), + this, qOverload (&file_editor_tab::interpreter_event)); + + connect (m_edit_area, qOverload (&octave_qscintilla::interpreter_event), + this, qOverload (&file_editor_tab::interpreter_event)); // connect modified signal connect (m_edit_area, SIGNAL (modificationChanged (bool)), diff -r ddd3da3f1a70 -r 775dde0cb3e5 libgui/src/m-editor/file-editor.cc --- a/libgui/src/m-editor/file-editor.cc Fri Mar 22 17:31:33 2024 -0700 +++ b/libgui/src/m-editor/file-editor.cc Fri Mar 22 23:51:33 2024 -0400 @@ -2629,8 +2629,8 @@ connect (this, &file_editor::fetab_change_request, f, &file_editor_tab::change_editor_state); - connect (this, QOverload::of (&file_editor::fetab_save_file), - f, QOverload::of (&file_editor_tab::save_file)); + connect (this, qOverload (&file_editor::fetab_save_file), + f, qOverload (&file_editor_tab::save_file)); // Signals from the file_editor trivial operations connect (this, &file_editor::fetab_recover_from_exit, @@ -2652,11 +2652,11 @@ connect (this, &file_editor::fetab_context_edit, f, &file_editor_tab::context_edit); - connect (this, QOverload::of (&file_editor::fetab_save_file), - f, QOverload::of (&file_editor_tab::save_file)); + connect (this, qOverload (&file_editor::fetab_save_file), + f, qOverload (&file_editor_tab::save_file)); connect (this, &file_editor::fetab_save_file_as, - f, QOverload::of (&file_editor_tab::save_file_as)); + f, qOverload (&file_editor_tab::save_file_as)); connect (this, &file_editor::fetab_print_file, f, &file_editor_tab::print_file); @@ -2756,11 +2756,11 @@ // Any interpreter_event signal from a file_editor_tab_widget is // handled the same as for the parent main_window object. - connect (f, QOverload::of (&file_editor_tab::interpreter_event), - this, QOverload::of (&file_editor::interpreter_event)); - - connect (f, QOverload::of (&file_editor_tab::interpreter_event), - this, QOverload::of (&file_editor::interpreter_event)); + connect (f, qOverload (&file_editor_tab::interpreter_event), + this, qOverload (&file_editor::interpreter_event)); + + connect (f, qOverload (&file_editor_tab::interpreter_event), + this, qOverload (&file_editor::interpreter_event)); return f; } diff -r ddd3da3f1a70 -r 775dde0cb3e5 libgui/src/main-window.cc --- a/libgui/src/main-window.cc Fri Mar 22 17:31:33 2024 -0700 +++ b/libgui/src/main-window.cc Fri Mar 22 23:51:33 2024 -0400 @@ -258,7 +258,7 @@ this, &main_window::edit_mfile); connect (cmd_widget, &QTerminal::request_open_file_signal, - this, QOverload::of (&main_window::open_file_signal)); + this, qOverload (&main_window::open_file_signal)); connect (cmd_widget, &QTerminal::set_screen_size_signal, this, &main_window::set_screen_size); @@ -289,7 +289,7 @@ make_dock_widget_connections (m_file_browser_window); connect (m_file_browser_window, &files_dock_widget::open_file, - this, QOverload::of (&main_window::open_file_signal)); + this, qOverload (&main_window::open_file_signal)); connect (m_file_browser_window, &files_dock_widget::displayed_directory_changed, this, &main_window::set_current_working_directory); @@ -357,7 +357,7 @@ &workspace_model::show_symbol_tooltip); connect (editor, &file_editor::request_settings_dialog, - this, QOverload::of (&main_window::process_settings_dialog_request)); + this, qOverload (&main_window::process_settings_dialog_request)); connect (editor, &file_editor::request_dbcont_signal, this, &main_window::debug_continue); @@ -1894,11 +1894,11 @@ // Any interpreter_event signal from a set_path_dialog object is // handled the same as for the main_window object. - connect (m_set_path_dlg, QOverload::of (&set_path_dialog::interpreter_event), - this, QOverload::of (&main_window::interpreter_event)); - - connect (m_set_path_dlg, QOverload::of (&set_path_dialog::interpreter_event), - this, QOverload::of (&main_window::interpreter_event)); + connect (m_set_path_dlg, qOverload (&set_path_dialog::interpreter_event), + this, qOverload (&main_window::interpreter_event)); + + connect (m_set_path_dlg, qOverload (&set_path_dialog::interpreter_event), + this, qOverload (&main_window::interpreter_event)); connect (m_set_path_dlg, &set_path_dialog::modify_path_signal, this, &main_window::modify_path); @@ -1931,7 +1931,7 @@ m_file_browser_window, &files_dock_widget::set_current_directory); connect (m_find_files_dlg, &find_files_dialog::file_selected, - this, QOverload::of (&main_window::open_file_signal)); + this, qOverload (&main_window::open_file_signal)); m_find_files_dlg->setWindowModality (Qt::NonModal); } @@ -2166,11 +2166,11 @@ connect (qt_link, &qt_interpreter_events::file_remove_signal, this, &main_window::file_remove_proxy); - connect (this, QOverload::of (&main_window::interpreter_event), - &m_octave_qobj, QOverload::of (&base_qobject::interpreter_event)); - - connect (this, QOverload::of (&main_window::interpreter_event), - &m_octave_qobj, QOverload::of (&base_qobject::interpreter_event)); + connect (this, qOverload (&main_window::interpreter_event), + &m_octave_qobj, qOverload (&base_qobject::interpreter_event)); + + connect (this, qOverload (&main_window::interpreter_event), + &m_octave_qobj, qOverload (&base_qobject::interpreter_event)); configure_shortcuts (); } diff -r ddd3da3f1a70 -r 775dde0cb3e5 libgui/src/octave-qobject.cc --- a/libgui/src/octave-qobject.cc Fri Mar 22 17:31:33 2024 -0700 +++ b/libgui/src/octave-qobject.cc Fri Mar 22 23:51:33 2024 -0400 @@ -239,11 +239,11 @@ // Handle any interpreter_event signal from the octave_qapplication // object here. - connect (m_qapplication, QOverload::of (&octave_qapplication::interpreter_event), - this, QOverload::of (&base_qobject::interpreter_event)); + connect (m_qapplication, qOverload (&octave_qapplication::interpreter_event), + this, qOverload (&base_qobject::interpreter_event)); - connect (m_qapplication, QOverload::of (&octave_qapplication::interpreter_event), - this, QOverload::of (&base_qobject::interpreter_event)); + connect (m_qapplication, qOverload (&octave_qapplication::interpreter_event), + this, qOverload (&base_qobject::interpreter_event)); if (m_app_context.experimental_terminal_widget ()) { diff -r ddd3da3f1a70 -r 775dde0cb3e5 libgui/src/octave-qobject.h --- a/libgui/src/octave-qobject.h Fri Mar 22 17:31:33 2024 -0700 +++ b/libgui/src/octave-qobject.h Fri Mar 22 23:51:33 2024 -0400 @@ -194,11 +194,11 @@ template void connect_interpreter_events (T *widget) { - connect (widget, QOverload::of (&T::interpreter_event), - this, QOverload::of (&base_qobject::interpreter_event)); + connect (widget, qOverload (&T::interpreter_event), + this, qOverload (&base_qobject::interpreter_event)); - connect (widget, QOverload::of (&T::interpreter_event), - this, QOverload::of (&base_qobject::interpreter_event)); + connect (widget, qOverload (&T::interpreter_event), + this, qOverload (&base_qobject::interpreter_event)); } public slots: diff -r ddd3da3f1a70 -r 775dde0cb3e5 libgui/src/set-path-dialog.cc --- a/libgui/src/set-path-dialog.cc Fri Mar 22 17:31:33 2024 -0700 +++ b/libgui/src/set-path-dialog.cc Fri Mar 22 23:51:33 2024 -0400 @@ -120,11 +120,11 @@ // Any interpreter_event signal from a set_path_model object is // handled the same as for the parent set_path_dialog object. - connect (model, QOverload::of (&set_path_model::interpreter_event), - this, QOverload::of (&set_path_dialog::interpreter_event)); + connect (model, qOverload (&set_path_model::interpreter_event), + this, qOverload (&set_path_dialog::interpreter_event)); - connect (model, QOverload::of (&set_path_model::interpreter_event), - this, QOverload::of (&set_path_dialog::interpreter_event)); + connect (model, qOverload (&set_path_model::interpreter_event), + this, qOverload (&set_path_dialog::interpreter_event)); m_path_list = new QListView (this); m_path_list->setWordWrap (false); diff -r ddd3da3f1a70 -r 775dde0cb3e5 libgui/src/settings-dialog.cc --- a/libgui/src/settings-dialog.cc Fri Mar 22 17:31:33 2024 -0700 +++ b/libgui/src/settings-dialog.cc Fri Mar 22 23:51:33 2024 -0400 @@ -437,7 +437,7 @@ { proxy_type->addItems (global_proxy_all_types); // Connect relevant signals for dis-/enabling some elements - connect (proxy_type, QOverload::of (&QComboBox::currentIndexChanged), + connect (proxy_type, qOverload (&QComboBox::currentIndexChanged), this, &settings_dialog::proxy_items_update); connect (use_proxy_server, &QCheckBox::toggled, this, &settings_dialog::proxy_items_update); diff -r ddd3da3f1a70 -r 775dde0cb3e5 libgui/src/terminal-dock-widget.cc --- a/libgui/src/terminal-dock-widget.cc Fri Mar 22 17:31:33 2024 -0700 +++ b/libgui/src/terminal-dock-widget.cc Fri Mar 22 23:51:33 2024 -0400 @@ -92,11 +92,11 @@ connect (this, &terminal_dock_widget::visibilityChanged, widget, &QTerminal::handle_visibility_changed); - connect (widget, QOverload::of (&QTerminal::interpreter_event), - this, QOverload::of (&terminal_dock_widget::interpreter_event)); + connect (widget, qOverload (&QTerminal::interpreter_event), + this, qOverload (&terminal_dock_widget::interpreter_event)); - connect (widget, QOverload::of (&QTerminal::interpreter_event), - this, QOverload::of (&terminal_dock_widget::interpreter_event)); + connect (widget, qOverload (&QTerminal::interpreter_event), + this, qOverload (&terminal_dock_widget::interpreter_event)); m_terminal = widget; } diff -r ddd3da3f1a70 -r 775dde0cb3e5 libgui/src/variable-editor.cc --- a/libgui/src/variable-editor.cc Fri Mar 22 17:31:33 2024 -0700 +++ b/libgui/src/variable-editor.cc Fri Mar 22 23:51:33 2024 -0400 @@ -1297,11 +1297,11 @@ // Any interpreter_event signal from a variable_editor_stack object is // handled the same as for the parent variable_editor object. - connect (stack, QOverload::of (&variable_editor_stack::interpreter_event), - this, QOverload::of (&variable_editor::interpreter_event)); + connect (stack, qOverload (&variable_editor_stack::interpreter_event), + this, qOverload (&variable_editor::interpreter_event)); - connect (stack, QOverload::of (&variable_editor_stack::interpreter_event), - this, QOverload::of (&variable_editor::interpreter_event)); + connect (stack, qOverload (&variable_editor_stack::interpreter_event), + this, qOverload (&variable_editor::interpreter_event)); connect (stack, &variable_editor_stack::edit_variable_signal, this, &variable_editor::edit_variable); @@ -1373,11 +1373,11 @@ // Any interpreter_event signal from a variable_editor_model object is // handled the same as for the parent variable_editor object. - connect (model, QOverload::of (&variable_editor_model::interpreter_event), - this, QOverload::of (&variable_editor::interpreter_event)); + connect (model, qOverload (&variable_editor_model::interpreter_event), + this, qOverload (&variable_editor::interpreter_event)); - connect (model, QOverload::of (&variable_editor_model::interpreter_event), - this, QOverload::of (&variable_editor::interpreter_event)); + connect (model, qOverload (&variable_editor_model::interpreter_event), + this, qOverload (&variable_editor::interpreter_event)); // Must supply a title for a QLabel to be created. Calling set_title() // more than once will add more QLabels. Could change octave_dock_widget