Fix a memory leak around buttons in Colors prefs page.
[claws.git] / src / prefs_msg_colors.c
index 37675c30c893af9476ed9c451b278c9fc49d327a..13ba614c0538effa58b97635c8f66692420c717c 100644 (file)
@@ -1,6 +1,6 @@
 /*
  * Sylpheed -- a GTK+ based, lightweight, and fast e-mail client
- * Copyright (C) 2004-2009 Hiroyuki Yamamoto & The Claws Mail Team
+ * Copyright (C) 2004-2012 Hiroyuki Yamamoto & The Claws Mail Team
  *
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
@@ -19,6 +19,7 @@
 
 #ifdef HAVE_CONFIG_H
 #  include "config.h"
+#include "claws-features.h"
 #endif
 
 #include "defs.h"
@@ -133,7 +134,6 @@ static void prefs_msg_colors_create_widget(PrefsPage *_page, GtkWindow *window,
        gint c;
        gchar *tooltip_btn_text = NULL;
        gchar *tooltip_entry_text = NULL;
-       CLAWS_TIP_DECL();
 
        notebook = gtk_notebook_new();
        gtk_widget_show(notebook);
@@ -201,7 +201,7 @@ static void prefs_msg_colors_create_widget(PrefsPage *_page, GtkWindow *window,
        SET_TOGGLE_SENSITIVITY(checkbtn_enable_colors, color_buttons.btn_quote_level1);
 
        CLAWS_SET_TIP(color_buttons.btn_quote_level1,
-                            Q_("Tooltip|Pick color for 1st level text"));
+                            C_("Tooltip", "Pick color for 1st level text"));
 
        hbox = gtk_hbox_new(FALSE, VBOX_BORDER);
        gtk_widget_show (hbox);
@@ -227,7 +227,7 @@ static void prefs_msg_colors_create_widget(PrefsPage *_page, GtkWindow *window,
        SET_TOGGLE_SENSITIVITY(checkbtn_enable_colors, color_buttons.btn_quote_level2);
 
        CLAWS_SET_TIP(color_buttons.btn_quote_level2,
-                            Q_("Tooltip|Pick color for 2nd level text"));
+                            C_("Tooltip", "Pick color for 2nd level text"));
 
        hbox = gtk_hbox_new(FALSE, VBOX_BORDER);
        gtk_widget_show (hbox);
@@ -253,7 +253,7 @@ static void prefs_msg_colors_create_widget(PrefsPage *_page, GtkWindow *window,
        SET_TOGGLE_SENSITIVITY(checkbtn_enable_colors, color_buttons.btn_quote_level3);
 
        CLAWS_SET_TIP(color_buttons.btn_quote_level3,
-                            Q_("Tooltip|Pick color for 3rd level text"));
+                            C_("Tooltip", "Pick color for 3rd level text"));
 
        hbox = gtk_hbox_new(FALSE, VBOX_BORDER);
        gtk_widget_show (hbox);
@@ -276,7 +276,7 @@ static void prefs_msg_colors_create_widget(PrefsPage *_page, GtkWindow *window,
        SET_TOGGLE_SENSITIVITY(checkbtn_enable_bgcolors, color_buttons.btn_quote_level1_bg);
 
        CLAWS_SET_TIP(color_buttons.btn_quote_level1_bg,
-                            Q_("Tooltip|Pick color for 1st level text background"));
+                            C_("Tooltip", "Pick color for 1st level text background"));
 
        label_quote_bgcolor1 = gtk_label_new (_("Background"));
        gtk_widget_show(label_quote_bgcolor1);
@@ -297,7 +297,7 @@ static void prefs_msg_colors_create_widget(PrefsPage *_page, GtkWindow *window,
        SET_TOGGLE_SENSITIVITY(checkbtn_enable_bgcolors, color_buttons.btn_quote_level2_bg);
 
        CLAWS_SET_TIP(color_buttons.btn_quote_level2_bg,
-                            Q_("Tooltip|Pick color for 2nd level text background"));
+                            C_("Tooltip", "Pick color for 2nd level text background"));
 
        label_quote_bgcolor2 = gtk_label_new (_("Background"));
        gtk_widget_show(label_quote_bgcolor2);
@@ -318,7 +318,7 @@ static void prefs_msg_colors_create_widget(PrefsPage *_page, GtkWindow *window,
        SET_TOGGLE_SENSITIVITY(checkbtn_enable_bgcolors, color_buttons.btn_quote_level3_bg);
 
        CLAWS_SET_TIP(color_buttons.btn_quote_level3_bg,
-                            Q_("Tooltip|Pick color for 3rd level text background"));
+                            C_("Tooltip", "Pick color for 3rd level text background"));
 
        label_quote_bgcolor3 = gtk_label_new (_("Background"));
        gtk_widget_show(label_quote_bgcolor3);
@@ -338,7 +338,7 @@ static void prefs_msg_colors_create_widget(PrefsPage *_page, GtkWindow *window,
        SET_TOGGLE_SENSITIVITY(checkbtn_enable_colors, color_buttons.btn_uri);
 
        CLAWS_SET_TIP(color_buttons.btn_uri,
-                            Q_("Tooltip|Pick color for links"));
+                            C_("Tooltip", "Pick color for links"));
 
        lable_uri = gtk_label_new (_("URI link"));
        gtk_widget_show(lable_uri);
@@ -357,7 +357,7 @@ static void prefs_msg_colors_create_widget(PrefsPage *_page, GtkWindow *window,
        SET_TOGGLE_SENSITIVITY(checkbtn_enable_colors, color_buttons.btn_signature);
 
        CLAWS_SET_TIP(color_buttons.btn_signature,
-                            Q_("Tooltip|Pick color for signatures"));
+                            C_("Tooltip", "Pick color for signatures"));
 
        label_signature = gtk_label_new (_("Signatures"));
        gtk_widget_show(label_signature);
@@ -425,7 +425,7 @@ static void prefs_msg_colors_create_widget(PrefsPage *_page, GtkWindow *window,
        for (c = 0; c < (COLORLABELS>>1)+(COLORLABELS&1); c++) {
                /* TRANSLATORS: 'color %d' refers to the filtering/processing 
                   rule name and should not be translated */
