[sound-juicer] Fix unsetting of the play icon in the main treeview



commit 00dfe04290c7b57407b7a25911b4d30864539a8e
Author: Christophe Fergeau <cfergeau redhat com>
Date:   Sun May 13 22:22:42 2012 +0200

    Fix unsetting of the play icon in the main treeview
    
    When trying to unset the "play" icon for the GtkCellRendererPixbuf
    used in the main treeview, an empty string was used ("") as the
    name of the stock icon name to use instead of a NULL pointer (which
    is the default stock icon name used by GtkCellRendererPixbuf). This
    led to warnings in the console:
    (sound-juicer:7060): Gtk-CRITICAL **: gtk_icon_set_render_icon_pixbuf:
    assertion `icon_set != NULL' failed
    
    https://bugzilla.gnome.org/show_bug.cgi?id=675986

 src/sj-main.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)
---
diff --git a/src/sj-main.c b/src/sj-main.c
index 3695a9a..a0dc6b7 100644
--- a/src/sj-main.c
+++ b/src/sj-main.c
@@ -305,7 +305,7 @@ static void number_cell_icon_data_cb (GtkTreeViewColumn *tree_column,
   gtk_tree_model_get (tree_model, iter, COLUMN_STATE, &state, -1);
   switch (state) {
   case STATE_IDLE:
-    g_object_set (G_OBJECT (cell), "stock-id", "", NULL);
+    g_object_set (G_OBJECT (cell), "stock-id", NULL, NULL);
     break;
   case STATE_PLAYING:
     g_object_set (G_OBJECT (cell), "stock-id", GTK_STOCK_MEDIA_PLAY, NULL);



[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]