Sort headers using clang-tidy
[geeqie.git] / src / main.cc
index b3cc698..89328fc 100644 (file)
  * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
  */
 
-#include <signal.h>
+#include "main.h"
+
 #include <sys/mman.h>
 
+#include <clocale>
+#include <csignal>
+
+#include <config.h>
+
+#ifdef HAVE_CLUTTER
+#include <clutter-gtk/clutter-gtk.h>
+#endif
+
+#ifdef HAVE_EXECINFO_H
+#include <execinfo.h>
+#endif
+
 #ifdef G_OS_UNIX
 #include <pwd.h>
 #endif
-#include <locale.h>
 
-#include "main.h"
+#ifdef HAVE_DEVELOPER
+#include "backward.h"
+#endif
 
+#include "cache-maint.h"
 #include "cache.h"
-#include "collect.h"
 #include "collect-io.h"
+#include "collect.h"
+#include "compat.h"
+#include "debug.h"
+#include "exif.h"
 #include "filedata.h"
 #include "filefilter.h"
+#include "glua.h"
+#include "histogram.h"
 #include "history-list.h"
 #include "image.h"
 #include "img-view.h"
+#include "intl.h"
 #include "layout-image.h"
 #include "layout-util.h"
+#include "main-defines.h"
+#include "metadata.h"
 #include "misc.h"
+#include "pixbuf-util.h"
 #include "rcfile.h"
 #include "remote.h"
 #include "secure-save.h"
+#include "thumb.h"
 #include "ui-fileops.h"
 #include "ui-utildlg.h"
-#include "cache-maint.h"
-#include "thumb.h"
-#include "metadata.h"
-#include "exif.h"
-#include "histogram.h"
-#include "pixbuf-util.h"
-#include "glua.h"
 #include "whereami.h"
 
-#ifdef HAVE_CLUTTER
-#include <clutter-gtk/clutter-gtk.h>
-#endif
-
-#ifdef HAVE_EXECINFO_H
-#include <execinfo.h>
-#endif
-
 gboolean thumb_format_changed = FALSE;
-static RemoteConnection *remote_connection = NULL;
+static RemoteConnection *remote_connection = nullptr;
 
 gchar *gq_prefix;
 gchar *gq_localedir;
@@ -76,13 +87,14 @@ gchar *desktop_file_template;
 gchar *instance_identifier;
 
 #if defined(SA_SIGINFO)
-void sig_handler_cb(int signo, siginfo_t *info, void *UNUSED(context))
+void sig_handler_cb(int signo, siginfo_t *info, void *)
 {
        gchar hex_char[16];
-       gchar *signal_name = NULL;
+       const gchar *signal_name = nullptr;
        gint i = 0;
        guint64 addr;
        guint64 char_index;
+       ssize_t len;
 #ifdef HAVE_EXECINFO_H
        gint bt_size;
        void *bt[1024];
@@ -90,7 +102,7 @@ void sig_handler_cb(int signo, siginfo_t *info, void *UNUSED(context))
        struct signals
                {
                gint sig_no;
-               gchar *sig_name;
+               const gchar *sig_name;
                };
        struct signals signals_list[7];
 
@@ -137,26 +149,26 @@ void sig_handler_cb(int signo, siginfo_t *info, void *UNUSED(context))
                i++;
                }
 
-       write(STDERR_FILENO, "Geeqie fatal error\n", 19);
-       write(STDERR_FILENO, "Signal: ", 8);
-       write(STDERR_FILENO, signal_name, strlen(signal_name));
-       write(STDERR_FILENO, "\n", 1);
+       len = write(STDERR_FILENO, "Geeqie fatal error\n", 19);
+       len = write(STDERR_FILENO, "Signal: ", 8);
+       len = write(STDERR_FILENO, signal_name, strlen(signal_name));
+       len = write(STDERR_FILENO, "\n", 1);
 
-       write(STDERR_FILENO, "Code: ", 6);
-       write(STDERR_FILENO,  (info->si_code == SEGV_MAPERR) ? "Address not mapped" : "Invalid permissions", strlen((info->si_code == SEGV_MAPERR) ? "Address not mapped" : "Invalid permissions"));
-       write(STDERR_FILENO, "\n", 1);
+       len = write(STDERR_FILENO, "Code: ", 6);
+       len = write(STDERR_FILENO,  (info->si_code == SEGV_MAPERR) ? "Address not mapped" : "Invalid permissions", strlen((info->si_code == SEGV_MAPERR) ? "Address not mapped" : "Invalid permissions"));
+       len = write(STDERR_FILENO, "\n", 1);
 
-       write(STDERR_FILENO, "Address: ", 9);
+       len = write(STDERR_FILENO, "Address: ", 9);
 
