First batch

1 files changed, 44 insertions, 47 deletions

diff --git a/src/ui/interface.cpp b/src/ui/interface.cpp
index 7a9b92378..f6a353b33 100644
--- a/src/ui/interface.cpp
+++ b/src/ui/interface.cpp
@@ -25,6 +25,8 @@
 
 #include "ui/dialog/dialog-manager.h"
 #include <gtkmm/icontheme.h>
+#include <gtkmm/radiomenuitem.h>
+#include <gtkmm/separatormenuitem.h>
 #include "file.h"
 #include <glibmm/miscutils.h>
 
@@ -220,19 +222,14 @@ sp_create_window(SPViewWidget *vw, bool editable)
 
     if ( completeDropTargets == 0 || completeDropTargetsCount == 0 )
     {
-        std::vector<gchar*> types;
-
-        GSList *list = gdk_pixbuf_get_formats();
-        while ( list ) {
-            int i = 0;
-            GdkPixbufFormat *one = (GdkPixbufFormat*)list->data;
-            gchar** typesXX = gdk_pixbuf_format_get_mime_types(one);
-            for ( i = 0; typesXX[i]; i++ ) {
-                types.push_back(g_strdup(typesXX[i]));
-            }
-            g_strfreev(typesXX);
+        std::vector<Glib::ustring> types;
 
-            list = g_slist_next(list);
+        std::vector<Gdk::PixbufFormat> list = Gdk::Pixbuf::get_formats();
+        for (auto one:list) {
+            std::vector<Glib::ustring> typesXX = one.get_mime_types();
+            for (auto i:typesXX) {
+                types.push_back(i);
+            }
         }
         completeDropTargetsCount = nui_drop_target_entries + types.size();
         completeDropTargets = new GtkTargetEntry[completeDropTargetsCount];
@@ -241,8 +238,8 @@ sp_create_window(SPViewWidget *vw, bool editable)
         }
         int pos = nui_drop_target_entries;
 
-        for (std::vector<gchar*>::iterator it = types.begin() ; it != types.end() ; ++it) {
-            completeDropTargets[pos].target = *it;
+        for (std::vector<Glib::ustring>::iterator it = types.begin() ; it != types.end() ; ++it) {
+            completeDropTargets[pos].target = g_strdup((*it).c_str());
             completeDropTargets[pos].flags = 0;
             completeDropTargets[pos].info = IMAGE_DATA;
             pos++;
@@ -451,14 +448,14 @@ static GtkWidget *sp_ui_menu_append_item_from_verb(GtkMenu                  *men
                                                    Inkscape::UI::View::View *view,
                                                    bool                      show_icon = false,
                                                    bool                      radio     = false,
-                                                   GSList                   *group     = NULL)
+                                                   Gtk::RadioMenuItem::Group *group    = NULL)
 {
-    GtkWidget *item;
+    Gtk::Widget *item;
 
     // Just create a menu separator if this isn't a real action.
     // Otherwise, create a real menu item
     if (verb->get_code() == SP_VERB_NONE) {
-        item = gtk_separator_menu_item_new();
+        item = new Gtk::SeparatorMenuItem();
     } else {
         SPAction *action = verb->get_action(Inkscape::ActionContext(view));
 
@@ -467,9 +464,9 @@ static GtkWidget *sp_ui_menu_append_item_from_verb(GtkMenu                  *men
         // Create the menu item itself, either as a radio menu item, or just
         // a regular menu item depending on whether the "radio" flag is set
         if (radio) {
-            item = gtk_radio_menu_item_new(group);
+            item = new Gtk::RadioMenuItem(*group);
         } else {
-            item = gtk_menu_item_new();
+            item = new Gtk::MenuItem();
         }
 
         // Create a box to contain all the widgets (icon, label, accelerator)
@@ -490,8 +487,8 @@ static GtkWidget *sp_ui_menu_append_item_from_verb(GtkMenu                  *men
         GtkAccelGroup *accel_group = sp_shortcut_get_accel_group();
         gtk_menu_set_accel_group(menu, accel_group);
 
-        sp_shortcut_add_accelerator(item, sp_shortcut_get_primary(verb));
-        gtk_accel_label_set_accel_widget(GTK_ACCEL_LABEL(label), item);
+        sp_shortcut_add_accelerator(item->gobj(), sp_shortcut_get_primary(verb));
+        gtk_accel_label_set_accel_widget(GTK_ACCEL_LABEL(label), item->gobj());
 
         GtkWidget *icon;
 
@@ -508,33 +505,34 @@ static GtkWidget *sp_ui_menu_append_item_from_verb(GtkMenu                  *men
         gtk_box_pack_start(GTK_BOX(box), label, TRUE, TRUE, 0);
 
         // Finally, pack all the widgets into the menu item
-        gtk_container_add(GTK_CONTAINER(item), box);
+        gtk_container_add(GTK_CONTAINER(item->gobj()), box);
 
 
         action->signal_set_sensitive.connect(
             sigc::bind<0>(
                 sigc::ptr_fun(&gtk_widget_set_sensitive),
-                item));
+                item->gobj()));
         action->signal_set_name.connect(
             sigc::bind<0>(
                 sigc::ptr_fun(&sp_ui_menu_item_set_name),
-                item));
+                item->gobj()));
 
         if (!action->sensitive) {
-            gtk_widget_set_sensitive(item, FALSE);
+            item->set_sensitive(false);
         }
 
-        gtk_widget_set_events(item, GDK_KEY_PRESS_MASK);
-        g_object_set_data(G_OBJECT(item), "view", (gpointer) view);
-        g_signal_connect( G_OBJECT(item), "activate", G_CALLBACK(sp_ui_menu_activate), action );
-        g_signal_connect( G_OBJECT(item), "select", G_CALLBACK(sp_ui_menu_select_action), action );
-        g_signal_connect( G_OBJECT(item), "deselect", G_CALLBACK(sp_ui_menu_deselect_action), action );
+        gtk_widget_set_events(item->gobj(), GDK_KEY_PRESS_MASK);
+
+        g_object_set_data(G_OBJECT(item->gobj()), "view", (gpointer) view);
+        g_signal_connect( G_OBJECT(item->gobj()), "activate", G_CALLBACK(sp_ui_menu_activate), action );
+        g_signal_connect( G_OBJECT(item->gobj()), "select", G_CALLBACK(sp_ui_menu_select_action), action );
+        g_signal_connect( G_OBJECT(item->gobj()), "deselect", G_CALLBACK(sp_ui_menu_deselect_action), action );
     }
 
-    gtk_widget_show_all(item);
-    gtk_menu_shell_append(GTK_MENU_SHELL(menu), item);
+    item->show_all();
+    gtk_menu_shell_append(GTK_MENU_SHELL(menu), item->gobj());
 
-    return item;
+    return item->gobj();
 
 } // end of sp_ui_menu_append_item_from_verb
 
@@ -769,25 +767,24 @@ static void addTaskMenuItems(GtkMenu *menu, Inkscape::UI::View::View *view)
         0, 0
     };
 