-               tooltip_btn_text = g_strdup_printf(Q_("Tooltip|Pick color for 'color %d'"), c+1);
+               tooltip_btn_text = g_strdup_printf(C_("Tooltip", "Pick color for 'color %d'"), c+1);
 
                /* TRANSLATORS: 'color %d' refers to the filtering/processing 
                   rule name and should not be translated */
@@ -444,6 +444,7 @@ static void prefs_msg_colors_create_widget(PrefsPage *_page, GtkWindow *window,
 
                CLAWS_SET_TIP(color_buttons.custom_color[c],
                                     tooltip_btn_text);
+               g_free(tooltip_btn_text);
 
                entry_custom_colorlabel[c] = gtk_entry_new();
                gtk_widget_show (entry_custom_colorlabel[c]);
@@ -451,12 +452,13 @@ static void prefs_msg_colors_create_widget(PrefsPage *_page, GtkWindow *window,
                                   FALSE, FALSE, 0);
                CLAWS_SET_TIP(entry_custom_colorlabel[c],
                                     tooltip_entry_text);
+               g_free(tooltip_entry_text);
        }
 
        for (c = (COLORLABELS>>1)+(COLORLABELS&1); c < COLORLABELS; c++) {
                /* TRANSLATORS: 'color %d' refers to the filtering/processing 
                   rule name and should not be translated */
-               tooltip_btn_text = g_strdup_printf(Q_("Tooltip|Pick color for 'color %d'"), c+1);
+               tooltip_btn_text = g_strdup_printf(C_("Tooltip", "Pick color for 'color %d'"), c+1);
 
                /* TRANSLATORS: 'color %d' refers to the filtering/processing 
                   rule name and should not be translated */
@@ -474,6 +476,7 @@ static void prefs_msg_colors_create_widget(PrefsPage *_page, GtkWindow *window,
                                   FALSE, FALSE, 0);
                CLAWS_SET_TIP(color_buttons.custom_color[c],
                                     tooltip_btn_text);
+               g_free(tooltip_btn_text);
 
                entry_custom_colorlabel[c] = gtk_entry_new();
                gtk_widget_show (entry_custom_colorlabel[c]);
@@ -481,10 +484,10 @@ static void prefs_msg_colors_create_widget(PrefsPage *_page, GtkWindow *window,
                                   FALSE, FALSE, 0);
                CLAWS_SET_TIP(entry_custom_colorlabel[c],
                                     tooltip_entry_text);
+
+               g_free(tooltip_entry_text);
        }
 
-       g_free(tooltip_btn_text);
-       g_free(tooltip_entry_text);
 
        hbox_reset_custom_colors = gtk_hbox_new(FALSE, VBOX_BORDER);
        gtk_widget_show (hbox_reset_custom_colors);
