Merge branch 'gz-gui7' into port/7_to_8

Signed-off-by: Michael Carroll <[email protected]>
gazebosim · Nov 2, 2023 · 9c1f858 · 9c1f858
2 parents 9b1b7b0 + 8dae037
commit 9c1f858
Show file tree

Hide file tree

Showing 86 changed files with 957 additions and 1,174 deletions.
diff --git a/examples/plugin/custom_context_menu/CustomContext.cc b/examples/plugin/custom_context_menu/CustomContext.cc
@@ -20,12 +20,9 @@
 
 #include "CustomContext.hh"
 
-using namespace gz;
-using namespace gui;
-
 /////////////////////////////////////////////////
 CustomContext::CustomContext()
-  : Plugin()
+  : gz::gui::Plugin()
 {
 }
 
@@ -35,5 +32,5 @@ CustomContext::~CustomContext()
 }
 
 // Register this plugin
-GZ_ADD_PLUGIN(gz::gui::CustomContext,
-                    gz::gui::Plugin);
+GZ_ADD_PLUGIN(CustomContext,
+              gz::gui::Plugin);
diff --git a/examples/plugin/custom_context_menu/CustomContext.hh b/examples/plugin/custom_context_menu/CustomContext.hh
@@ -23,21 +23,15 @@
   #include <gz/gui/Plugin.hh>
 #endif
 
-namespace gz
+class CustomContext : public gz::gui::Plugin
 {
-  namespace gui
-  {
-    class CustomContext : public Plugin
-    {
-      Q_OBJECT
+  Q_OBJECT
 
-      /// \brief Constructor
-      public: CustomContext();
+  /// \brief Constructor
+  public: CustomContext();
 
-      /// \brief Destructor
-      public: virtual ~CustomContext();
-    };
-  }
-}
+  /// \brief Destructor
+  public: virtual ~CustomContext();
+};
 
-#endif
+#endif  // GZ_GUI_CUSTOMCONTEXTPLUGIN_HH_
diff --git a/examples/plugin/dialog_from_plugin/DialogFromPlugin.cc b/examples/plugin/dialog_from_plugin/DialogFromPlugin.cc
@@ -20,12 +20,9 @@
 
 #include "DialogFromPlugin.hh"
 
-using namespace gz;
-using namespace gui;
-
 /////////////////////////////////////////////////
 DialogFromPlugin::DialogFromPlugin()
-  : Plugin()
+  : gz::gui::Plugin()
 {
 }
 
@@ -35,5 +32,5 @@ DialogFromPlugin::~DialogFromPlugin()
 }
 
 // Register this plugin
-GZ_ADD_PLUGIN(gz::gui::DialogFromPlugin,
-                    gz::gui::Plugin);
+GZ_ADD_PLUGIN(DialogFromPlugin,
+              gz::gui::Plugin);
diff --git a/examples/plugin/dialog_from_plugin/DialogFromPlugin.hh b/examples/plugin/dialog_from_plugin/DialogFromPlugin.hh
@@ -23,21 +23,15 @@
   #include <gz/gui/Plugin.hh>
 #endif
 
-namespace gz
+class DialogFromPlugin : public gz::gui::Plugin
 {
-  namespace gui
-  {
-    class DialogFromPlugin : public Plugin
-    {
-      Q_OBJECT
+  Q_OBJECT
 
-      /// \brief Constructor
-      public: DialogFromPlugin();
+  /// \brief Constructor
+  public: DialogFromPlugin();
 
-      /// \brief Destructor
-      public: virtual ~DialogFromPlugin();
-    };
-  }
-}
+  /// \brief Destructor
+  public: virtual ~DialogFromPlugin();
+};
 
-#endif
+#endif  // GZ_GUI_DIALOGFROMPLUGIN_HH_
diff --git a/examples/plugin/gz_components/GzComponents.cc b/examples/plugin/gz_components/GzComponents.cc
@@ -19,5 +19,5 @@
 #include "GzComponents.hh"
 
 // Register this plugin
-GZ_ADD_PLUGIN(gz::gui::GzComponents,
+GZ_ADD_PLUGIN(GzComponents,
               gz::gui::Plugin);
diff --git a/examples/plugin/gz_components/GzComponents.hh b/examples/plugin/gz_components/GzComponents.hh
@@ -20,18 +20,12 @@
 
 #include <gz/gui/Plugin.hh>
 
-namespace gz
+class GzComponents : public gz::gui::Plugin
 {
-  namespace gui
-  {
-    class GzComponents : public Plugin
-    {
-      Q_OBJECT
+  Q_OBJECT
 
-      /// \brief Constructor
-      public: GzComponents() = default;
-    };
-  }
-}
+  /// \brief Constructor
+  public: GzComponents() = default;
+};
 