-       if (info->si_addr == 0)
+       if (info->si_addr == nullptr)
                {
-               write(STDERR_FILENO, "0x0\n", 4);
+               len = write(STDERR_FILENO, "0x0\n", 4);
                }
        else
                {
                /* Assume the address is 64-bit */
-               write(STDERR_FILENO, "0x", 2);
-               addr = info->si_addr;
+               len = write(STDERR_FILENO, "0x", 2);
+               addr = reinterpret_cast<guint64>(info->si_addr);
 
                for (i = 0; i < 16; i++)
                        {
@@ -164,9 +176,9 @@ void sig_handler_cb(int signo, siginfo_t *info, void *UNUSED(context))
                        char_index = char_index >> 60;
                        addr = addr << 4;
 
-                       write(STDERR_FILENO, &hex_char[char_index], 1);
+                       len = write(STDERR_FILENO, &hex_char[char_index], 1);
                        }
-               write(STDERR_FILENO, "\n", 1);
+               len = write(STDERR_FILENO, "\n", 1);
                }
 
 #ifdef HAVE_EXECINFO_H
@@ -174,10 +186,13 @@ void sig_handler_cb(int signo, siginfo_t *info, void *UNUSED(context))
        backtrace_symbols_fd(bt, bt_size, STDERR_FILENO);
 #endif
 
+       /* Avoid "not used" warning */
+       len++;
+
        exit(EXIT_FAILURE);
 }
 #else /* defined(SA_SIGINFO) */
-void sig_handler_cb(int UNUSED(signo))
+void sig_handler_cb(int)
 {
 #ifdef HAVE_EXECINFO_H
        gint bt_size;
@@ -269,8 +284,7 @@ static void parse_command_line_add_file(const gchar *file_path, gchar **path, gc
                }
 }
 
