Prevent implicit Widget construction

[xonotic/netradiant.git] / radiant / pluginmenu.cpp

diff --git a/radiant/pluginmenu.cpp b/radiant/pluginmenu.cpp
index fb97825abbc39098d4fa31b7b3096f094c369e91..69e5c0841eaaa6898abc56a3ae1221db3dd082f7 100644 (file)
--- a/radiant/pluginmenu.cpp
+++ b/radiant/pluginmenu.cpp
@@ -21,10 +21,10 @@
 
 #include "pluginmenu.h"
 
-#include "stream/textstream.h"
-
 #include <gtk/gtk.h>
 
+#include "stream/textstream.h"
+
 #include "gtkutil/pointer.h"
 #include "gtkutil/menu.h"
 
@@ -41,22 +41,22 @@ void plugin_activated( ui::Widget widget, gpointer data ){
 }
 
 #include <stack>
-typedef std::stack<ui::Widget> WidgetStack;
 
-void PlugInMenu_Add( GtkMenu* plugin_menu, IPlugIn* pPlugIn ){
-       ui::Widget menu, item, parent, subMenu;
+void PlugInMenu_Add( ui::Menu plugin_menu, IPlugIn* pPlugIn ){
+       ui::Widget item{ui::null}, parent{ui::null};
+       ui::Menu menu{ui::null}, subMenu{ui::null};
        const char *menuText, *menuCommand;
-       WidgetStack menuStack;
+       std::stack<ui::Menu> menuStack;
 
        parent = ui::MenuItem( pPlugIn->getMenuName() );
-       gtk_widget_show( parent );
-       gtk_container_add( GTK_CONTAINER( plugin_menu ), parent );
+       parent.show();
+       plugin_menu.add(parent);
 
        std::size_t nCount = pPlugIn->getCommandCount();
        if ( nCount > 0 ) {
-               menu = ui::Menu();
+               menu = ui::Menu(ui::New);
                if ( g_Layout_enableDetachableMenus.m_value ) {
-                       menu_tearoff( GTK_MENU( menu ) );
+                       menu_tearoff( menu );
                }
                while ( nCount > 0 )
                {
@@ -77,10 +77,10 @@ void PlugInMenu_Add( GtkMenu* plugin_menu, IPlugIn* pPlugIn ){
                                        }
 
                                        item = ui::MenuItem( menuText );
-                                       gtk_widget_show( item );
-                                       gtk_container_add( GTK_CONTAINER( menu ), item );
+                                       item.show();
+                                       menu.add(item);
 
-                                       subMenu = ui::Menu();
+                                       subMenu = ui::Menu(ui::New);
                                        gtk_menu_item_set_submenu( GTK_MENU_ITEM( item ), subMenu );
                                        menuStack.push( menu );
                                        menu = subMenu;
@@ -101,10 +101,10 @@ void PlugInMenu_Add( GtkMenu* plugin_menu, IPlugIn* pPlugIn ){
                                {
                                        item = ui::MenuItem( menuText );
                                        g_object_set_data( G_OBJECT( item ),"command", const_cast<gpointer>( static_cast<const void*>( menuCommand ) ) );
-                                       g_signal_connect( G_OBJECT( item ), "activate", G_CALLBACK( plugin_activated ), gint_to_pointer( m_nNextPlugInID ) );
+                                       item.connect( "activate", G_CALLBACK( plugin_activated ), gint_to_pointer( m_nNextPlugInID ) );
                                }
-                               gtk_widget_show( item );
-                               gtk_container_add( GTK_CONTAINER( menu ), item );
+                               item.show();
+                               menu.add(item);
                                pPlugIn->addMenuID( m_nNextPlugInID++ );
                        }
                }
@@ -124,15 +124,15 @@ void PlugInMenu_Add( GtkMenu* plugin_menu, IPlugIn* pPlugIn ){
        }
 }
 
-GtkMenu* g_plugins_menu = 0;
+ui::Menu g_plugins_menu{ui::null};
 GtkMenuItem* g_plugins_menu_separator = 0;
 
 void PluginsMenu_populate(){
        class PluginsMenuConstructor : public PluginsVisitor
        {
-       GtkMenu* m_menu;
+       ui::Menu m_menu;
 public:
-       PluginsMenuConstructor( GtkMenu* menu ) : m_menu( menu ){
+       PluginsMenuConstructor( ui::Menu menu ) : m_menu( menu ){
        }
        void visit( IPlugIn& plugin ){
                PlugInMenu_Add( m_menu, &plugin );
@@ -154,10 +154,10 @@ void PluginsMenu_clear(){
        }
 }
 
-GtkMenuItem* create_plugins_menu(){
+ui::MenuItem create_plugins_menu(){
        // Plugins menu
-       GtkMenuItem* plugins_menu_item = new_sub_menu_item_with_mnemonic( "_Plugins" );
-       GtkMenu* menu = GTK_MENU( gtk_menu_item_get_submenu( plugins_menu_item ) );
+       auto plugins_menu_item = new_sub_menu_item_with_mnemonic( "_Plugins" );
+       auto menu = ui::Menu(GTK_MENU( gtk_menu_item_get_submenu( plugins_menu_item ) ));
        if ( g_Layout_enableDetachableMenus.m_value ) {
                menu_tearoff( menu );
        }