-#endif
+#endif  // GZ_GUI_EXAMPLES_PLUGIN_GZCOMPONENTS_HH_
diff --git a/examples/plugin/hello_plugin/HelloPlugin.cc b/examples/plugin/hello_plugin/HelloPlugin.cc
@@ -20,12 +20,9 @@
 
 #include "HelloPlugin.hh"
 
-using namespace gz;
-using namespace gui;
-
 /////////////////////////////////////////////////
 HelloPlugin::HelloPlugin()
-  : Plugin()
+  : gz::gui::Plugin()
 {
 }
 
@@ -53,5 +50,5 @@ void HelloPlugin::OnButton()
 }
 
 // Register this plugin
-GZ_ADD_PLUGIN(gz::gui::HelloPlugin,
-                    gz::gui::Plugin);
+GZ_ADD_PLUGIN(HelloPlugin,
+              gz::gui::Plugin);
diff --git a/examples/plugin/hello_plugin/HelloPlugin.hh b/examples/plugin/hello_plugin/HelloPlugin.hh
@@ -23,32 +23,26 @@
 #include <gz/gui/qt.h>
 #include <gz/gui/Plugin.hh>
 
-namespace gz
+class HelloPlugin : public gz::gui::Plugin
 {
-  namespace gui
-  {
-    class HelloPlugin : public Plugin
-    {
-      Q_OBJECT
+  Q_OBJECT
 
-      /// \brief Constructor
-      public: HelloPlugin();
+  /// \brief Constructor
+  public: HelloPlugin();
 
-      /// \brief Destructor
-      public: virtual ~HelloPlugin();
+  /// \brief Destructor
+  public: virtual ~HelloPlugin();
 
-      /// \brief Called by Gazebo GUI when plugin is instantiated.
-      /// \param[in] _pluginElem XML configuration for this plugin.
-      public: virtual void LoadConfig(const tinyxml2::XMLElement *_pluginElem)
-          override;
+  /// \brief Called by Gazebo GUI when plugin is instantiated.
+  /// \param[in] _pluginElem XML configuration for this plugin.
+  public: virtual void LoadConfig(const tinyxml2::XMLElement *_pluginElem)
+      override;
 
-      /// \brief Callback trigged when the button is pressed.
-      protected slots: void OnButton();
+  /// \brief Callback trigged when the button is pressed.
+  protected slots: void OnButton();
 
-      /// \brief Message to be printed when button is pressed.
-      private: std::string message{"Hello, plugin!"};
-    };
-  }
-}
+  /// \brief Message to be printed when button is pressed.
+  private: std::string message{"Hello, plugin!"};
+};
 
-#endif
+#endif  // GZ_GUI_HELLOPLUGIN_HH_
diff --git a/examples/plugin/multiple_qml/MultipleQml.cc b/examples/plugin/multiple_qml/MultipleQml.cc
@@ -20,12 +20,9 @@
 
 #include "MultipleQml.hh"
 
-using namespace gz;
-using namespace gui;
-
 /////////////////////////////////////////////////
 MultipleQml::MultipleQml()
-  : Plugin()
+  : gz::gui::Plugin()
 {
 }
 
@@ -41,5 +38,5 @@ void MultipleQml::OnButton(const QString &_text)
 }
 
 // Register this plugin
-GZ_ADD_PLUGIN(gz::gui::MultipleQml,
-                    gz::gui::Plugin);
+GZ_ADD_PLUGIN(MultipleQml,
+              gz::gui::Plugin);
diff --git a/examples/plugin/multiple_qml/MultipleQml.hh b/examples/plugin/multiple_qml/MultipleQml.hh
@@ -23,25 +23,19 @@
   #include <gz/gui/Plugin.hh>
 #endif
 
-namespace gz
+class MultipleQml : public gz::gui::Plugin
 {
-  namespace gui
-  {
-    class MultipleQml : public Plugin
-    {
-      Q_OBJECT
+  Q_OBJECT
 
-      /// \brief Constructor
-      public: MultipleQml();
+  /// \brief Constructor
+  public: MultipleQml();
 
-      /// \brief Destructor
-      public: virtual ~MultipleQml();
+  /// \brief Destructor
+  public: virtual ~MultipleQml();
 
-      /// \brief Callback trigged when the button is pressed.
-      /// \param[in] _text Button text.
-      protected slots: void OnButton(const QString &_text);
-    };
-  }
-}
+  /// \brief Callback trigged when the button is pressed.
+  /// \param[in] _text Button text.
+  protected slots: void OnButton(const QString &_text);
+};
 