-static void parse_command_line_add_dir(const gchar *dir, gchar **UNUSED(path), gchar **UNUSED(file),
-                                      GList **UNUSED(list))
+static void parse_command_line_add_dir(const gchar *dir, gchar **, gchar **, GList **)
 {
 #if 0
        /* This is broken because file filter is not initialized yet.
@@ -294,7 +308,7 @@ static void parse_command_line_add_dir(const gchar *dir, gchar **UNUSED(path), g
                work = files;
                while (work)
                        {
-                       FileData *fd = work->data;
+                       FileData *fd = static_cast<FileData *>(work->data);
                        if (!*path) *path = remove_level_from_path(fd->path);
                        if (!*file) *file = g_strdup(fd->path);
                        *list = g_list_prepend(*list, fd);
@@ -326,7 +340,7 @@ static void parse_command_line_process_dir(const gchar *dir, gchar **path, gchar
                        {
                        parse_command_line_add_dir(*first_dir, path, file, list);
                        g_free(*first_dir);
-                       *first_dir = NULL;
+                       *first_dir = nullptr;
                        }
                parse_command_line_add_dir(dir, path, file, list);
                }
@@ -340,30 +354,30 @@ static void parse_command_line_process_file(const gchar *file_path, gchar **path
                {
                parse_command_line_add_dir(*first_dir, path, file, list);
                g_free(*first_dir);
-               *first_dir = NULL;
+               *first_dir = nullptr;
                }
        parse_command_line_add_file(file_path, path, file, list, collection_list);
 }
 
 static void parse_command_line(gint argc, gchar *argv[])
 {
-       GList *list = NULL;
-       GList *remote_list = NULL;
-       GList *remote_errors = NULL;
+       GList *list = nullptr;
+       GList *remote_list = nullptr;
+       GList *remote_errors = nullptr;
        gboolean remote_do = FALSE;
-       gchar *first_dir = NULL;
+       gchar *first_dir = nullptr;
        gchar *app_lock;
        gchar *pwd;
        gchar *current_dir;
-       gchar *geometry = NULL;
+       gchar *geometry = nullptr;
        GtkWidget *dialog_warning;
-       GString *command_line_errors = g_string_new(NULL);
+       GString *command_line_errors = g_string_new(nullptr);
 
        command_line = g_new0(CommandLine, 1);
 
        command_line->argc = argc;
        command_line->argv = argv;
-       command_line->regexp = NULL;
+       command_line->regexp = nullptr;
 
        if (argc > 1)
                {
@@ -393,12 +407,12 @@ static void parse_command_line(gint argc, gchar *argv[])
                                parse_command_line_process_file(cmd_all, &command_line->path, &command_line->file,
                                                                &list, &command_line->collection_list, &first_dir);
                                }
-                       else if (download_web_file(cmd_line, FALSE, NULL))
+                       else if (download_web_file(cmd_line, FALSE, nullptr))
                                {
                                }
                        else if (is_collection(cmd_line))
                                {
-                               gchar *path = NULL;
+                               gchar *path = nullptr;
 
                                path = collection_path(cmd_line);
                                parse_command_line_process_file(path, &command_line->path, &command_line->file,
@@ -418,14 +432,14 @@ static void parse_command_line(gint argc, gchar *argv[])
                                {
                                command_line->tools_show = TRUE;
 
-                               remote_list = g_list_append(remote_list, "+t");
+                               remote_list = g_list_append(remote_list, g_strdup("+t"));
                                }
                        else if (strcmp(cmd_line, "-t") == 0 ||
                                 strcmp(cmd_line, "--without-tools") == 0)
                                {
                                command_line->tools_hide = TRUE;
 
-                               remote_list = g_list_append(remote_list, "-t");
+                               remote_list = g_list_append(remote_list, g_strdup("-t"));
                                }
                        else if (strcmp(cmd_line, "-f") == 0 ||
                                 strcmp(cmd_line, "--fullscreen") == 0)
@@ -500,12 +514,6 @@ static void parse_command_line(gint argc, gchar *argv[])
                                printf_term(FALSE, "%s %s GTK%d\n", GQ_APPNAME, VERSION, gtk_major_version);
                                exit(0);
                                }
-                       else if (strcmp(cmd_line, "--alternate") == 0)
-                               {
-                               /* enable faster experimental algorithm */
-                               log_printf("Alternate similarity algorithm enabled\n");
-                               image_sim_alternate_set(TRUE);
-                               }
                        else if (strcmp(cmd_line, "-h") == 0 ||
                                 strcmp(cmd_line, "--help") == 0)
                                {
@@ -519,8 +527,8 @@ static void parse_command_line(gint argc, gchar *argv[])
                                print_term(FALSE, _("      --geometry=WxH+XOFF+YOFF     set main window location\n"));
                                print_term(FALSE, _("  -h, --help                       show this message\n"));
                                print_term(FALSE, _("  -l, --list [files] [collections] open collection window for command line\n"));
-                               print_term(FALSE, _("  -n, --new-instance               open a new instance of Geeqie\n"));
-                               print_term(FALSE, _("  -o:, --log-file:<file>     save log data to file\n"));
+                               print_term(FALSE, _("  -n, --new-instance               open a new instance of Geeqie *\n"));
+                               print_term(FALSE, _("  -o:, --log-file:<file>           save log data to file\n"));
                                print_term(FALSE, _("  -r, --remote                     send following commands to open window\n"));
                                print_term(FALSE, _("  -rh, --remote-help               print remote command list\n"));
                                print_term(FALSE, _("  -s, --slideshow                  start in slideshow mode\n"));
@@ -530,15 +538,11 @@ static void parse_command_line(gint argc, gchar *argv[])
                                print_term(FALSE, _("  +w, --show-log-window            show log window\n"));
 #ifdef DEBUG
                                print_term(FALSE, _("      --debug[=level]              turn on debug output\n"));
-                               print_term(FALSE, _("  -g:, --grep:<regexp>     filter debug output\n"));
+                               print_term(FALSE, _("  -g:, --grep:<regexp>             filter debug output\n"));
 #endif
 
-#if 0
-                               /* these options are not officially supported!
-                                * only for testing new features, no need to translate them */
-                               print_term(FALSE, "  --alternate                use alternate similarity algorithm\n");
-#endif
                                print_term(FALSE, "\n");
+                               print_term(FALSE, "* Normally a single set of configuration files is used for all instances.\nHowever, the environment variables XDG_CONFIG_HOME, XDG_CACHE_HOME, XDG_DATA_HOME\ncan be used to modify this behavior on an individual basis e.g.\n\nXDG_CONFIG_HOME=/tmp/a XDG_CACHE_HOME=/tmp/b geeqie\n\n");
 
                                remote_help();
 
@@ -558,12 +562,12 @@ static void parse_command_line(gint argc, gchar *argv[])
 
                if (command_line_errors->len > 0)
                        {
-                       dialog_warning = gtk_message_dialog_new(NULL, GTK_DIALOG_MODAL, GTK_MESSAGE_ERROR, GTK_BUTTONS_OK, "%s", "Invalid parameter(s):");
+                       dialog_warning = gtk_message_dialog_new(nullptr, GTK_DIALOG_MODAL, GTK_MESSAGE_ERROR, GTK_BUTTONS_OK, "%s", "Invalid parameter(s):");
                        gtk_message_dialog_format_secondary_text(GTK_MESSAGE_DIALOG(dialog_warning), "%s", command_line_errors->str);
                        gtk_window_set_title(GTK_WINDOW(dialog_warning), GQ_APPNAME);
-                       gtk_window_set_keep_above(GTK_WINDOW(dialog_warning), TRUE);
+                       gq_gtk_window_set_keep_above(GTK_WINDOW(dialog_warning), TRUE);
                        gtk_dialog_run(GTK_DIALOG(dialog_warning));
-                       gtk_widget_destroy(dialog_warning);
+                       g_object_unref(dialog_warning);
                        g_string_free(command_line_errors, TRUE);
 
                        exit(EXIT_FAILURE);
@@ -579,7 +583,7 @@ static void parse_command_line(gint argc, gchar *argv[])
        if (!command_line->path && first_dir)
                {
                command_line->path = first_dir;
-               first_dir = NULL;
+               first_dir = nullptr;
 
                parse_out_relatives(command_line->path);
                }
@@ -599,7 +603,7 @@ static void parse_command_line(gint argc, gchar *argv[])
                                geometry = g_strdup_printf("--geometry=%s", command_line->geometry);
                                remote_list = g_list_prepend(remote_list, geometry);
                                }
-                       remote_list = g_list_prepend(remote_list, "--new-window");
+                       remote_list = g_list_prepend(remote_list, g_strdup("--new-window"));
                        }
                g_free(app_lock);
                }
