changeset 31626:9034cf663768

eliminate unnecessary base_qobject object from GUI dialog classes * dialog.h, dialog.cc (QUIWidgetCreator::m_octave_qobj): Delete unnecessary data member. (QUIWidgetCreator::QUIWidgetCreator, MessageDialog::MessageDialog, ListDialog::ListDialog, InputDialog::InputDialog, FileDialog::FileDialog): Eliminate base_qobject argument. Change all uses.
author John W. Eaton <jwe@octave.org>
date Sat, 03 Dec 2022 01:00:04 -0500
parents b26b768f23a4
children 1e270beb6982
files libgui/src/dialog.cc libgui/src/dialog.h libgui/src/qt-interpreter-events.cc
diffstat 3 files changed, 17 insertions(+), 25 deletions(-) [+]
line wrap: on
line diff
--- a/libgui/src/dialog.cc	Sat Dec 03 00:52:02 2022 -0500
+++ b/libgui/src/dialog.cc	Sat Dec 03 01:00:04 2022 -0500
@@ -44,13 +44,12 @@
 #include "dialog.h"
 #include "gui-preferences-global.h"
 #include "gui-settings.h"
-#include "octave-qobject.h"
 
 namespace octave
 {
-  QUIWidgetCreator::QUIWidgetCreator (base_qobject& oct_qobj)
-    : QObject (), m_octave_qobj (oct_qobj), m_dialog_result (-1),
-      m_dialog_button (), m_string_list (), m_list_index (), m_path_name ()
+  QUIWidgetCreator::QUIWidgetCreator ()
+    : QObject (), m_dialog_result (-1), m_dialog_button (),
+      m_string_list (), m_list_index (), m_path_name ()
   {
     connect (this, &QUIWidgetCreator::create_dialog,
              this, &QUIWidgetCreator::handle_create_dialog);
@@ -181,8 +180,7 @@
                                                const QStringList& role)
   {
     MessageDialog *message_dialog
-      = new MessageDialog (m_octave_qobj, message, title, icon,
-                           button, defbutton, role);
+      = new MessageDialog (message, title, icon, button, defbutton, role);
 
     connect (message_dialog, &MessageDialog::buttonClicked,
              this, &QUIWidgetCreator::dialog_button_clicked);
@@ -233,7 +231,7 @@
                                                  const QString& cancel_string)
   {
     ListDialog *list_dialog
-      = new ListDialog (m_octave_qobj, list, mode, wd, ht, initial,
+      = new ListDialog (list, mode, wd, ht, initial,
                         name, prompt, ok_string, cancel_string);
 
     connect (list_dialog, &ListDialog::finish_selection,
@@ -265,7 +263,7 @@
                                                     const QStringList& defaults)
   {
     InputDialog *input_dialog
-      = new InputDialog (m_octave_qobj, prompt, title, nr, nc, defaults);
+      = new InputDialog (prompt, title, nr, nc, defaults);
 
     connect (input_dialog, &InputDialog::finish_input,
              this, &QUIWidgetCreator::input_finished);
@@ -293,8 +291,7 @@
                                                    const QString& multimode)
   {
     FileDialog *file_dialog
-      = new FileDialog (m_octave_qobj, filters, title, filename,
-                        dirname, multimode);
+      = new FileDialog (filters, title, filename, dirname, multimode);
 
     connect (file_dialog, &FileDialog::finish_input,
              this, &QUIWidgetCreator::filedialog_finished);
@@ -317,7 +314,7 @@
     wake_all ();
   }
 
-  MessageDialog::MessageDialog (base_qobject&, const QString& message,
+  MessageDialog::MessageDialog (const QString& message,
                                 const QString& title, const QString& qsicon,
                                 const QStringList& qsbutton,
                                 const QString& defbutton,
@@ -389,7 +386,7 @@
       }
   }
 
-  ListDialog::ListDialog (base_qobject&, const QStringList& list,
+  ListDialog::ListDialog (const QStringList& list,
                           const QString& mode, int wd, int ht,
                           const QList<int>& initial, const QString& title,
                           const QStringList& prompt,
@@ -525,7 +522,7 @@
     buttonOk_clicked ();
   }
 
-  InputDialog::InputDialog (base_qobject&, const QStringList& prompt,
+  InputDialog::InputDialog (const QStringList& prompt,
                             const QString& title, const QFloatList& nr,
                             const QFloatList& nc, const QStringList& defaults)
     : QDialog ()
@@ -619,8 +616,7 @@
     buttonCancel_clicked ();
   }
 
-  FileDialog::FileDialog (base_qobject& oct_qobj,
-                          const QStringList& name_filters,
+  FileDialog::FileDialog (const QStringList& name_filters,
                           const QString& title, const QString& filename,
                           const QString& dirname, const QString& multimode)
     : QFileDialog ()
--- a/libgui/src/dialog.h	Sat Dec 03 00:52:02 2022 -0500
+++ b/libgui/src/dialog.h	Sat Dec 03 01:00:04 2022 -0500
@@ -44,15 +44,13 @@
 
 namespace octave
 {
-  class base_qobject;
-
   class QUIWidgetCreator : public QObject
   {
     Q_OBJECT
 
   public:
 
-    QUIWidgetCreator (base_qobject& oct_qobj);
+    QUIWidgetCreator (void);
 
     ~QUIWidgetCreator (void) = default;
 
@@ -148,8 +146,6 @@
 
   private:
 
-    base_qobject& m_octave_qobj;
-
     int m_dialog_result;
     QString m_dialog_button;
 
@@ -176,7 +172,7 @@
 
   public:
 
-    MessageDialog (base_qobject& oct_qobj, const QString& message,
+    MessageDialog (const QString& message,
                    const QString& title, const QString& icon,
                    const QStringList& button, const QString& defbutton,
                    const QStringList& role);
@@ -201,7 +197,7 @@
 
   public:
 
-    ListDialog (base_qobject& oct_qobj, const QStringList& list,
+    ListDialog (const QStringList& list,
                 const QString& mode, int width, int height,
                 const QList<int>& initial, const QString& name,
                 const QStringList& prompt, const QString& ok_string,
@@ -236,7 +232,7 @@
 
   public:
 
-    InputDialog (base_qobject& oct_qobj, const QStringList& prompt,
+    InputDialog (const QStringList& prompt,
                  const QString& title, const QFloatList& nr,
                  const QFloatList& nc, const QStringList& defaults);
 
@@ -261,7 +257,7 @@
 
   public:
 
-    FileDialog (base_qobject& oct_qobj, const QStringList& filters,
+    FileDialog (const QStringList& filters,
                 const QString& title, const QString& filename,
                 const QString& dirname, const QString& multimode);
 
--- a/libgui/src/qt-interpreter-events.cc	Sat Dec 03 00:52:02 2022 -0500
+++ b/libgui/src/qt-interpreter-events.cc	Sat Dec 03 01:00:04 2022 -0500
@@ -110,7 +110,7 @@
 
   qt_interpreter_events::qt_interpreter_events (base_qobject& oct_qobj)
     : interpreter_events (), m_octave_qobj (oct_qobj),
-      m_uiwidget_creator (oct_qobj), m_result (), m_mutex (),
+      m_uiwidget_creator (), m_result (), m_mutex (),
       m_waitcondition ()
   {
     qRegisterMetaType<QIntList> ("QIntList");