-#endif
+#endif  // GZ_GUI_EXAMPLES_PLUGINS_MULTIPLEQML_HH_
diff --git a/examples/standalone/custom_drawer/custom_drawer.cc b/examples/standalone/custom_drawer/custom_drawer.cc
@@ -37,7 +37,7 @@ int main(int _argc, char **_argv)
   app.LoadConfig("../custom_drawer.config");
 
   // Let QML files use CustomActions' functions and properties
-  gz::gui::CustomActions actions;
+  CustomActions actions;
   auto context = new QQmlContext(app.Engine()->rootContext());
   context->setContextProperty("CustomActions", &actions);
 
@@ -65,4 +65,3 @@ int main(int _argc, char **_argv)
 
   return 0;
 }
-
diff --git a/examples/standalone/custom_drawer/custom_drawer.hh b/examples/standalone/custom_drawer/custom_drawer.hh
@@ -24,25 +24,18 @@
   #include <gz/gui/qt.h>
 #endif
 
-namespace gz
+/// \brief Object holding actions which can be triggered from the custom
+/// drawer.
+class CustomActions : public QObject
 {
-  namespace gui
-  {
-    /// \brief Object holding actions which can be triggered from the custom
-    /// drawer.
-    class CustomActions : public QObject
-    {
-      Q_OBJECT
-
-      /// \brief Demonstrates a C++ function which can be called from the
-      /// drawer's QML code.
-      public: Q_INVOKABLE void cppActionFromQml() const
-        {
-          std::cout << "C++ action called from QML" << std::endl;
-        }
-    };
-  }
-}
+  Q_OBJECT
 
-#endif
+  /// \brief Demonstrates a C++ function which can be called from the
+  /// drawer's QML code.
+  public: Q_INVOKABLE void cppActionFromQml() const
+    {
+      std::cout << "C++ action called from QML" << std::endl;
+    }
+};
 
+#endif  //GZ_GUI_EXAMPLES_STANDALONE_CUSTOMDRAWER_HH_
diff --git a/include/gz/gui/Application.hh b/include/gz/gui/Application.hh
@@ -37,10 +37,8 @@ namespace tinyxml2
   class XMLElement;
 }
 
-namespace gz
+namespace gz::gui
 {
-  namespace gui
-  {
     class ApplicationPrivate;
     class Dialog;
     class MainWindow;
@@ -217,11 +215,10 @@ namespace gz
     /// \return Pointer to running application, or nullptr if none is running.
     GZ_GUI_VISIBLE
     Application *App();
-  }
-}
+}  // namespace gz::gui
 
 #ifdef _MSC_VER
 #pragma warning(pop)
 #endif
 
-#endif
+#endif  // GZ_GUI_APPLICATION_HH_
diff --git a/include/gz/gui/Conversions.hh b/include/gz/gui/Conversions.hh
@@ -27,15 +27,13 @@
 #include "gz/gui/qt.h"
 #include "gz/gui/Export.hh"
 
-namespace gz
+namespace gz::common
 {
-  namespace common
-  {
-    class MouseEvent;
-  }
+  class MouseEvent;
+}
 
-  namespace gui
-  {
+namespace gz::gui
+{
     /// \brief Return the equivalent Qt color
     /// \param[in] _color Gazebo color to convert
     /// \return Qt color value
@@ -95,6 +93,5 @@ namespace gz
     /// \return Gazebo key event
     GZ_GUI_VISIBLE
     common::KeyEvent convert(const QKeyEvent &_e);
-  }
-}
-#endif
+}  // namespace gz::gui
+#endif  // GZ_GUI_CONVERSIONS_HH_
diff --git a/include/gz/gui/Dialog.hh b/include/gz/gui/Dialog.hh
@@ -31,10 +31,8 @@
 #pragma warning(disable: 4251)
 #endif
 
-namespace gz
+namespace gz::gui
 {
-  namespace gui
-  {
     class DialogPrivate;
 
     /// \brief Gui plugin
@@ -94,11 +92,10 @@ namespace gz
       /// \brief Private data pointer
       private: std::unique_ptr<DialogPrivate> dataPtr;
     };
-  }
-}
+}  // namespace gz::gui
 
 #ifdef _MSC_VER
 #pragma warning(pop)
 #endif
 
-#endif
+#endif  // GZ_GUI_DIALOG_HH_