@@ -612,19 +616,19 @@ static void parse_command_line(gint argc, gchar *argv[])
 
                        while (work)
                                {
-                               gchar *opt = work->data;
+                               auto opt = static_cast<gchar *>(work->data);
 
                                command_line_errors = g_string_append(command_line_errors, opt);
                                command_line_errors = g_string_append(command_line_errors, "\n");
                                work = work->next;
                                }
 
-                       dialog_warning = gtk_message_dialog_new(NULL, GTK_DIALOG_MODAL, GTK_MESSAGE_ERROR, GTK_BUTTONS_OK, "%s", "Invalid parameter(s):");
+                       dialog_warning = gtk_message_dialog_new(nullptr, GTK_DIALOG_MODAL, GTK_MESSAGE_ERROR, GTK_BUTTONS_OK, "%s", "Invalid parameter(s):");
                        gtk_message_dialog_format_secondary_text(GTK_MESSAGE_DIALOG(dialog_warning), "%s", command_line_errors->str);
                        gtk_window_set_title(GTK_WINDOW(dialog_warning), GQ_APPNAME);
-                       gtk_window_set_keep_above(GTK_WINDOW(dialog_warning), TRUE);
+                       gq_gtk_window_set_keep_above(GTK_WINDOW(dialog_warning), TRUE);
                        gtk_dialog_run(GTK_DIALOG(dialog_warning));
-                       gtk_widget_destroy(dialog_warning);
+                       g_object_unref(dialog_warning);
                        g_string_free(command_line_errors, TRUE);
 
                        exit(EXIT_FAILURE);
@@ -652,20 +656,20 @@ static void parse_command_line(gint argc, gchar *argv[])
                }
        else
                {
-               string_list_free(list);
-               command_line->cmd_list = NULL;
+               g_list_free_full(list, g_free);
+               command_line->cmd_list = nullptr;
                }
 
        if (command_line->startup_blank)
                {
                g_free(command_line->path);
-               command_line->path = NULL;
+               command_line->path = nullptr;
                g_free(command_line->file);
-               command_line->file = NULL;
+               command_line->file = nullptr;
                filelist_free(command_line->cmd_list);
-               command_line->cmd_list = NULL;
-               string_list_free(command_line->collection_list);
-               command_line->collection_list = NULL;
+               command_line->cmd_list = nullptr;
+               g_list_free_full(command_line->collection_list, g_free);
+               command_line->collection_list = nullptr;
                }
 }
 
