rework Display/Summaries
[claws.git] / src / prefs_common.c
index 39d9a0ba8394524e461f82092696efcd066212b6..a305c729432293dec70bcbc091685d229ac08140 100644 (file)
@@ -41,6 +41,7 @@
 #include "prefs_display_header.h"
 #include "prefs_summary_column.h"
 #include "prefs_folder_column.h"
+#include "prefs_migration.h"
 #include "mainwindow.h"
 #include "summaryview.h"
 #include "folderview.h"
@@ -59,6 +60,9 @@
 #include "stock_pixmap.h"
 #include "prefswindow.h"
 #include "colorlabel.h"
+#include "passwordstore.h"
+#include "file-utils.h"
+
 #ifndef USE_ALT_ADDRBOOK
        #include "addrcustomattr.h"
 #endif
@@ -94,8 +98,6 @@ static PrefParam param_os_specific[] = {
         &prefs_common.newmail_notify_cmd, P_STRING, NULL, NULL, NULL},
 
        /* new fonts */
-       {"widget_font_gtk2",    NULL,
-         &prefs_common.widgetfont,             P_STRING, NULL, NULL, NULL},
        {"message_font_gtk2",   "Monospace 9",
         &prefs_common.textfont,                P_STRING, NULL, NULL, NULL},
         {"print_font_gtk2",     "Monospace 9",
@@ -121,6 +123,16 @@ static PrefParam param_os_specific[] = {
        /* Interface */
        {"pixmap_theme_path", DEFAULT_PIXMAP_THEME, 
         &prefs_common.pixmap_theme_path, P_STRING, NULL, NULL, NULL},
+#ifdef HAVE_SVG
+       {"enable_alpha_svg", "TRUE",
+        &prefs_common.enable_alpha_svg, P_BOOL, NULL, NULL, NULL},
+       {"enable_pixmap_scaling", "TRUE",
+        &prefs_common.enable_pixmap_scaling, P_BOOL, NULL, NULL, NULL},
+       {"pixmap_scaling_auto", "TRUE",
+        &prefs_common.pixmap_scaling_auto, P_BOOL, NULL, NULL, NULL},
+       {"pixmap_scaling_ppi", "96",
+        &prefs_common.pixmap_scaling_ppi, P_INT, NULL, NULL, NULL},
+#endif
 
        /* Other */
        {"ext_editor_command", "notepad %s",
@@ -199,8 +211,12 @@ static PrefParam param[] = {
         NULL, NULL, NULL},
        {"outgoing_fallback_to_ascii", "TRUE", &prefs_common.outgoing_fallback_to_ascii, P_BOOL,
         NULL, NULL, NULL},
+        {"rewrite_first_from", "TRUE", &prefs_common.rewrite_first_from,
+         P_BOOL, NULL, NULL, NULL},
        {"warn_empty_subj", "TRUE", &prefs_common.warn_empty_subj,
         P_BOOL, NULL, NULL, NULL},
+       {"warn_sending_many_recipients_num", "0", &prefs_common.warn_sending_many_recipients_num, P_INT,
+        NULL, NULL, NULL},
        {"hide_timezone", "FALSE", &prefs_common.hide_timezone,
         P_BOOL, NULL, NULL, NULL},
        {"allow_jisx0201_kana", "FALSE", &prefs_common.allow_jisx0201_kana,
@@ -263,8 +279,8 @@ static PrefParam param[] = {
         NULL, NULL, NULL},
        {"recheck_when_changing_dict", "TRUE", &prefs_common.recheck_when_changing_dict,
         P_BOOL, NULL, NULL, NULL},
-       {"misspelled_color", "#ff0000", &prefs_common.misspelled_col, P_COLOR,
-        NULL, NULL, NULL},
+       {"misspelled_color", "#ff0000", &prefs_common.color[COL_MISSPELLED],
+        P_COLOR, NULL, NULL, NULL},
        {"use_both_dicts", "FALSE", &prefs_common.use_both_dicts, P_BOOL,
         NULL, NULL, NULL},
 
@@ -313,22 +329,8 @@ static PrefParam param[] = {
         NULL, NULL, NULL},
 
        /* Display */
-       /* Obsolete fonts. For coexisting with Gtk+-1.2 version */
-       {"widget_font",         NULL,
-         &prefs_common.widgetfont_gtk1,        P_STRING, NULL, NULL, NULL},
-       {"message_font",        "-misc-fixed-medium-r-normal--14-*-*-*-*-*-*-*",
-        &prefs_common.textfont_gtk1,           P_STRING, NULL, NULL, NULL},
-       {"small_font",          "-*-helvetica-medium-r-normal--10-*-*-*-*-*-*-*",
-         &prefs_common.smallfont_gtk1,         P_STRING, NULL, NULL, NULL},
-       {"bold_font",           "-*-helvetica-bold-r-normal--12-*-*-*-*-*-*-*",
-         &prefs_common.boldfont_gtk1,          P_STRING, NULL, NULL, NULL},
-       {"normal_font",         "-*-helvetica-medium-r-normal--12-*-*-*-*-*-*-*",
-         &prefs_common.normalfont_gtk1,        P_STRING, NULL, NULL, NULL},
-
-       /* new fonts */
+       /* fonts */
 #ifndef GENERIC_UMPC
-       {"widget_font_gtk2",    NULL,
-         &SPECIFIC_PREFS.widgetfont,           P_STRING, NULL, NULL, NULL},
        {"message_font_gtk2",   "Monospace 9",
         &SPECIFIC_PREFS.textfont,                      P_STRING, NULL, NULL, NULL},
         {"print_font_gtk2",     "Monospace 9",
@@ -340,8 +342,6 @@ static PrefParam param[] = {
        {"bold_font_gtk2",      "Sans Bold 9",
          &SPECIFIC_PREFS.boldfont,             P_STRING, NULL, NULL, NULL},
 #else
-       {"widget_font_gtk2",    NULL,
-         &SPECIFIC_PREFS.widgetfont,           P_STRING, NULL, NULL, NULL},
        {"message_font_gtk2",   "Monospace 8",
         &SPECIFIC_PREFS.textfont,                      P_STRING, NULL, NULL, NULL},
         {"print_font_gtk2",     "Monospace 8",
@@ -459,8 +459,6 @@ static PrefParam param[] = {
         {"bold_unread", "TRUE", &prefs_common.bold_unread, P_BOOL,
         NULL, NULL, NULL},
 
-       {"enable_thread", "TRUE", &prefs_common.enable_thread, P_BOOL,
-        NULL, NULL, NULL},
 #ifndef GENERIC_UMPC
        {"toolbar_style", "3", &prefs_common.toolbar_style, P_ENUM,
         NULL, NULL, NULL},
@@ -618,6 +616,18 @@ static PrefParam param[] = {
        {"folder_col_size_total", "32",
         &prefs_common.folder_col_size[F_COL_TOTAL], P_INT, NULL, NULL, NULL},
 #endif
+
+       {"folder_default_thread", "TRUE", &prefs_common.folder_default_thread, P_BOOL,
+        NULL, NULL, NULL},
+       {"folder_default_thread_collapsed", "FALSE", &prefs_common.folder_default_thread_collapsed, P_BOOL,
+        NULL, NULL, NULL},
+       {"folder_default_hide_read_threads", "FALSE", &prefs_common.folder_default_hide_read_threads, P_BOOL,
+        NULL, NULL, NULL},
+       {"folder_default_hide_read_msgs", "FALSE", &prefs_common.folder_default_hide_read_msgs, P_BOOL,
+        NULL, NULL, NULL},
+       {"folder_default_hide_del_msgs", "FALSE", &prefs_common.folder_default_hide_del_msgs, P_BOOL,
+        NULL, NULL, NULL},
+
        {"summaryview_width", "500", &prefs_common.summaryview_width, P_INT,
         NULL, NULL, NULL},
        {"summaryview_height", "244", &prefs_common.summaryview_height, P_INT,
@@ -699,31 +709,48 @@ static PrefParam param[] = {
        {"enable_color", "TRUE", &prefs_common.enable_color, P_BOOL,
         NULL, NULL, NULL},
 
-       {"quote_level1_color", "#0000b3", &prefs_common.quote_level1_col, P_COLOR,
-        NULL, NULL, NULL},
-       {"quote_level2_color", "#0000b3", &prefs_common.quote_level2_col, P_COLOR,
-        NULL, NULL, NULL},
-       {"quote_level3_color", "#0000b3", &prefs_common.quote_level3_col, P_COLOR,
-        NULL, NULL, NULL},
+       {"quote_level1_color", "#0000b3", &prefs_common.color[COL_QUOTE_LEVEL1],
+        P_COLOR, NULL, NULL, NULL},
+       {"quote_level2_color", "#0000b3", &prefs_common.color[COL_QUOTE_LEVEL2],
+        P_COLOR, NULL, NULL, NULL},
+       {"quote_level3_color", "#0000b3", &prefs_common.color[COL_QUOTE_LEVEL3],
+        P_COLOR, NULL, NULL, NULL},
        {"enable_bgcolor", "FALSE", &prefs_common.enable_bgcolor, P_BOOL,
         NULL, NULL, NULL},
-       {"quote_level1_bgcolor", "#cccccc", &prefs_common.quote_level1_bgcol, P_COLOR,
-        NULL, NULL, NULL},
-       {"quote_level2_bgcolor", "#d4d4d4", &prefs_common.quote_level2_bgcol, P_COLOR,
-        NULL, NULL, NULL},
-       {"quote_level3_bgcolor", "#dddddd", &prefs_common.quote_level3_bgcol, P_COLOR,
-        NULL, NULL, NULL},
-       {"uri_color", "#007f00", &prefs_common.uri_col, P_COLOR,
-        NULL, NULL, NULL},
-       {"emphasis_color", "#0000cf", &prefs_common.emphasis_col, P_COLOR,
-        NULL, NULL, NULL},
-       {"target_folder_color", "#da1cca", &prefs_common.tgt_folder_col, P_COLOR,
-        NULL, NULL, NULL},
-       {"signature_color", "#797979", &prefs_common.signature_col, P_COLOR,
-        NULL, NULL, NULL},
+       {"quote_level1_bgcolor", "#cccccc", &prefs_common.color[COL_QUOTE_LEVEL1_BG],
+        P_COLOR, NULL, NULL, NULL},
+       {"quote_level2_bgcolor", "#d4d4d4", &prefs_common.color[COL_QUOTE_LEVEL2_BG],
+        P_COLOR, NULL, NULL, NULL},
+       {"quote_level3_bgcolor", "#dddddd", &prefs_common.color[COL_QUOTE_LEVEL3_BG],
+        P_COLOR, NULL, NULL, NULL},
+       {"uri_color", "#007f00", &prefs_common.color[COL_URI],
+        P_COLOR, NULL, NULL, NULL},
+       {"emphasis_color", "#0000cf", &prefs_common.color[COL_EMPHASIS],
+        P_COLOR, NULL, NULL, NULL},
+       {"target_folder_color", "#da1cca", &prefs_common.color[COL_TGT_FOLDER],
+        P_COLOR, NULL, NULL, NULL},
+       {"signature_color", "#797979", &prefs_common.color[COL_SIGNATURE],
+        P_COLOR, NULL, NULL, NULL},
        {"recycle_quote_colors", "FALSE", &prefs_common.recycle_quote_colors,
         P_BOOL, NULL, NULL, NULL},
 
+       {"default_header_color", "#000000", &prefs_common.color[COL_DEFAULT_HEADER],
+        P_COLOR, NULL, NULL, NULL},
+       {"default_header_bgcolor", "#f5f6be", &prefs_common.color[COL_DEFAULT_HEADER_BG],
+        P_COLOR, NULL, NULL, NULL},
+       {"tags_color", "#000000", &prefs_common.color[COL_TAGS],
+        P_COLOR, NULL, NULL, NULL},
+       {"tags_bgcolor", "#f5f6be", &prefs_common.color[COL_TAGS_BG],
+        P_COLOR, NULL, NULL, NULL},
+       {"qs_active_color", "#000000", &prefs_common.color[COL_QS_ACTIVE],
+        P_COLOR, NULL, NULL, NULL},
+       {"qs_active_bgcolor", "#f5f6be", &prefs_common.color[COL_QS_ACTIVE_BG],
+        P_COLOR, NULL, NULL, NULL},
+       {"qs_error_color", "#000000", &prefs_common.color[COL_QS_ERROR],
+        P_COLOR, NULL, NULL, NULL},
+       {"qs_error_bgcolor", "#ff7070", &prefs_common.color[COL_QS_ERROR_BG],
+        P_COLOR, NULL, NULL, NULL},
+
        {"display_header_pane", "FALSE", &prefs_common.display_header_pane,
         P_BOOL, NULL, NULL, NULL},
        {"display_header", "TRUE", &prefs_common.display_header, P_BOOL,
@@ -769,7 +796,7 @@ static PrefParam param[] = {
        /* MIME viewer */
        {"mime_textviewer",   NULL,
         &SPECIFIC_PREFS.mime_textviewer,   P_STRING, NULL, NULL, NULL},
-       {"mime_open_command", "gedit '%s'",
+       {"mime_open_command", "xdg-open '%s'",
         &SPECIFIC_PREFS.mime_open_cmd,     P_STRING, NULL, NULL, NULL},
        {"show_inline_attachments", "TRUE", 
         &prefs_common.show_inline_attachments, P_BOOL, NULL, NULL, NULL},
@@ -784,9 +811,24 @@ static PrefParam param[] = {
 #endif
        /* {"emulate_emacs", "FALSE", &prefs_common.emulate_emacs, P_BOOL,
         NULL, NULL, NULL}, */
-       {"always_show_message_when_selected", "0",
+       {"open_selected_message_on_folder_open", "FALSE",
+        &prefs_common.open_selected_on_folder_open,
+        P_BOOL, NULL, NULL, NULL},
+       {"open_selected_message_on_search_results", "FALSE",
+        &prefs_common.open_selected_on_search_results,
+        P_BOOL, NULL, NULL, NULL},
+       {"open_selected_message_on_prevnext", "FALSE",
+        &prefs_common.open_selected_on_prevnext,
+        P_BOOL, NULL, NULL, NULL},
+       {"open_selected_message_on_deletemove", "FALSE",
+        &prefs_common.open_selected_on_deletemove,
+        P_BOOL, NULL, NULL, NULL},
+       {"open_selected_message_on_directional", "FALSE",
+        &prefs_common.open_selected_on_directional,
+        P_BOOL, NULL, NULL, NULL},
+       {"always_show_message_when_selected", "FALSE",
         &prefs_common.always_show_msg,
-        P_ENUM, NULL, NULL, NULL},
+        P_BOOL, NULL, NULL, NULL},
        {"select_on_entry", "3", &prefs_common.select_on_entry,
         P_ENUM, NULL, NULL, NULL},
        {"show_tooltips", "TRUE", &prefs_common.show_tooltips,
@@ -807,6 +849,12 @@ static PrefParam param[] = {
         NULL, NULL, NULL},
        {"summary_select_prio7", "0", &prefs_common.summary_select_prio[6], P_ENUM,
         NULL, NULL, NULL},
+       {"summary_select_prio8", "0", &prefs_common.summary_select_prio[7], P_ENUM,
+        NULL, NULL, NULL},
+       {"summary_select_prio9", "0", &prefs_common.summary_select_prio[8], P_ENUM,
+        NULL, NULL, NULL},
+       {"summary_select_prio10", "0", &prefs_common.summary_select_prio[9], P_ENUM,
+        NULL, NULL, NULL},
 
        {"mark_as_read_on_new_window", "FALSE",
         &prefs_common.mark_as_read_on_new_window,
@@ -824,9 +872,25 @@ static PrefParam param[] = {
        {"pixmap_theme_path", DEFAULT_PIXMAP_THEME, 
         &SPECIFIC_PREFS.pixmap_theme_path, P_STRING,
         NULL, NULL, NULL},
+#ifdef HAVE_SVG
+       {"enable_alpha_svg", "TRUE",
+        &SPECIFIC_PREFS.enable_alpha_svg, P_BOOL,
+        NULL, NULL, NULL},
+       {"enable_pixmap_scaling", "TRUE",
+        &SPECIFIC_PREFS.enable_pixmap_scaling, P_BOOL,
+        NULL, NULL, NULL},
+       {"pixmap_scaling_auto", "TRUE",
+        &SPECIFIC_PREFS.pixmap_scaling_auto, P_BOOL,
+        NULL, NULL, NULL},
+       {"pixmap_scaling_ppi", "96",
+        &SPECIFIC_PREFS.pixmap_scaling_ppi, P_INT,
+        NULL, NULL, NULL},
+#endif
 
        {"ask_mark_all_read", "TRUE", &prefs_common.ask_mark_all_read, P_BOOL,
         NULL, NULL, NULL},
+       {"ask_override_colorlabel", "TRUE", &prefs_common.ask_override_colorlabel, P_BOOL,
+        NULL, NULL, NULL},
 
        {"ask_apply_per_account_filtering_rules", "TRUE", &prefs_common.ask_apply_per_account_filtering_rules, P_BOOL,
         NULL, NULL, NULL},
@@ -912,22 +976,22 @@ static PrefParam param[] = {
        {"enable_log_status", "FALSE", &prefs_common.enable_log_status, P_BOOL,
         NULL, NULL, NULL},
 #endif
-       {"log_msg_color", "#00af00", &prefs_common.log_msg_color, P_COLOR,
-        NULL, NULL, NULL},
-       {"log_warn_color", "#af0000", &prefs_common.log_warn_color, P_COLOR,
-        NULL, NULL, NULL},
-       {"log_error_color", "#af0000", &prefs_common.log_error_color, P_COLOR,
-        NULL, NULL, NULL},
-       {"log_in_color", "#000000", &prefs_common.log_in_color, P_COLOR,
-        NULL, NULL, NULL},
-       {"log_out_color", "#0000ef", &prefs_common.log_out_color, P_COLOR,
-        NULL, NULL, NULL},
-       {"log_status_ok_color", "#00af00", &prefs_common.log_status_ok_color, P_COLOR,
-        NULL, NULL, NULL},
-       {"log_status_nok_color", "#0000af", &prefs_common.log_status_nok_color, P_COLOR,
-        NULL, NULL, NULL},
-       {"log_status_skip_color", "#aa00aa", &prefs_common.log_status_skip_color, P_COLOR,
-        NULL, NULL, NULL},
+       {"log_msg_color", "#00af00", &prefs_common.color[COL_LOG_MSG],
+        P_COLOR, NULL, NULL, NULL},
+       {"log_warn_color", "#af0000", &prefs_common.color[COL_LOG_WARN],
+        P_COLOR, NULL, NULL, NULL},
+       {"log_error_color", "#af0000", &prefs_common.color[COL_LOG_ERROR],
+        P_COLOR, NULL, NULL, NULL},
+       {"log_in_color", "#000000", &prefs_common.color[COL_LOG_IN],
+        P_COLOR, NULL, NULL, NULL},
+       {"log_out_color", "#0000ef", &prefs_common.color[COL_LOG_OUT],
+        P_COLOR, NULL, NULL, NULL},
+       {"log_status_ok_color", "#00af00", &prefs_common.color[COL_LOG_STATUS_OK],
+        P_COLOR, NULL, NULL, NULL},
+       {"log_status_nok_color", "#0000af", &prefs_common.color[COL_LOG_STATUS_NOK],
+        P_COLOR, NULL, NULL, NULL},
+       {"log_status_skip_color", "#aa00aa", &prefs_common.color[COL_LOG_STATUS_SKIP],
+        P_COLOR, NULL, NULL, NULL},
 
     {"enable_filtering_debug", "FALSE", &prefs_common.enable_filtering_debug, P_BOOL,
         NULL, NULL, NULL},
@@ -951,8 +1015,8 @@ static PrefParam param[] = {
        {"gtk_can_change_accels", "FALSE", &prefs_common.gtk_can_change_accels, P_BOOL,
         NULL, NULL, NULL},
 
-       {"color_new", "#0000b3", &prefs_common.color_new, P_COLOR,
-        NULL, NULL, NULL},
+       {"color_new", "#0000b3", &prefs_common.color[COL_NEW],
+        P_COLOR, NULL, NULL, NULL},
 
        /* Some windows' sizes */
        {"filteringwin_width", "500", &prefs_common.filteringwin_width, P_INT,
@@ -975,6 +1039,11 @@ static PrefParam param[] = {
        {"templateswin_height", "-1", &prefs_common.templateswin_height, P_INT,
         NULL, NULL, NULL},
 
+       {"actionsiodialog_width", "582", &prefs_common.actionsiodialog_width, P_INT,
+        NULL, NULL, NULL},
+       {"actionsiodialog_height", "310", &prefs_common.actionsiodialog_height, P_INT,
+        NULL, NULL, NULL},
+
        {"actionswin_width", "486", &prefs_common.actionswin_width, P_INT,
         NULL, NULL, NULL},
        {"actionswin_height", "-1", &prefs_common.actionswin_height, P_INT,
@@ -1180,12 +1249,12 @@ static PrefParam param[] = {
        {"nav_history_length", "50", &prefs_common.nav_history_length, P_INT,
         NULL, NULL, NULL},
 
-       {"diff_added_color", "#008b8b", &prefs_common.diff_added_color, P_COLOR,
-        NULL, NULL, NULL},
-       {"diff_deleted_color", "#6a5acd", &prefs_common.diff_deleted_color, P_COLOR,
-        NULL, NULL, NULL},
-       {"diff_hunk_color", "#a52a2a", &prefs_common.diff_hunk_color, P_COLOR,
-        NULL, NULL, NULL},
+       {"diff_added_color", "#008b8b", &prefs_common.color[COL_DIFF_ADDED],
+        P_COLOR, NULL, NULL, NULL},
+       {"diff_deleted_color", "#6a5acd", &prefs_common.color[COL_DIFF_DELETED],
+        P_COLOR, NULL, NULL, NULL},
+       {"diff_hunk_color", "#a52a2a", &prefs_common.color[COL_DIFF_HUNK],
+        P_COLOR, NULL, NULL, NULL},
 
        {"folder_search_wildcard", "TRUE", &prefs_common.folder_search_wildcard, P_BOOL,
         NULL, NULL, NULL},
@@ -1199,6 +1268,14 @@ static PrefParam param[] = {
        {"master_passphrase_pbkdf2_rounds", "50000", &prefs_common.master_passphrase_pbkdf2_rounds, P_INT, NULL, NULL, NULL},
 #endif
 
+       {"use_proxy", "FALSE", &prefs_common.use_proxy, P_BOOL, NULL, NULL, NULL},
+       {"proxy_type", "1", &prefs_common.proxy_info.proxy_type, P_ENUM, NULL, NULL, NULL},
+       {"proxy_host", "localhost", &prefs_common.proxy_info.proxy_host, P_STRING, NULL, NULL, NULL},
+       {"proxy_port", "1080", &prefs_common.proxy_info.proxy_port, P_USHORT, NULL, NULL, NULL},
+       {"use_proxy_auth", "FALSE", &prefs_common.proxy_info.use_proxy_auth, P_BOOL, NULL, NULL, NULL},
+       {"proxy_name", "", &prefs_common.proxy_info.proxy_name, P_STRING, NULL, NULL, NULL},
+       {"proxy_pass", NULL, &prefs_common.proxy_info.proxy_pass, P_STRING, NULL, NULL, NULL},
+
        {NULL, NULL, NULL, P_OTHER, NULL, NULL, NULL}
 };
 
@@ -1222,19 +1299,19 @@ GList *prefs_common_read_history_from_dir_with_defaults(const gchar *dirname, co
        path = g_strconcat(get_rc_dir(), G_DIR_SEPARATOR_S, history,
                           NULL);
        }
-       if ((fp = g_fopen(path, "rb")) == NULL) {
-               if (ENOENT != errno) FILE_OP_ERROR(path, "fopen");
+       if ((fp = claws_fopen(path, "rb")) == NULL) {
+               if (ENOENT != errno) FILE_OP_ERROR(path, "claws_fopen");
                g_free(path);
                /* returns default list if set, otherwise NULL */
                return default_list;
        }
        g_free(path);
-       while (fgets(buf, sizeof(buf), fp) != NULL) {
+       while (claws_fgets(buf, sizeof(buf), fp) != NULL) {
                g_strstrip(buf);
                if (buf[0] == '\0') continue;
                tmp = add_history(tmp, buf);
        }
-       fclose(fp);
+       claws_fclose(fp);
 
        tmp = g_list_reverse(tmp);
 
@@ -1324,18 +1401,18 @@ static void prefs_common_save_history_to_dir(const gchar *dirname, const gchar *
        }
        tmp_path = g_strconcat(path, ".tmp", NULL);
 
-       if ((fp = g_fopen(tmp_path, "wb")) == NULL) {
-               FILE_OP_ERROR(tmp_path, "fopen");
+       if ((fp = claws_fopen(tmp_path, "wb")) == NULL) {
+               FILE_OP_ERROR(tmp_path, "claws_fopen");
                goto out;
        }
 
        for (cur = list; cur != NULL; cur = cur->next) {
-               TRY(fputs((gchar *)cur->data, fp) != EOF &&
-                   fputc('\n', fp) != EOF);
+               TRY(claws_fputs((gchar *)cur->data, fp) != EOF &&
+                   claws_fputc('\n', fp) != EOF);
        }
 
-       if (fclose(fp) == EOF) {
-               FILE_OP_ERROR(tmp_path, "fclose");
+       if (claws_safe_fclose(fp) == EOF) {
+               FILE_OP_ERROR(tmp_path, "claws_fclose");
                fp = NULL;
                goto out;
        }
@@ -1350,7 +1427,7 @@ static void prefs_common_save_history_to_dir(const gchar *dirname, const gchar *
 
 out:
        if (fp)
-               fclose(fp);
+               claws_safe_fclose(fp);
        g_free(tmp_path);
        g_free(path);
 }