@@ -579,6 +582,8 @@ static void quote_color_set_dialog(GtkWidget *widget, gpointer data)
        GdkColor color;
        gint rgbvalue = 0;
        GtkColorSelectionDialog *dialog;
+       GtkWidget *button_ok;
+       GtkWidget *button_cancel;
        gint c;
 
        /* custom colors */
@@ -587,7 +592,7 @@ static void quote_color_set_dialog(GtkWidget *widget, gpointer data)
                if (GPOINTER_TO_INT(type) == c) {
                        /* TRANSLATORS: 'color %d' refers to the filtering/processing 
                           rule name and should not be translated */
-                       title = g_strdup_printf(Q_("Dialog title|Pick color for 'color %d'"), c+1);
+                       title = g_strdup_printf(C_("Dialog title", "Pick color for 'color %d'"), c+1);
                        rgbvalue = prefs_common.custom_colorlabel[c].color;
                        break;
                }
@@ -595,38 +600,38 @@ static void quote_color_set_dialog(GtkWidget *widget, gpointer data)
        /* other colors */
        if (c == COLORLABELS) {
                if(g_ascii_strcasecmp(type, "LEVEL1") == 0) {
-                       title = g_strdup(Q_("Dialog title|Pick color for 1st level text"));
+                       title = g_strdup(C_("Dialog title", "Pick color for 1st level text"));
                        rgbvalue = prefs_common.quote_level1_col;
                } else if(g_ascii_strcasecmp(type, "LEVEL2") == 0) {
-                       title = g_strdup(Q_("Dialog title|Pick color for 2nd level text"));
+                       title = g_strdup(C_("Dialog title", "Pick color for 2nd level text"));
                        rgbvalue = prefs_common.quote_level2_col;
                } else if(g_ascii_strcasecmp(type, "LEVEL3") == 0) {
-                       title = g_strdup(Q_("Dialog title|Pick color for 3rd level text"));
+                       title = g_strdup(C_("Dialog title", "Pick color for 3rd level text"));
                        rgbvalue = prefs_common.quote_level3_col;
                } else if(g_ascii_strcasecmp(type, "LEVEL1BG") == 0) {
-                       title = g_strdup(Q_("Dialog title|Pick color for 1st level text background"));
+                       title = g_strdup(C_("Dialog title", "Pick color for 1st level text background"));
                        rgbvalue = prefs_common.quote_level1_bgcol;
                } else if(g_ascii_strcasecmp(type, "LEVEL2BG") == 0) {
-                       title = g_strdup(Q_("Dialog title|Pick color for 2nd level text background"));
+                       title = g_strdup(C_("Dialog title", "Pick color for 2nd level text background"));
                        rgbvalue = prefs_common.quote_level2_bgcol;
                } else if(g_ascii_strcasecmp(type, "LEVEL3BG") == 0) {
-                       title = g_strdup(Q_("Dialog title|Pick color for 3rd level text background"));
+                       title = g_strdup(C_("Dialog title", "Pick color for 3rd level text background"));
                        rgbvalue = prefs_common.quote_level3_bgcol;
                } else if(g_ascii_strcasecmp(type, "URI") == 0) {
-                       title = g_strdup(Q_("Dialog title|Pick color for links"));
+                       title = g_strdup(C_("Dialog title", "Pick color for links"));
                        rgbvalue = prefs_common.uri_col;
                } else if(g_ascii_strcasecmp(type, "TGTFLD") == 0) {
-                       title = g_strdup(Q_("Dialog title|Pick color for target folder"));
+                       title = g_strdup(C_("Dialog title", "Pick color for target folder"));
                        rgbvalue = prefs_common.tgt_folder_col;
                } else if(g_ascii_strcasecmp(type, "SIGNATURE") == 0) {
-                       title = g_strdup(Q_("Dialog title|Pick color for signatures"));
+                       title = g_strdup(C_("Dialog title", "Pick color for signatures"));
                        rgbvalue = prefs_common.signature_col;
                } else if(g_ascii_strcasecmp(type, "NEW") == 0) {
-                       title = g_strdup(Q_("Dialog title|Pick color for folder"));
+                       title = g_strdup(C_("Dialog title", "Pick color for folder"));
                        rgbvalue = prefs_common.color_new;
                } else {
                        /* Should never be called */
-                       g_warning("Unrecognized datatype '%s' in quote_color_set_dialog\n", type);
+                       g_warning("Unrecognized datatype '%s' in quote_color_set_dialog", type);
                        return;
                }
        }