@@ -749,7 +753,7 @@ static gboolean parse_command_line_for_cache_maintenance_option(gint argc, gchar
 static void process_command_line_for_cache_maintenance_option(gint argc, gchar *argv[])
 {
        gchar *rc_path;
-       gchar *folder_path = NULL;
+       gchar *folder_path = nullptr;
        gsize size;
        gsize i = 0;
        gchar *buf_config_file;
@@ -765,7 +769,7 @@ static void process_command_line_for_cache_maintenance_option(gint argc, gchar *
 
                        if (isfile(rc_path))
                                {
-                               if (g_file_get_contents(rc_path, &buf_config_file, &size, NULL))
+                               if (g_file_get_contents(rc_path, &buf_config_file, &size, nullptr))
                                        {
                                        while (i < size)
                                                {
@@ -826,7 +830,7 @@ static void process_command_line_for_cache_maintenance_option(gint argc, gchar *
 #define RC_HISTORY_NAME "history"
 #define RC_MARKS_NAME "marks"
 
-static void setup_env_path(void)
+static void setup_env_path()
 {
        const gchar *old_path = g_getenv("PATH");
        gchar *path = g_strconcat(gq_bindir, ":", old_path, NULL);
@@ -834,7 +838,7 @@ static void setup_env_path(void)
        g_free(path);
 }
 
-static void keys_load(void)
+static void keys_load()
 {
        gchar *path;
 
@@ -843,7 +847,7 @@ static void keys_load(void)
        g_free(path);
 }
 
-static void keys_save(void)
+static void keys_save()
 {
        gchar *path;
 
@@ -852,7 +856,7 @@ static void keys_save(void)
        g_free(path);
 }
 
-static void marks_load(void)
+static void marks_load()
 {
        gchar *path;
 
@@ -894,20 +898,21 @@ static void gq_accel_map_print(
                    GdkModifierType accel_mods,
                    gboolean    changed)
 {
-       GString *gstring = g_string_new(changed ? NULL : "; ");
-       SecureSaveInfo *ssi = data;
-       gchar *tmp, *name;
+       GString *gstring = g_string_new(changed ? nullptr : "; ");
+       auto ssi = static_cast<SecureSaveInfo *>(data);
+       gchar *tmp;
+       gchar *name;
 
        g_string_append(gstring, "(gtk_accel_path \"");
 
-       tmp = g_strescape(accel_path, NULL);
+       tmp = g_strescape(accel_path, nullptr);
        g_string_append(gstring, tmp);
        g_free(tmp);
 
        g_string_append(gstring, "\" \"");
 
        name = gtk_accelerator_name(accel_key, accel_mods);
-       tmp = g_strescape(name, NULL);
+       tmp = g_strescape(name, nullptr);
        g_free(name);
        g_string_append(gstring, tmp);
        g_free(tmp);
@@ -945,7 +950,7 @@ static gboolean gq_accel_map_save(const gchar *path)
 
        g_string_free(gstring, TRUE);
 
-       gtk_accel_map_foreach((gpointer) ssi, gq_accel_map_print);
+       gtk_accel_map_foreach(ssi, gq_accel_map_print);
 
        if (secure_close(ssi))
                {
@@ -957,12 +962,12 @@ static gboolean gq_accel_map_save(const gchar *path)
        return TRUE;
 }
 
-static gchar *accep_map_filename(void)
+static gchar *accep_map_filename()
 {
        return g_build_filename(get_rc_dir(), "accels", NULL);
 }
 
-static void accel_map_save(void)
+static void accel_map_save()
 {
        gchar *path;
 
@@ -971,7 +976,7 @@ static void accel_map_save(void)
        g_free(path);
 }
 
-static void accel_map_load(void)
+static void accel_map_load()
 {
        gchar *path;
        gchar *pathl;
@@ -983,7 +988,7 @@ static void accel_map_load(void)
        g_free(path);
 }
 
-static void gtkrc_load(void)
+static void gtkrc_load()
 {
        gchar *path;
        gchar *pathl;
@@ -998,9 +1003,9 @@ static void gtkrc_load(void)
        g_free(path);
 }
 
-static void exit_program_final(void)
+static void exit_program_final()
 {
-       LayoutWindow *lw = NULL;
+       LayoutWindow *lw = nullptr;
        GList *list;
        LayoutWindow *tmp_lw;
        gchar *archive_dir;
@@ -1019,10 +1024,10 @@ static void exit_program_final(void)
                list = layout_window_list;
                while (list)
                        {
-                       tmp_lw = list->data;
+                       tmp_lw = static_cast<LayoutWindow *>(list->data);
                        if (!g_str_has_prefix(tmp_lw->options.id, "lw"))
                                {
-                               save_layout(list->data);
+                               save_layout(static_cast<LayoutWindow *>(list->data));
                                }
                        list = list->next;
                        }
@@ -1042,7 +1047,7 @@ static void exit_program_final(void)
        if (isdir(archive_dir))
                {
                archive_file = g_file_new_for_path(archive_dir);
-               rmdir_recursive(archive_file, NULL, NULL);
+               rmdir_recursive(archive_file, nullptr, nullptr);
                g_free(archive_dir);
                g_object_unref(archive_file);
                }
@@ -1053,7 +1058,7 @@ static void exit_program_final(void)
        if (isdir(archive_dir))
                {
                archive_file = g_file_new_for_path(archive_dir);
-               g_file_delete(archive_file, NULL, NULL);
+               g_file_delete(archive_file, nullptr, nullptr);
                g_free(archive_dir);
                g_object_unref(archive_file);
                }
@@ -1063,26 +1068,27 @@ static void exit_program_final(void)
        gtk_main_quit();
 }
 
-static GenericDialog *exit_dialog = NULL;
+static GenericDialog *exit_dialog = nullptr;
 
-static void exit_confirm_cancel_cb(GenericDialog *gd, gpointer UNUSED(data))
+static void exit_confirm_cancel_cb(GenericDialog *gd, gpointer)
 {
-       exit_dialog = NULL;
+       exit_dialog = nullptr;
        generic_dialog_close(gd);
 }
 
-static void exit_confirm_exit_cb(GenericDialog *gd, gpointer UNUSED(data))
+static void exit_confirm_exit_cb(GenericDialog *gd, gpointer)
 {
-       exit_dialog = NULL;
+       exit_dialog = nullptr;
        generic_dialog_close(gd);
        exit_program_final();
 }
 
-static gint exit_confirm_dlg(void)
+static gint exit_confirm_dlg()
 {
        GtkWidget *parent;
        LayoutWindow *lw;
        gchar *msg;
+       GString *message;
 
        if (exit_dialog)
                {
@@ -1090,10 +1096,10 @@ static gint exit_confirm_dlg(void)
                return TRUE;
                }
 
-       if (!collection_window_modified_exists()) return FALSE;
+       if (!collection_window_modified_exists() && (layout_window_count() == 1)) return FALSE;
 
-       parent = NULL;
-       lw = NULL;
+       parent = nullptr;
+       lw = nullptr;
        if (layout_valid(&lw))
                {
                parent = lw->window;
@@ -1102,29 +1108,45 @@ static gint exit_confirm_dlg(void)
        msg = g_strdup_printf("%s - %s", GQ_APPNAME, _("exit"));
        exit_dialog = generic_dialog_new(msg,
                                "exit", parent, FALSE,
-                               exit_confirm_cancel_cb, NULL);
+                               exit_confirm_cancel_cb, nullptr);
        g_free(msg);
        msg = g_strdup_printf(_("Quit %s"), GQ_APPNAME);
-       generic_dialog_add_message(exit_dialog, GTK_STOCK_DIALOG_QUESTION,
-                                  msg, _("Collections have been modified. Quit anyway?"), TRUE);
+
+       message = g_string_new(nullptr);
+
+       if (collection_window_modified_exists())
+               {
+               message = g_string_append(message, _("Collections have been modified.\n"));
+               }
+
+       if (layout_window_count() > 1)
+               {
+               g_string_append_printf(message, _("%d windows are open.\n\n"), layout_window_count());
+               }
+
+       message = g_string_append(message, _("Quit anyway?"));
+
+       generic_dialog_add_message(exit_dialog, GQ_ICON_DIALOG_QUESTION, msg, message->str, TRUE);
        g_free(msg);
-       generic_dialog_add_button(exit_dialog, GTK_STOCK_QUIT, NULL, exit_confirm_exit_cb, TRUE);
+       generic_dialog_add_button(exit_dialog, GQ_ICON_QUIT, _("Quit"), exit_confirm_exit_cb, TRUE);
 
        gtk_widget_show(exit_dialog->dialog);
 
+       g_string_free(message, TRUE);
+
        return TRUE;
 }
 
-static void exit_program_write_metadata_cb(gint success, const gchar *UNUSED(dest_path), gpointer UNUSED(data))
+static void exit_program_write_metadata_cb(gint success, const gchar *, gpointer)
 {
        if (success) exit_program();
 }
 
-void exit_program(void)
+void exit_program()
 {
-       layout_image_full_screen_stop(NULL);
+       layout_image_full_screen_stop(nullptr);
 
-       if (metadata_write_queue_confirm(FALSE, exit_program_write_metadata_cb, NULL)) return;
+       if (metadata_write_queue_confirm(FALSE, exit_program_write_metadata_cb, nullptr)) return;
 
        options->marks_save ? marks_save(TRUE) : marks_save(FALSE);
 
@@ -1138,7 +1160,7 @@ void exit_program(void)
  * This code is incorrect according to POSIX, because:
  *
  *   mmap is not async-signal-safe and thus may not be called from a signal handler
- * 
+ *
  *   mmap must be called with a valid file descriptor.  POSIX requires that
  *   a fildes argument of -1 must cause mmap to return EBADF.
  *
@@ -1146,9 +1168,12 @@ void exit_program(void)
  * an alternative approach.
  */
 /** @FIXME this probably needs some better ifdefs. Please report any compilation problems */
+/** @FIXME This section needs revising */
 
+#pragma GCC diagnostic push
+#pragma GCC diagnostic ignored "-Wunused-function"
 #if defined(SIGBUS) && defined(SA_SIGINFO)
-static void sigbus_handler_cb(int UNUSED(signum), siginfo_t *info, void *UNUSED(context))
+static void sigbus_handler_cb_unused(int, siginfo_t *info, void *)
 {
        /*
         * @FIXME Design and implement a POSIX-acceptable approach,
@@ -1161,37 +1186,40 @@ static void sigbus_handler_cb(int UNUSED(signum), siginfo_t *info, void *UNUSED(
 }
 #endif
 
-static void setup_sigbus_handler(void)
+static void setup_sigbus_handler_unused()
 {
 #if defined(SIGBUS) && defined(SA_SIGINFO)
        struct sigaction sigbus_action;
        sigfillset(&sigbus_action.sa_mask);
-       sigbus_action.sa_sigaction = sigbus_handler_cb;
+       sigbus_action.sa_sigaction = sigbus_handler_cb_unused;
        sigbus_action.sa_flags = SA_SIGINFO;
 
-       sigaction(SIGBUS, &sigbus_action, NULL);
+       sigaction(SIGBUS, &sigbus_action, nullptr);
 #endif
 }
+#pragma GCC diagnostic pop
 
-static void setup_sig_handler(void)
+#ifndef HAVE_DEVELOPER
+static void setup_sig_handler()
 {
        struct sigaction sigsegv_action;
        sigfillset(&sigsegv_action.sa_mask);
        sigsegv_action.sa_sigaction = sig_handler_cb;
        sigsegv_action.sa_flags = SA_SIGINFO;
 
-       sigaction(SIGABRT, &sigsegv_action, NULL);
-       sigaction(SIGBUS, &sigsegv_action, NULL);
-       sigaction(SIGFPE, &sigsegv_action, NULL);
-       sigaction(SIGILL, &sigsegv_action, NULL);
-       sigaction(SIGIOT, &sigsegv_action, NULL);
-       sigaction(SIGSEGV, &sigsegv_action, NULL);
+       sigaction(SIGABRT, &sigsegv_action, nullptr);
+       sigaction(SIGBUS, &sigsegv_action, nullptr);
+       sigaction(SIGFPE, &sigsegv_action, nullptr);
+       sigaction(SIGILL, &sigsegv_action, nullptr);
+       sigaction(SIGIOT, &sigsegv_action, nullptr);
+       sigaction(SIGSEGV, &sigsegv_action, nullptr);
 }
+#endif
 
 static void set_theme_bg_color()
 {
        GdkRGBA bg_color;
-       GdkColor theme_color;
+       GdkRGBA theme_color;
        GtkStyleContext *style_context;
        GList *work;
        LayoutWindow *lw;
@@ -1199,18 +1227,18 @@ static void set_theme_bg_color()
        if (!options->image.use_custom_border_color)
                {
                work = layout_window_list;
-               lw = work->data;
+               lw = static_cast<LayoutWindow *>(work->data);
 
                style_context = gtk_widget_get_style_context(lw->window);
                gtk_style_context_get_background_color(style_context, GTK_STATE_FLAG_NORMAL, &bg_color);
 
-               theme_color.red = bg_color.red * 65535;
-               theme_color.green = bg_color.green * 65535;
-               theme_color.blue = bg_color.blue * 65535;
+               theme_color.red = bg_color.red  ;
+               theme_color.green = bg_color.green  ;
+               theme_color.blue = bg_color.blue ;
 
                while (work)
                        {
-                       lw = work->data;
+                       lw = static_cast<LayoutWindow *>(work->data);
                        image_background_set_color(lw->image, &theme_color);
                        work = work->next;
                        }
@@ -1219,7 +1247,7 @@ static void set_theme_bg_color()
        view_window_colors_update();
 }
 
-static gboolean theme_change_cb(GObject *UNUSED(gobject), GParamSpec *UNUSED(pspec), gpointer UNUSED(data))
+static gboolean theme_change_cb(GObject *, GParamSpec *, gpointer)
 {
        set_theme_bg_color();
 
@@ -1228,7 +1256,7 @@ static gboolean theme_change_cb(GObject *UNUSED(gobject), GParamSpec *UNUSED(psp
 
 /**
  * @brief Set up the application paths
- * 
+ *
  * This function is required for use of AppImages. AppImages are
  * relocatable, and therefore cannot use fixed paths to various components.
  * These paths were originally #defines created during compilation.
@@ -1241,9 +1269,9 @@ static void create_application_paths()
        gint length;
        gchar *path;
 
-       length = wai_getExecutablePath(NULL, 0, NULL);
-       path = (gchar *)malloc(length + 1);
-       wai_getExecutablePath(path, length, NULL);
+       length = wai_getExecutablePath(nullptr, 0, nullptr);
+       path = static_cast<gchar *>(malloc(length + 1));
+       wai_getExecutablePath(path, length, nullptr);
        path[length] = '\0';
 
        gq_executable_path = g_strdup(path);
@@ -1255,7 +1283,7 @@ static void create_application_paths()
        gq_htmldir = g_build_filename(gq_prefix, GQ_HTMLDIR, NULL);
        gq_appdir = g_build_filename(gq_prefix, GQ_APPDIR, NULL);
        gq_bindir = g_build_filename(gq_prefix, GQ_BINDIR, NULL);
-       desktop_file_template = g_build_filename(gq_appdir, "template.desktop", NULL);
+       desktop_file_template = g_build_filename(gq_appdir, "org.geeqie.template.desktop", NULL);
 
        g_free(dirname);
        g_free(path);
@@ -1263,15 +1291,15 @@ static void create_application_paths()
 
 gint main(gint argc, gchar *argv[])
 {
-       CollectionData *first_collection = NULL;
-       gchar *buf;
-       CollectionData *cd = NULL;
+       CollectionData *cd = nullptr;
+       CollectionData *first_collection = nullptr;
        gboolean disable_clutter = FALSE;
        gboolean single_dir = TRUE;
-       LayoutWindow *lw;
+       gchar *buf;
+       GdkScreen *screen;
+       GtkCssProvider *provider;
        GtkSettings *default_settings;
-       gint fd_stderr[2];
-       GIOChannel *stderr_channel;
+       LayoutWindow *lw;
 
        gdk_set_allowed_backends("x11,*");
 
@@ -1306,7 +1334,7 @@ gint main(gint argc, gchar *argv[])
 #endif
 
        /* setup random seed for random slideshow */
-       srand(time(NULL));
+       srand(time(nullptr));
 
 #if 0
        /* See later comment; this handler leads to UB. */
@@ -1314,11 +1342,11 @@ gint main(gint argc, gchar *argv[])
 #endif
 
        /* register global notify functions */
-       file_data_register_notify_func(cache_notify_cb, NULL, NOTIFY_PRIORITY_HIGH);
-       file_data_register_notify_func(thumb_notify_cb, NULL, NOTIFY_PRIORITY_HIGH);
-       file_data_register_notify_func(histogram_notify_cb, NULL, NOTIFY_PRIORITY_HIGH);
-       file_data_register_notify_func(collect_manager_notify_cb, NULL, NOTIFY_PRIORITY_LOW);
-       file_data_register_notify_func(metadata_notify_cb, NULL, NOTIFY_PRIORITY_LOW);
+       file_data_register_notify_func(cache_notify_cb, nullptr, NOTIFY_PRIORITY_HIGH);
+       file_data_register_notify_func(thumb_notify_cb, nullptr, NOTIFY_PRIORITY_HIGH);
+       file_data_register_notify_func(histogram_notify_cb, nullptr, NOTIFY_PRIORITY_HIGH);
+       file_data_register_notify_func(collect_manager_notify_cb, nullptr, NOTIFY_PRIORITY_LOW);
+       file_data_register_notify_func(metadata_notify_cb, nullptr, NOTIFY_PRIORITY_LOW);
 
 
        gtkrc_load();
@@ -1359,7 +1387,7 @@ gint main(gint argc, gchar *argv[])
        pixbuf_inline_register_stock_icons();
 
        DEBUG_1("%s main: setting default options before commandline handling", get_exec_time());
-       options = init_options(NULL);
+       options = init_options(nullptr);
        setup_default_options(options);
        if (disable_clutter)
                {
@@ -1379,6 +1407,11 @@ gint main(gint argc, gchar *argv[])
 
        setup_env_path();
 
+       screen = gdk_screen_get_default();
+       provider = gtk_css_provider_new();
+       gtk_css_provider_load_from_resource(provider, GQ_RESOURCE_PATH_UI "/custom.css");
+       gtk_style_context_add_provider_for_screen(screen, GTK_STYLE_PROVIDER(provider), GTK_STYLE_PROVIDER_PRIORITY_APPLICATION);
+
        if (parse_command_line_for_cache_maintenance_option(argc, argv))
                {
                process_command_line_for_cache_maintenance_option(argc, argv);
@@ -1441,7 +1474,7 @@ gint main(gint argc, gchar *argv[])
                                CollectWindow *cw;
                                const gchar *path;
 
-                               path = work->data;
+                               path = static_cast<const gchar *>(work->data);
                                work = work->next;
 
                                cw = collection_window_new(path);
@@ -1464,7 +1497,7 @@ gint main(gint argc, gchar *argv[])
                if (command_line->cmd_list && !(command_line->startup_command_line_collection))
                        {
                        GList *work;
-                       gchar *path = NULL;
+                       gchar *path = nullptr;
 
                        work = command_line->cmd_list;
 
@@ -1472,7 +1505,7 @@ gint main(gint argc, gchar *argv[])
                                {
                                gchar *dirname;
 
-                               dirname = g_path_get_dirname(work->data);
+                               dirname = g_path_get_dirname(static_cast<const gchar *>(work->data));
                                if (!path)
                                        {
                                        path = g_strdup(dirname);
@@ -1498,7 +1531,7 @@ gint main(gint argc, gchar *argv[])
                        GList *work;
                        CollectWindow *cw;
 
-                       cw = collection_window_new(NULL);
+                       cw = collection_window_new(nullptr);
                        cd = cw->cd;
 
                        collection_path_changed(cd);
@@ -1508,7 +1541,7 @@ gint main(gint argc, gchar *argv[])
                                {
                                FileData *fd;
 
-                               fd = file_data_new_simple(work->data);
+                               fd = file_data_new_simple(static_cast<const gchar *>(work->data));
                                collection_add(cd, fd, FALSE);
                                file_data_unref(fd);
                                work = work->next;
@@ -1517,11 +1550,11 @@ gint main(gint argc, gchar *argv[])
                        work = command_line->collection_list;
                        while (work)
                                {
-                               collection_load(cd, (gchar *)work->data, COLLECTION_LOAD_APPEND);
+                               collection_load(cd, static_cast<gchar *>(work->data), COLLECTION_LOAD_APPEND);
                                work = work->next;
                                }
 
-                       if (cd->list) layout_image_set_collection(NULL, cd, cd->list->data);
+                       if (cd->list) layout_image_set_collection(nullptr, cd, static_cast<CollectInfo *>(cd->list->data));
 
                        /* mem leak, we never unref this collection when !startup_command_line_collection
                         * (the image view of the main window does not hold a ref to the collection)
@@ -1534,14 +1567,14 @@ gint main(gint argc, gchar *argv[])
                        }
                else if (first_collection)
                        {
-                       layout_image_set_collection(NULL, first_collection,
+                       layout_image_set_collection(nullptr, first_collection,
                                                    collection_get_first(first_collection));
                        }
 
                /* If the files on the command line are from one folder, select those files
                 * unless it is a command line collection - then leave focus on collection window
                 */
-               lw = NULL;
+               lw = nullptr;
                layout_valid(&lw);
 
                if (single_dir && command_line->cmd_list && !command_line->startup_command_line_collection)
@@ -1550,11 +1583,11 @@ gint main(gint argc, gchar *argv[])
                        GList *selected;
                        FileData *fd;
 
-                       selected = NULL;
+                       selected = nullptr;
                        work = command_line->cmd_list;
                        while (work)
                                {
-                               fd = file_data_new_simple((gchar *)work->data);
+                               fd = file_data_new_simple(static_cast<gchar *>(work->data));
                                selected = g_list_append(selected, fd);
                                file_data_unref(fd);
                                work = work->next;
@@ -1574,9 +1607,17 @@ gint main(gint argc, gchar *argv[])
                }
 
        /* Show a fade-out notification window if the server has a newer AppImage version */
-       if (options->appimage_notifications && g_getenv("APPDIR") && strstr(g_getenv("APPDIR"), "/tmp/.mount_Geeqie"))
+       if (options->appimage_notifications)
                {
-               appimage_notification();
+               if (g_getenv("APPDIR") && strstr(g_getenv("APPDIR"), "/tmp/.mount_Geeqie"))
+                       {
+                       appimage_notification();
+                       }
+               else if (g_strstr_len(gq_executable_path, -1, "AppRun"))
+                       {
+                       /* Probably running an extracted AppImage */
+                       appimage_notification();
+                       }
                }
 
        DEBUG_1("%s main: gtk_main", get_exec_time());