* 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
*/
-#include <signal.h>
-#include <sys/mman.h>
+#include "main.h"
+
+#include <unistd.h>
+
+#include <clocale>
+#include <csignal>
+#include <cstdlib>
+#include <cstring>
+#include <ctime>
+
+#include <config.h>
-#ifdef G_OS_UNIX
-#include <pwd.h>
+#if HAVE_CLUTTER
+# include <clutter-gtk/clutter-gtk.h>
+# include <clutter/clutter.h>
#endif
-#include <locale.h>
-#include "main.h"
+#if HAVE_EXECINFO_H
+#include <execinfo.h>
+#endif
+
+#include <gio/gio.h>
+#include <glib-object.h>
+#ifdef ENABLE_NLS
+# include <libintl.h>
+#endif
+
+#if HAVE_DEVELOPER
+#include "third-party/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 "layout.h"
+#include "main-defines.h"
+#include "metadata.h"
#include "misc.h"
+#include "options.h"
+#include "pixbuf-util.h"
#include "rcfile.h"
#include "remote.h"
#include "secure-save.h"
+#include "third-party/whereami.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;
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;
-#ifdef HAVE_EXECINFO_H
+ ssize_t len;
+#if HAVE_EXECINFO_H
gint bt_size;
void *bt[1024];
#endif
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++)
{
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
+#if HAVE_EXECINFO_H
bt_size = backtrace(bt, 1024);
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
+#if HAVE_EXECINFO_H
gint bt_size;
void *bt[1024];
#endif
write(STDERR_FILENO, "Geeqie fatal error\n", 19);
write(STDERR_FILENO, "Signal: Segmentation fault\n", 27);
-#ifdef HAVE_EXECINFO_H
+#if HAVE_EXECINFO_H
bt_size = backtrace(bt, 1024);
backtrace_symbols_fd(bt, bt_size, STDERR_FILENO);
#endif
}
}
-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.
{
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);
}
{
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)
{
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,
{
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)
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)
{
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"));
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();
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);
if (!command_line->path && first_dir)
{
command_line->path = first_dir;
- first_dir = NULL;
+ first_dir = nullptr;
parse_out_relatives(command_line->path);
}
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);
}
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);
}
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;
}
}
#endif
}
-#ifdef HAVE_CLUTTER
+#if HAVE_CLUTTER
static gboolean parse_command_line_for_clutter_option(gint argc, gchar *argv[])
{
const gchar *clutter_option = "--disable-clutter";
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;
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)
{
#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);
g_free(path);
}
-static void keys_load(void)
+static void keys_load()
{
gchar *path;
g_free(path);
}
-static void keys_save(void)
+static void keys_save()
{
gchar *path;
g_free(path);
}
-static void marks_load(void)
+static void marks_load()
{
gchar *path;
GdkModifierType accel_mods,
gboolean changed)
{
- GString *gstring = g_string_new(changed ? NULL : "; ");
- SecureSaveInfo *ssi = static_cast<SecureSaveInfo *>(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);
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))
{
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;
g_free(path);
}
-static void accel_map_load(void)
+static void accel_map_load()
{
gchar *path;
gchar *pathl;
g_free(path);
}
-static void gtkrc_load(void)
+static void gtkrc_load()
{
gchar *path;
gchar *pathl;
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;
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;
}
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);
}
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);
}
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)
{
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;
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);
* 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.
*
* 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,
}
#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)
+#if !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;
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;
}
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();
/**
* @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.
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);
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);
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,*");
gdk_threads_enter();
/* seg. fault handler */
-#ifdef HAVE_DEVELOPER
+#if HAVE_DEVELOPER
backward::SignalHandling sh{};
#else
setup_sig_handler();
exif_init();
-#ifdef HAVE_LUA
+#if HAVE_LUA
lua_init();
#endif
/* setup random seed for random slideshow */
- srand(time(NULL));
+ srand(time(nullptr));
#if 0
/* See later comment; this handler leads to UB. */
#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();
parse_command_line_for_debug_option(argc, argv);
DEBUG_1("%s main: gtk_init", get_exec_time());
-#ifdef HAVE_CLUTTER
+#if HAVE_CLUTTER
if (parse_command_line_for_clutter_option(argc, argv))
{
disable_clutter = TRUE;
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)
{
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);
filter_rebuild();
}
- #ifdef HAVE_CLUTTER
+ #if HAVE_CLUTTER
/** @FIXME For the background of this see:
* https://github.com/BestImageViewer/geeqie/issues/397
* The feature CLUTTER_FEATURE_SWAP_EVENTS indictates if the
CollectWindow *cw;
const gchar *path;
- path = work->data;
+ path = static_cast<const gchar *>(work->data);
work = work->next;
cw = collection_window_new(path);
if (command_line->cmd_list && !(command_line->startup_command_line_collection))
{
GList *work;
- gchar *path = NULL;
+ gchar *path = nullptr;
work = command_line->cmd_list;
{
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);
GList *work;
CollectWindow *cw;
- cw = collection_window_new(NULL);
+ cw = collection_window_new(nullptr);
cd = cw->cd;
collection_path_changed(cd);
{
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;
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)
}
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)
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;
}
/* 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());