-    GSList *group = 0;
+    Gtk::RadioMenuItem::Group group;
     int count = 0;
     gint active = Inkscape::UI::UXManager::getInstance()->getDefaultTask( dynamic_cast<SPDesktop*>(view) );
     for (gchar const **strs = data; strs[0]; strs += 2, count++)
     {
-        GtkWidget *item = gtk_radio_menu_item_new_with_label( group, strs[0] );
-        group = gtk_radio_menu_item_get_group( GTK_RADIO_MENU_ITEM(item) );
+        Gtk::RadioMenuItem *item = new Gtk::RadioMenuItem(group,Glib::ustring(strs[0]));
         if ( count == active )
         {
-            gtk_check_menu_item_set_active( GTK_CHECK_MENU_ITEM(item), TRUE );
+            item->set_active();
         }
 
-        g_object_set_data( G_OBJECT(item), "view", view );
-        g_signal_connect( G_OBJECT(item), "toggled", reinterpret_cast<GCallback>(taskToggled), GINT_TO_POINTER(count) );
-        g_signal_connect( G_OBJECT(item), "select", G_CALLBACK(sp_ui_menu_select), const_cast<gchar*>(strs[1]) );
-        g_signal_connect( G_OBJECT(item), "deselect", G_CALLBACK(sp_ui_menu_deselect), 0 );
+        g_object_set_data( G_OBJECT(item->gobj()), "view", view );
+        g_signal_connect( G_OBJECT(item->gobj()), "toggled", reinterpret_cast<GCallback>(taskToggled), GINT_TO_POINTER(count) );
+        g_signal_connect( G_OBJECT(item->gobj()), "select", G_CALLBACK(sp_ui_menu_select), const_cast<gchar*>(strs[1]) );
+        g_signal_connect( G_OBJECT(item->gobj()), "deselect", G_CALLBACK(sp_ui_menu_deselect), 0 );
 
-        gtk_widget_show( item );
-        gtk_menu_shell_append( GTK_MENU_SHELL(menu), item );
+        item->show();
+        gtk_menu_shell_append( GTK_MENU_SHELL(menu), GTK_WIDGET(item->gobj()) );
     }
 }
 
@@ -831,7 +828,7 @@ static void sp_ui_build_dyn_menus(Inkscape::XML::Node *menus, GtkWidget *menu, I
 {
     if (menus == NULL) return;
     if (menu == NULL)  return;
-    GSList *group = NULL;
+    Gtk::RadioMenuItem::Group group;
 
     for (Inkscape::XML::Node *menu_pntr = menus;
          menu_pntr != NULL;
@@ -863,8 +860,7 @@ static void sp_ui_build_dyn_menus(Inkscape::XML::Node *menus, GtkWidget *menu, I
 
             if (verb != NULL) {
                 if (menu_pntr->attribute("radio") != NULL) {
-                    GtkWidget *item = sp_ui_menu_append_item_from_verb (GTK_MENU(menu), verb, view, show_icon, true, group);
-                    group = gtk_radio_menu_item_get_group( GTK_RADIO_MENU_ITEM(item));
+                    GtkWidget *item = sp_ui_menu_append_item_from_verb (GTK_MENU(menu), verb, view, show_icon, true, &group);
                     if (menu_pntr->attribute("default") != NULL) {
                         gtk_check_menu_item_set_active (GTK_CHECK_MENU_ITEM (item), TRUE);
                     }
@@ -880,7 +876,8 @@ static void sp_ui_build_dyn_menus(Inkscape::XML::Node *menus, GtkWidget *menu, I
                     }
                 } else {
                     sp_ui_menu_append_item_from_verb(GTK_MENU(menu), verb, view, show_icon);
-                    group = NULL;
+                    Gtk::RadioMenuItem::Group group2;
+                    group = group2;
                 }
             } else {
                 gchar string[120];