@@ -638,9 +643,13 @@ static void quote_color_set_dialog(GtkWidget *widget, gpointer data)
        gtk_window_set_resizable(GTK_WINDOW(color_dialog), FALSE);
        manage_window_set_transient(GTK_WINDOW(color_dialog));
 
-       g_signal_connect(G_OBJECT(GTK_COLOR_SELECTION_DIALOG(color_dialog)->cancel_button),
+       g_object_get(color_dialog, "ok-button", &button_ok,
+                               "cancel-button", &button_cancel,
+                               NULL);
+
+       g_signal_connect(G_OBJECT(button_cancel),
                         "clicked", G_CALLBACK(quote_colors_set_dialog_cancel), data);
-       g_signal_connect(G_OBJECT(GTK_COLOR_SELECTION_DIALOG(color_dialog)->ok_button),
+       g_signal_connect(G_OBJECT(button_ok),
                         "clicked", G_CALLBACK(quote_colors_set_dialog_ok), data);
        g_signal_connect(G_OBJECT(color_dialog), "key_press_event",
                         G_CALLBACK(quote_colors_set_dialog_key_pressed),data);
@@ -651,15 +660,16 @@ static void quote_color_set_dialog(GtkWidget *widget, gpointer data)
 
        dialog = GTK_COLOR_SELECTION_DIALOG(color_dialog);
        gtk_color_selection_set_current_color
-               (GTK_COLOR_SELECTION(dialog->colorsel), &color);
+               (GTK_COLOR_SELECTION(gtk_color_selection_dialog_get_color_selection(dialog)), &color);
 
        gtk_widget_show(color_dialog);
 }
 
 static void quote_colors_set_dialog_ok(GtkWidget *widget, gpointer data)
 {
-       GtkColorSelection *colorsel = (GtkColorSelection *)
-                                               ((GtkColorSelectionDialog *)color_dialog)->colorsel;
+       GtkColorSelection *colorsel = GTK_COLOR_SELECTION(
+               gtk_color_selection_dialog_get_color_selection
+                                               ((GtkColorSelectionDialog *)color_dialog));
        GdkColor color;
        gint rgbvalue;
        gchar *type = (gchar *)data;
@@ -727,23 +737,27 @@ static gboolean quote_colors_set_dialog_key_pressed(GtkWidget *widget,
                                                GdkEventKey *event,
                                                gpointer data)
 {
+       GtkColorSelectionDialog *dialog;
+       GtkWidget *button_ok;
+       GtkWidget *button_cancel;
+
+       dialog = GTK_COLOR_SELECTION_DIALOG(widget);
+       g_object_get(dialog, "ok-button", &button_ok,
+                         "cancel-button", &button_cancel,
+                          NULL);
+
        if (event) {
                switch (event->keyval) {
-                       case GDK_Escape:
-                               gtk_button_clicked(GTK_BUTTON(GTK_COLOR_SELECTION_DIALOG
-                                                       (widget)->cancel_button));
+                       case GDK_KEY_Escape:
+                               gtk_button_clicked(GTK_BUTTON(button_cancel));
                                return TRUE;
-                       case GDK_Return: 
-                       case GDK_KP_Enter:
+                       case GDK_KEY_Return: 
+                       case GDK_KEY_KP_Enter:
                                /* NOTE: changing focus makes widget accept all currently 
                                 * changed settings! */
-                               gtk_widget_grab_focus
-                                       (GTK_COLOR_SELECTION_DIALOG
-                                               (widget)->ok_button);
+                               gtk_widget_grab_focus(button_ok);
                                /* call ok handler */                                           
-                               gtk_button_clicked(GTK_BUTTON
-                                       (GTK_COLOR_SELECTION_DIALOG
-                                               (widget)->ok_button));
+                               gtk_button_clicked(GTK_BUTTON(button_ok));
                                return TRUE;
                        default:
                                break;
@@ -764,6 +778,7 @@ static void set_button_bg_color(GtkWidget *widget, gint rgbvalue)
        newstyle->bg[GTK_STATE_ACTIVE]   = color;
 
        gtk_widget_set_style(GTK_WIDGET(widget), newstyle);
+       gtk_style_unref(newstyle);
 }
 
 static void prefs_msg_colors_save(PrefsPage *_page)