Sort headers using clang-tidy
[geeqie.git] / src / pan-view / pan-view.cc
index 7cf3f92..8db42c2 100644 (file)
 
 #include "pan-view.h"
 
+#include <config.h>
+
 #include "bar-exif.h"
+#include "compat.h"
+#include "debug.h"
 #include "dnd.h"
 #include "editors.h"
 #include "exif.h"
 #include "fullscreen.h"
 #include "image.h"
 #include "img-view.h"
+#include "intl.h"
 #include "layout-util.h"
+#include "main-defines.h"
+#include "main.h"
 #include "menu.h"
 #include "metadata.h"
+#include "misc.h"
 #include "pan-calendar.h"
 #include "pan-folder.h"
 #include "pan-grid.h"
+#include "pan-item.h"
 #include "pan-timeline.h"
 #include "pan-util.h"
 #include "pan-view-filter.h"
 #include "pan-view-search.h"
+#include "pixbuf-renderer.h"
 #include "pixbuf-util.h"
 #include "thumb.h"
 #include "ui-fileops.h"
 #include "utilops.h"
 #include "window.h"
 
-#include <math.h>
+#include <cmath>
 
 
-#define PAN_WINDOW_DEFAULT_WIDTH 720
-#define PAN_WINDOW_DEFAULT_HEIGHT 500
+enum {
+       PAN_WINDOW_DEFAULT_WIDTH = 720,
+       PAN_WINDOW_DEFAULT_HEIGHT = 500
+};
 
-#define PAN_TILE_SIZE 512
+enum {
+       PAN_TILE_SIZE = 512
+};
 
 #define ZOOM_INCREMENT 1.0
-#define ZOOM_LABEL_WIDTH 64
+enum {
+       ZOOM_LABEL_WIDTH = 64
+};
 
 
 #define PAN_PREF_GROUP         "pan_view_options"
@@ -67,7 +83,7 @@
 #define PAN_PREF_INFO_EXIF     "info_includes_exif"
 
 
-static GList *pan_window_list = NULL;
+static GList *pan_window_list = nullptr;
 
 
 static void pan_layout_update_idle(PanWindow *pw);
@@ -125,7 +141,7 @@ hard_coded_window_keys pan_view_window_keys[] = {
        {static_cast<GdkModifierType>(0), GDK_KEY_Page_Down, N_("Scroll display half screen down")},
        {static_cast<GdkModifierType>(0), GDK_KEY_Home, N_("Scroll display half screen left")},
        {static_cast<GdkModifierType>(0), GDK_KEY_End, N_("Scroll display half screen right")},
-       {static_cast<GdkModifierType>(0), 0, NULL}
+       {static_cast<GdkModifierType>(0), 0, nullptr}
 };
 
 /*
@@ -147,7 +163,7 @@ static void pan_queue_thumb_done_cb(ThumbLoader *tl, gpointer data)
                gint rc;
 
                pi = pw->queue_pi;
-               pw->queue_pi = NULL;
+               pw->queue_pi = nullptr;
 
                pi->queued = FALSE;
 
@@ -160,7 +176,7 @@ static void pan_queue_thumb_done_cb(ThumbLoader *tl, gpointer data)
                }
 
        thumb_loader_free(pw->tl);
-       pw->tl = NULL;
+       pw->tl = nullptr;
 
        while (pan_queue_step(pw));
 }
@@ -168,7 +184,7 @@ static void pan_queue_thumb_done_cb(ThumbLoader *tl, gpointer data)
 static void pan_queue_image_done_cb(ImageLoader *il, gpointer data)
 {
        auto pw = static_cast<PanWindow *>(data);
-       GdkPixbuf *rotated = NULL;
+       GdkPixbuf *rotated = nullptr;
 
        if (pw->queue_pi)
                {
@@ -176,7 +192,7 @@ static void pan_queue_image_done_cb(ImageLoader *il, gpointer data)
                gint rc;
 
                pi = pw->queue_pi;
-               pw->queue_pi = NULL;
+               pw->queue_pi = nullptr;
 
                pi->queued = FALSE;
 
@@ -224,7 +240,7 @@ static void pan_queue_image_done_cb(ImageLoader *il, gpointer data)
                }
 
        image_loader_free(pw->il);
-       pw->il = NULL;
+       pw->il = nullptr;
 
        while (pan_queue_step(pw));
 }
@@ -242,14 +258,14 @@ static gboolean pan_queue_step(PanWindow *pw)
        if (!pw->queue_pi->fd)
                {
                pw->queue_pi->queued = FALSE;
-               pw->queue_pi = NULL;
+               pw->queue_pi = nullptr;
                return TRUE;
                }
 
        image_loader_free(pw->il);
-       pw->il = NULL;
+       pw->il = nullptr;
        thumb_loader_free(pw->tl);
-       pw->tl = NULL;
+       pw->tl = nullptr;
 
        if (pi->type == PAN_ITEM_IMAGE)
                {
@@ -266,7 +282,7 @@ static gboolean pan_queue_step(PanWindow *pw)
                if (image_loader_start(pw->il)) return FALSE;
 
                image_loader_free(pw->il);
-               pw->il = NULL;
+               pw->il = nullptr;
                }
        else if (pi->type == PAN_ITEM_THUMB)
                {
@@ -284,16 +300,16 @@ static gboolean pan_queue_step(PanWindow *pw)
                thumb_loader_set_callbacks(pw->tl,
                                           pan_queue_thumb_done_cb,
                                           pan_queue_thumb_done_cb,
-                                          NULL, pw);
+                                          nullptr, pw);
 
                if (thumb_loader_start(pw->tl, pi->fd)) return FALSE;
 
                thumb_loader_free(pw->tl);
-               pw->tl = NULL;
+               pw->tl = nullptr;
                }
 
        pw->queue_pi->queued = FALSE;
-       pw->queue_pi = NULL;
+       pw->queue_pi = nullptr;
        return TRUE;
 }
 
@@ -333,7 +349,10 @@ static gboolean pan_window_request_tile_cb(PixbufRenderer *pr, gint x, gint y,
 
        for (i = (x / PAN_GRID_SIZE) * PAN_GRID_SIZE; i < x + width; i += PAN_GRID_SIZE)
                {
-               gint rx, ry, rw, rh;
+               gint rx;
+               gint ry;
+               gint rw;
+               gint rh;
 
                if (util_clip_region(x, y, width, height,
                                     i, y, 1, height,
@@ -346,7 +365,10 @@ static gboolean pan_window_request_tile_cb(PixbufRenderer *pr, gint x, gint y,
                }
        for (i = (y / PAN_GRID_SIZE) * PAN_GRID_SIZE; i < y + height; i += PAN_GRID_SIZE)
                {
-               gint rx, ry, rw, rh;
+               gint rx;
+               gint ry;
+               gint rw;
+               gint rh;
 
                if (util_clip_region(x, y, width, height,
                                     x, i, width, 1,
@@ -400,8 +422,8 @@ static gboolean pan_window_request_tile_cb(PixbufRenderer *pr, gint x, gint y,
        return TRUE;
 }
 
-static void pan_window_dispose_tile_cb(PixbufRenderer *UNUSED(pr), gint x, gint y,
-                                      gint width, gint height, GdkPixbuf *UNUSED(pixbuf), gpointer data)
+static void pan_window_dispose_tile_cb(PixbufRenderer *, gint x, gint y,
+                                      gint width, gint height, GdkPixbuf *, gpointer data)
 {
        auto pw = static_cast<PanWindow *>(data);
        GList *list;
@@ -427,11 +449,11 @@ static void pan_window_dispose_tile_cb(PixbufRenderer *UNUSED(pr), gint x, gint
                                        pw->queue = g_list_remove(pw->queue, pi);
                                        pi->queued = FALSE;
                                        }
-                               if (pw->queue_pi == pi) pw->queue_pi = NULL;
+                               if (pw->queue_pi == pi) pw->queue_pi = nullptr;
                                if (pi->pixbuf)
                                        {
                                        g_object_unref(pi->pixbuf);
-                                       pi->pixbuf = NULL;
+                                       pi->pixbuf = nullptr;
                                        }
                                }
                        }
@@ -511,7 +533,7 @@ static void pan_warning_folder(const gchar *path, GtkWidget *parent)
 
        message = g_strdup_printf(_("The pan view does not support the folder \"%s\"."), path);
        warning_dialog(_("Folder not supported"), message,
-                     GTK_STOCK_DIALOG_INFO, parent);
+                     GQ_ICON_DIALOG_INFO, parent);
        g_free(message);
 }
 
@@ -561,41 +583,32 @@ static gint pan_cache_sort_file_cb(gpointer a, gpointer b)
        auto pcb = static_cast<PanCacheData *>(b);
        return filelist_sort_compare_filedata(pca->fd, pcb->fd);
 }
-GList *pan_cache_sort(GList *list, SortType method, gboolean ascend)
+GList *pan_cache_sort(GList *list, SortType method, gboolean ascend, gboolean case_sensitive)
 {
-       return filelist_sort_full(list, method, ascend, reinterpret_cast<GCompareFunc>(pan_cache_sort_file_cb));
+       return filelist_sort_full(list, method, ascend, case_sensitive, reinterpret_cast<GCompareFunc>(pan_cache_sort_file_cb));
 }
 
 
 static void pan_cache_free(PanWindow *pw)
 {
-       GList *work;
-
-       work = pw->cache_list;
-       while (work)
+       g_list_free_full(pw->cache_list, [](gpointer data)
                {
-               PanCacheData *pc;
-
-               pc = static_cast<PanCacheData *>(work->data);
-               work = work->next;
-
+               auto pc = static_cast<PanCacheData *>(data);
                cache_sim_data_free(pc->cd);
                file_data_unref(pc->fd);
                g_free(pc);
-               }
-
-       g_list_free(pw->cache_list);
-       pw->cache_list = NULL;
+               });
+       pw->cache_list = nullptr;
 
        filelist_free(pw->cache_todo);
-       pw->cache_todo = NULL;
+       pw->cache_todo = nullptr;
 
        pw->cache_count = 0;
        pw->cache_total = 0;
        pw->cache_tick = 0;
 
        cache_loader_free(pw->cache_cl);
-       pw->cache_cl = NULL;
+       pw->cache_cl = nullptr;
 }
 
 static void pan_cache_fill(PanWindow *pw, FileData *dir_fd)
@@ -604,13 +617,13 @@ static void pan_cache_fill(PanWindow *pw, FileData *dir_fd)
 
        pan_cache_free(pw);
 
-       list = pan_list_tree(dir_fd, SORT_NAME, TRUE, pw->ignore_symlinks);
+       list = pan_list_tree(dir_fd, SORT_NAME, TRUE, TRUE, pw->ignore_symlinks);
        pw->cache_todo = g_list_reverse(list);
 
        pw->cache_total = g_list_length(pw->cache_todo);
 }
 
-static void pan_cache_step_done_cb(CacheLoader *cl, gint UNUSED(error), gpointer data)
+static void pan_cache_step_done_cb(CacheLoader *cl, gint, gpointer data)
 {
        auto pw = static_cast<PanWindow *>(data);
 
@@ -622,12 +635,12 @@ static void pan_cache_step_done_cb(CacheLoader *cl, gint UNUSED(error), gpointer
                if (!pc->cd)
                        {
                        pc->cd = cl->cd;
-                       cl->cd = NULL;
+                       cl->cd = nullptr;
                        }
                }
 
        cache_loader_free(cl);
-       pw->cache_cl = NULL;
+       pw->cache_cl = nullptr;
 
        pan_layout_update_idle(pw);
 }
@@ -646,7 +659,7 @@ static gboolean pan_cache_step(PanWindow *pw)
        pc = g_new0(PanCacheData, 1);
        pc->fd = file_data_ref(fd);
 
-       pc->cd = NULL;
+       pc->cd = nullptr;
 
        pw->cache_list = g_list_prepend(pw->cache_list, pc);
 
@@ -657,7 +670,7 @@ static gboolean pan_cache_step(PanWindow *pw)
        if (pw->exif_date_enable) load_mask = static_cast<CacheDataType>(load_mask | CACHE_LOADER_DATE);
        pw->cache_cl = cache_loader_new(pc->fd, load_mask,
                                        pan_cache_step_done_cb, pw);
-       return (pw->cache_cl == NULL);
+       return (pw->cache_cl == nullptr);
 }
 
 /* This sync date function is optimized for lists with a common sort */
@@ -691,7 +704,7 @@ void pan_cache_sync_date(PanWindow *pw, GList *list)
                                        }
 
                                haystack = g_list_delete_link(haystack, needle);
-                               needle = NULL;
+                               needle = nullptr;
                                }
                        else
                                {
@@ -711,34 +724,28 @@ void pan_cache_sync_date(PanWindow *pw, GList *list)
 
 static void pan_grid_clear(PanWindow *pw)
 {
-       GList *work;
-
-       work = pw->list_grid;
-       while (work)
+       g_list_free_full(pw->list_grid, [](gpointer data)
                {
-               PanGrid *pg;
-
-               pg = static_cast<PanGrid *>(work->data);
-               work = work->next;
-
+               auto pg = static_cast<PanGrid *>(data);
                g_list_free(pg->list);
                g_free(pg);
-               }
-
-       g_list_free(pw->list_grid);
-       pw->list_grid = NULL;
+               });
+       pw->list_grid = nullptr;
 
        pw->list = g_list_concat(pw->list, pw->list_static);
-       pw->list_static = NULL;
+       pw->list_static = nullptr;
 }
 
 static void pan_grid_build(PanWindow *pw, gint width, gint height, gint grid_size)
 {
        GList *work;
-       gint col, row;
-       gint cw, ch;
+       gint col;
+       gint row;
+       gint cw;
+       gint ch;
        gint l;
-       gint i, j;
+       gint i;
+       gint j;
 
        pan_grid_clear(pw);
 
@@ -792,7 +799,10 @@ static void pan_grid_build(PanWindow *pw, gint width, gint height, gint grid_siz
                while (grid)
                        {
                        PanGrid *pg;
-                       gint rx, ry, rw, rh;
+                       gint rx;
+                       gint ry;
+                       gint rw;
+                       gint rh;
 
                        pg = static_cast<PanGrid *>(grid->data);
                        grid = grid->next;
@@ -818,7 +828,7 @@ static void pan_grid_build(PanWindow *pw, gint width, gint height, gint grid_siz
                }
 
        pw->list_static = pw->list;
-       pw->list = NULL;
+       pw->list = nullptr;
 }
 
 
@@ -830,34 +840,23 @@ static void pan_grid_build(PanWindow *pw, gint width, gint height, gint grid_siz
 
 static void pan_window_items_free(PanWindow *pw)
 {
-       GList *work;
-
        pan_grid_clear(pw);
 
-       work = pw->list;
-       while (work)
-               {
-               auto pi = static_cast<PanItem *>(work->data);
-               work = work->next;
-
-               pan_item_free(pi);
-               }
-
-       g_list_free(pw->list);
-       pw->list = NULL;
+       g_list_free_full(pw->list, reinterpret_cast<GDestroyNotify>(pan_item_free));
+       pw->list = nullptr;
 
        g_list_free(pw->queue);
-       pw->queue = NULL;
-       pw->queue_pi = NULL;
+       pw->queue = nullptr;
+       pw->queue_pi = nullptr;
 
        image_loader_free(pw->il);
-       pw->il = NULL;
+       pw->il = nullptr;
 
        thumb_loader_free(pw->tl);
-       pw->tl = NULL;
+       pw->tl = nullptr;
 
-       pw->click_pi = NULL;
-       pw->search_pi = NULL;
+       pw->click_pi = nullptr;
+       pw->search_pi = nullptr;
 }
 
 
@@ -957,7 +956,10 @@ static GList *pan_layout_intersect_l(GList *list, GList *item_list,
        while (work)
                {
                PanItem *pi;
-               gint rx, ry, rw, rh;
+               gint rx;
+               gint ry;
+               gint rw;
+               gint rh;
 
                pi = static_cast<PanItem *>(work->data);
                work = work->next;
@@ -975,9 +977,9 @@ static GList *pan_layout_intersect_l(GList *list, GList *item_list,
 
 GList *pan_layout_intersect(PanWindow *pw, gint x, gint y, gint width, gint height)
 {
-       GList *list = NULL;
+       GList *list = nullptr;
        GList *grid;
-       PanGrid *pg = NULL;
+       PanGrid *pg = nullptr;
 
        grid = pw->list_grid;
        while (grid && !pg)
@@ -988,7 +990,7 @@ GList *pan_layout_intersect(PanWindow *pw, gint x, gint y, gint width, gint heig
                if (x < pg->x || x + width > pg->x + pg->w ||
                    y < pg->y || y + height > pg->y + pg->h)
                        {
-                       pg = NULL;
+                       pg = nullptr;
                        }
                }
 
@@ -1121,7 +1123,7 @@ static gint pan_layout_update_idle_cb(gpointer data)
                pixbuf_renderer_scroll_to_point(PIXBUF_RENDERER(pw->imd->pr), scroll_x, scroll_y, align, align);
                }
 
-       pan_window_message(pw, NULL);
+       pan_window_message(pw, nullptr);
 
        pw->idle_id = 0;
        return G_SOURCE_REMOVE;
@@ -1167,7 +1169,7 @@ static void pan_layout_set_fd(PanWindow *pw, FileData *dir_fd)
 FileData *pan_menu_click_fd(PanWindow *pw)
 {
        if (pw->click_pi && pw->click_pi->fd) return pw->click_pi->fd;
-       return NULL;
+       return nullptr;
 }
 
 static gboolean pan_window_key_press_cb(GtkWidget *widget, GdkEventKey *event, gpointer data)
@@ -1257,19 +1259,19 @@ static gboolean pan_window_key_press_cb(GtkWidget *widget, GdkEventKey *event, g
                        case '0':
                                break;
                        case 'C': case 'c':
-                               if (fd) file_util_copy(fd, NULL, NULL, GTK_WIDGET(pr));
+                               if (fd) file_util_copy(fd, nullptr, nullptr, GTK_WIDGET(pr));
                                break;
                        case 'M': case 'm':
-                               if (fd) file_util_move(fd, NULL, NULL, GTK_WIDGET(pr));
+                               if (fd) file_util_move(fd, nullptr, nullptr, GTK_WIDGET(pr));
                                break;
                        case 'R': case 'r':
-                               if (fd) file_util_rename(fd, NULL, GTK_WIDGET(pr));
+                               if (fd) file_util_rename(fd, nullptr, GTK_WIDGET(pr));
                                break;
                        case 'D': case 'd':
                                if (fd)
                                        {
                                        options->file_ops.safe_delete_enable = TRUE;
-                                       file_util_delete(fd, NULL, GTK_WIDGET(pr));
+                                       file_util_delete(fd, nullptr, GTK_WIDGET(pr));
                                        }
                                break;
                        case 'F': case 'f':
@@ -1353,7 +1355,7 @@ static gboolean pan_window_key_press_cb(GtkWidget *widget, GdkEventKey *event, g
                                case GDK_KEY_Menu:
                                case GDK_KEY_F10:
                                        menu = pan_popup_menu(pw);
-                                       gtk_menu_popup_at_widget(GTK_MENU(menu), widget, GDK_GRAVITY_SOUTH, GDK_GRAVITY_CENTER, NULL);
+                                       gtk_menu_popup_at_widget(GTK_MENU(menu), widget, GDK_GRAVITY_SOUTH, GDK_GRAVITY_CENTER, nullptr);
 
                                        break;
                                case '/':
@@ -1405,7 +1407,7 @@ static void pan_info_add_exif(PanTextAlignment *ta, FileData *fd)
                g_free(text);
                }
 
-       string_list_free(exif_list);
+       g_list_free_full(exif_list, g_free);
 }
 
 
@@ -1415,11 +1417,19 @@ void pan_info_update(PanWindow *pw, PanItem *pi)
        PanItem *pbox;
        PanItem *p;
        gchar *buf;
-       gint x1, y1, x2, y2, x3, y3;
-       gint x, y, w, h;
+       gint x1;
+       gint y1;
+       gint x2;
+       gint y2;
+       gint x3;
+       gint y3;
+       gint x;
+       gint y;
+       gint w;
+       gint h;
 
        if (pw->click_pi == pi) return;
-       if (pi && !pi->fd) pi = NULL;
+       if (pi && !pi->fd) pi = nullptr;
 
        while ((p = pan_item_find_by_key(pw, PAN_ITEM_NONE, "info"))) pan_item_remove(pw, p);
        pw->click_pi = pi;
@@ -1428,7 +1438,7 @@ void pan_info_update(PanWindow *pw, PanItem *pi)
 
        DEBUG_1("info set to %s", pi->fd->path);
 
-       pbox = pan_item_box_new(pw, NULL, pi->x + pi->width + 4, pi->y, 10, 10,
+       pbox = pan_item_box_new(pw, nullptr, pi->x + pi->width + 4, pi->y, 10, 10,
                                PAN_POPUP_BORDER,
                                PAN_POPUP_COLOR, PAN_POPUP_ALPHA,
                                PAN_POPUP_BORDER_COLOR, PAN_POPUP_ALPHA);
@@ -1455,7 +1465,7 @@ void pan_info_update(PanWindow *pw, PanItem *pi)
        util_clip_triangle(x1, y1, x2, y2, x3, y3,
                           &x, &y, &w, &h);
 
-       p = pan_item_tri_new(pw, NULL, x, y, w, h,
+       p = pan_item_tri_new(pw, nullptr, x, y, w, h,
                             x1, y1, x2, y2, x3, y3,
                             PAN_POPUP_COLOR, PAN_POPUP_ALPHA);
        pan_item_tri_border(p, PAN_BORDER_1 | PAN_BORDER_3, PAN_POPUP_BORDER_COLOR, PAN_POPUP_ALPHA);
@@ -1486,7 +1496,8 @@ void pan_info_update(PanWindow *pw, PanItem *pi)
 
        if (pw->info_image_size > PAN_IMAGE_SIZE_THUMB_NONE)
                {
-               gint iw, ih;
+               gint iw;
+               gint ih;
                if (image_load_dimensions(pi->fd, &iw, &ih))
                        {
                        gint scale = 25;
@@ -1513,7 +1524,7 @@ void pan_info_update(PanWindow *pw, PanItem *pi)
                        iw = MAX(1, iw * scale / 100);
                        ih = MAX(1, ih * scale / 100);
 
-                       pbox = pan_item_box_new(pw, NULL, pbox->x, pbox->y + pbox->height + 8, 10, 10,
+                       pbox = pan_item_box_new(pw, nullptr, pbox->x, pbox->y + pbox->height + 8, 10, 10,
                                                PAN_POPUP_BORDER,
                                                PAN_POPUP_COLOR, PAN_POPUP_ALPHA,
                                                PAN_POPUP_BORDER_COLOR, PAN_POPUP_ALPHA);
@@ -1542,9 +1553,10 @@ void pan_info_update(PanWindow *pw, PanItem *pi)
 static void button_cb(PixbufRenderer *pr, GdkEventButton *event, gpointer data)
 {
        auto pw = static_cast<PanWindow *>(data);
-       PanItem *pi = NULL;
+       PanItem *pi = nullptr;
        GtkWidget *menu;
-       gint rx, ry;
+       gint rx;
+       gint ry;
 
        rx = ry = 0;
        if (pr->scale)
@@ -1556,12 +1568,12 @@ static void button_cb(PixbufRenderer *pr, GdkEventButton *event, gpointer data)
        pi = pan_item_find_by_coord(pw, PAN_ITEM_BOX, rx, ry, "info");
        if (pi && event->button == MOUSE_BUTTON_LEFT)
                {
-               pan_info_update(pw, NULL);
+               pan_info_update(pw, nullptr);
                return;
                }
 
        pi = pan_item_find_by_coord(pw, (pw->size > PAN_IMAGE_SIZE_THUMB_LARGE) ? PAN_ITEM_IMAGE : PAN_ITEM_THUMB,
-                                   rx, ry, NULL);
+                                   rx, ry, nullptr);
 
        switch (event->button)
                {
@@ -1579,16 +1591,17 @@ static void button_cb(PixbufRenderer *pr, GdkEventButton *event, gpointer data)
                case MOUSE_BUTTON_RIGHT:
                        pan_info_update(pw, pi);
                        menu = pan_popup_menu(pw);
-                       gtk_menu_popup_at_pointer(GTK_MENU(menu), NULL);
+                       gtk_menu_popup_at_pointer(GTK_MENU(menu), nullptr);
                        break;
                default:
                        break;
                }
 }
 
-static void scroll_cb(PixbufRenderer *pr, GdkEventScroll *event, gpointer UNUSED(data))
+static void scroll_cb(PixbufRenderer *pr, GdkEventScroll *event, gpointer)
 {
-       gint w, h;
+       gint w;
+       gint h;
 
        w = pr->vis_width;
        h = pr->vis_height;
@@ -1645,11 +1658,11 @@ static void pan_image_set_buttons(PanWindow *pw, ImageWindow *imd)
                         G_CALLBACK(scroll_cb), pw);
 }
 
-static void pan_fullscreen_stop_func(FullScreenData *UNUSED(fs), gpointer data)
+static void pan_fullscreen_stop_func(FullScreenData *, gpointer data)
 {
        auto pw = static_cast<PanWindow *>(data);
 
-       pw->fs = NULL;
+       pw->fs = nullptr;
        pw->imd = pw->imd_normal;
 }
 
@@ -1672,7 +1685,7 @@ static void pan_fullscreen_toggle(PanWindow *pw, gboolean force_off)
                }
 }
 
-static void pan_window_image_zoom_cb(PixbufRenderer *UNUSED(pr), gdouble UNUSED(zoom), gpointer data)
+static void pan_window_image_zoom_cb(PixbufRenderer *, gdouble, gpointer data)
 {
        auto pw = static_cast<PanWindow *>(data);
        gchar *text;
@@ -1687,7 +1700,8 @@ static void pan_window_image_scroll_notify_cb(PixbufRenderer *pr, gpointer data)
        auto pw = static_cast<PanWindow *>(data);
        GtkAdjustment *adj;
        GdkRectangle rect;
-       gint width, height;
+       gint width;
+       gint height;
 
        if (pr->scale == 0.0) return;
 
@@ -1703,8 +1717,6 @@ static void pan_window_image_scroll_notify_cb(PixbufRenderer *pr, gpointer data)
        gtk_adjustment_set_value(adj, static_cast<gdouble>(rect.x));
 
        pref_signal_block_data(pw->scrollbar_h, pw);
-       gtk_adjustment_changed(adj);
-       gtk_adjustment_value_changed(adj);
        pref_signal_unblock_data(pw->scrollbar_h, pw);
 
        adj = gtk_range_get_adjustment(GTK_RANGE(pw->scrollbar_v));
@@ -1716,8 +1728,6 @@ static void pan_window_image_scroll_notify_cb(PixbufRenderer *pr, gpointer data)
        gtk_adjustment_set_value(adj, static_cast<gdouble>(rect.y));
 
        pref_signal_block_data(pw->scrollbar_v, pw);
-       gtk_adjustment_changed(adj);
-       gtk_adjustment_value_changed(adj);
        pref_signal_unblock_data(pw->scrollbar_v, pw);
 }
 
@@ -1779,7 +1789,7 @@ static void pan_window_entry_activate_cb(const gchar *new_text, gpointer data)
                {
                warning_dialog(_("Folder not found"),
                               _("The entered path is not a folder"),
-                              GTK_STOCK_DIALOG_WARNING, pw->path_entry);
+                              GQ_ICON_DIALOG_WARNING, pw->path_entry);
                }
        else
                {
@@ -1809,7 +1819,7 @@ static void pan_window_close(PanWindow *pw)
        pan_fullscreen_toggle(pw, TRUE);
        pan_search_ui_destroy(&pw->search_ui);
        pan_filter_ui_destroy(&pw->filter_ui);
-       gtk_widget_destroy(pw->window);
+       gq_gtk_widget_destroy(pw->window);
 
        pan_window_items_free(pw);
        pan_cache_free(pw);
@@ -1819,7 +1829,7 @@ static void pan_window_close(PanWindow *pw)
        g_free(pw);
 }
 
-static gboolean pan_window_delete_cb(GtkWidget *UNUSED(w), GdkEventAny *UNUSED(event), gpointer data)
+static gboolean pan_window_delete_cb(GtkWidget *, GdkEventAny *, gpointer data)
 {
        auto pw = static_cast<PanWindow *>(data);
 
@@ -1863,12 +1873,12 @@ static void pan_window_new_real(FileData *dir_fd)
 
        pw->idle_id = 0;
 
-       pw->window = window_new(GTK_WINDOW_TOPLEVEL, "panview", NULL, NULL, _("Pan View"));
+       pw->window = window_new("panview", nullptr, nullptr, _("Pan View"));
        DEBUG_NAME(pw->window);
 
        geometry.min_width = DEFAULT_MINIMAL_WINDOW_SIZE;
        geometry.min_height = DEFAULT_MINIMAL_WINDOW_SIZE;
-       gtk_window_set_geometry_hints(GTK_WINDOW(pw->window), NULL, &geometry, GDK_HINT_MIN_SIZE);
+       gtk_window_set_geometry_hints(GTK_WINDOW(pw->window), nullptr, &geometry, GDK_HINT_MIN_SIZE);
 
        gtk_window_set_resizable(GTK_WINDOW(pw->window), TRUE);
        gtk_container_set_border_width(GTK_CONTAINER(pw->window), 0);
@@ -1884,7 +1894,7 @@ static void pan_window_new_real(FileData *dir_fd)
        pref_label_new(box, _("Location:"));
        combo = tab_completion_new_with_history(&pw->path_entry, dir_fd->path, "pan_view_path", -1,
                                                pan_window_entry_activate_cb, pw);
-       gtk_box_pack_start(GTK_BOX(box), combo, TRUE, TRUE, 0);
+       gq_gtk_box_pack_start(GTK_BOX(box), combo, TRUE, TRUE, 0);
        gtk_widget_show(combo);
 
        combo = gtk_combo_box_text_new();
@@ -1897,7 +1907,7 @@ static void pan_window_new_real(FileData *dir_fd)
        gtk_combo_box_set_active(GTK_COMBO_BOX(combo), pw->layout);
        g_signal_connect(G_OBJECT(combo), "changed",
                         G_CALLBACK(pan_window_layout_change_cb), pw);
-       gtk_box_pack_start(GTK_BOX(box), combo, FALSE, FALSE, 0);
+       gq_gtk_box_pack_start(GTK_BOX(box), combo, FALSE, FALSE, 0);
        gtk_widget_show(combo);
 
        combo = gtk_combo_box_text_new();
@@ -1915,21 +1925,14 @@ static void pan_window_new_real(FileData *dir_fd)
        gtk_combo_box_set_active(GTK_COMBO_BOX(combo), pw->size);
        g_signal_connect(G_OBJECT(combo), "changed",
                         G_CALLBACK(pan_window_layout_size_cb), pw);
-       gtk_box_pack_start(GTK_BOX(box), combo, FALSE, FALSE, 0);
+       gq_gtk_box_pack_start(GTK_BOX(box), combo, FALSE, FALSE, 0);
        gtk_widget_show(combo);
 
        table = pref_table_new(vbox, 2, 2, FALSE, TRUE);
-       gtk_table_set_row_spacings(GTK_TABLE(table), 2);
-       gtk_table_set_col_spacings(GTK_TABLE(table), 2);
-
-       /** @FIXME pan view does not work correctly when renderer-clutter()
-        * is used.
-        */
-       gint temp = options->image.use_clutter_renderer;
-       options->image.use_clutter_renderer = FALSE;
-       pw->imd = image_new(TRUE);
-       options->image.use_clutter_renderer = temp;
+       gtk_grid_set_row_spacing(GTK_GRID(table), 2);
+       gtk_grid_set_column_spacing(GTK_GRID(table), 2);
 
+       pw->imd = image_new(TRUE);
        pw->imd_normal = pw->imd;
 
        g_signal_connect(G_OBJECT(pw->imd->pr), "zoom",
@@ -1937,46 +1940,43 @@ static void pan_window_new_real(FileData *dir_fd)
        g_signal_connect(G_OBJECT(pw->imd->pr), "scroll_notify",
                         G_CALLBACK(pan_window_image_scroll_notify_cb), pw);
 
-       gtk_table_attach(GTK_TABLE(table), pw->imd->widget, 0, 1, 0, 1,
-                        static_cast<GtkAttachOptions>(GTK_FILL | GTK_EXPAND), static_cast<GtkAttachOptions>(GTK_FILL | GTK_EXPAND), 0, 0);
+       gq_gtk_grid_attach(GTK_GRID(table), pw->imd->widget, 0, 1, 0, 1, static_cast<GtkAttachOptions>(GTK_FILL | GTK_EXPAND), static_cast<GtkAttachOptions>(GTK_FILL | GTK_EXPAND), 0, 0);
        gtk_widget_show(GTK_WIDGET(pw->imd->widget));
 
        pan_window_dnd_init(pw);
 
        pan_image_set_buttons(pw, pw->imd);
 
-       pw->scrollbar_h = gtk_hscrollbar_new(NULL);
+       pw->scrollbar_h = gtk_scrollbar_new(GTK_ORIENTATION_HORIZONTAL, nullptr);
        g_signal_connect(G_OBJECT(pw->scrollbar_h), "value_changed",
                         G_CALLBACK(pan_window_scrollbar_h_value_cb), pw);
-       gtk_table_attach(GTK_TABLE(table), pw->scrollbar_h, 0, 1, 1, 2,
-                        static_cast<GtkAttachOptions>(GTK_FILL | GTK_EXPAND), static_cast<GtkAttachOptions>(0), 0, 0);
+       gq_gtk_grid_attach(GTK_GRID(table), pw->scrollbar_h, 0, 1, 1, 2,  static_cast<GtkAttachOptions>(GTK_FILL | GTK_EXPAND), static_cast<GtkAttachOptions>(0), 0, 0);
        gtk_widget_show(pw->scrollbar_h);
 
-       pw->scrollbar_v = gtk_vscrollbar_new(NULL);
+       pw->scrollbar_v = gtk_scrollbar_new(GTK_ORIENTATION_VERTICAL, nullptr);
        g_signal_connect(G_OBJECT(pw->scrollbar_v), "value_changed",
                         G_CALLBACK(pan_window_scrollbar_v_value_cb), pw);
-       gtk_table_attach(GTK_TABLE(table), pw->scrollbar_v, 1, 2, 0, 1,
-                        static_cast<GtkAttachOptions>(0), static_cast<GtkAttachOptions>(GTK_FILL | GTK_EXPAND), 0, 0);
+       gq_gtk_grid_attach(GTK_GRID(table), pw->scrollbar_v, 1, 2, 0, 1,  static_cast<GtkAttachOptions>(0), static_cast<GtkAttachOptions>(GTK_FILL | GTK_EXPAND), 0, 0);
        gtk_widget_show(pw->scrollbar_v);
 
        /* find bar */
 
        pw->search_ui = pan_search_ui_new(pw);
-       gtk_box_pack_start(GTK_BOX(vbox), pw->search_ui->search_box, FALSE, FALSE, 2);
+       gq_gtk_box_pack_start(GTK_BOX(vbox), pw->search_ui->search_box, FALSE, FALSE, 2);
 
     /* filter bar */
     pw->filter_ui = pan_filter_ui_new(pw);
-    gtk_box_pack_start(GTK_BOX(vbox), pw->filter_ui->filter_box, FALSE, FALSE, 2);
+    gq_gtk_box_pack_start(GTK_BOX(vbox), pw->filter_ui->filter_box, FALSE, FALSE, 2);
 
        /* status bar */
 
        box = pref_box_new(vbox, FALSE, GTK_ORIENTATION_HORIZONTAL, 0);
 
-       frame = gtk_frame_new(NULL);
+       frame = gtk_frame_new(nullptr);
        DEBUG_NAME(frame);
-       gtk_frame_set_shadow_type(GTK_FRAME(frame), GTK_SHADOW_IN);
+       gq_gtk_frame_set_shadow_type(GTK_FRAME(frame), GTK_SHADOW_IN);
        gtk_widget_set_size_request(frame, ZOOM_LABEL_WIDTH, -1);
-       gtk_box_pack_start(GTK_BOX(box), frame, TRUE, TRUE, 0);
+       gq_gtk_box_pack_start(GTK_BOX(box), frame, TRUE, TRUE, 0);
        gtk_widget_show(frame);
 
        hbox = gtk_box_new(GTK_ORIENTATION_HORIZONTAL, PREF_PAD_SPACE);
@@ -1986,11 +1986,11 @@ static void pan_window_new_real(FileData *dir_fd)
        pref_spacer(hbox, 0);
        pw->label_message = pref_label_new(hbox, "");
 
-       frame = gtk_frame_new(NULL);
+       frame = gtk_frame_new(nullptr);
        DEBUG_NAME(frame);
-       gtk_frame_set_shadow_type(GTK_FRAME(frame), GTK_SHADOW_IN);
+       gq_gtk_frame_set_shadow_type(GTK_FRAME(frame), GTK_SHADOW_IN);
        gtk_widget_set_size_request(frame, ZOOM_LABEL_WIDTH, -1);
-       gtk_box_pack_end(GTK_BOX(box), frame, FALSE, FALSE, 0);
+       gq_gtk_box_pack_end(GTK_BOX(box), frame, FALSE, FALSE, 0);
        gtk_widget_show(frame);
 
        pw->label_zoom = gtk_label_new("");
@@ -1998,11 +1998,11 @@ static void pan_window_new_real(FileData *dir_fd)
        gtk_widget_show(pw->label_zoom);
 
        // Add the "Find" button to the status bar area.
-       gtk_box_pack_end(GTK_BOX(box), pw->search_ui->search_button, FALSE, FALSE, 0);
+       gq_gtk_box_pack_end(GTK_BOX(box), pw->search_ui->search_button, FALSE, FALSE, 0);
        gtk_widget_show(pw->search_ui->search_button);
 
        // Add the "Filter" button to the status bar area.
-       gtk_box_pack_end(GTK_BOX(box), pw->filter_ui->filter_button, FALSE, FALSE, 0);
+       gq_gtk_box_pack_end(GTK_BOX(box), pw->filter_ui->filter_button, FALSE, FALSE, 0);
        gtk_widget_show(pw->filter_ui->filter_button);
 
        g_signal_connect(G_OBJECT(pw->window), "delete_event",
@@ -2036,7 +2036,7 @@ static void pan_warning_ok_cb(GenericDialog *gd, gpointer data)
        file_data_unref(dir_fd);
 }
 
-static void pan_warning_hide_cb(GtkWidget *button, gpointer UNUSED(data))
+static void pan_warning_hide_cb(GtkWidget *button, gpointer)
 {
        gboolean hide_dlg;
 
@@ -2049,13 +2049,13 @@ static gboolean pan_warning(FileData *dir_fd)
        GenericDialog *gd;
        GtkWidget *box;
        GtkWidget *group;
-       GtkWidget *button;
+       GtkWidget *checkbox;
        GtkWidget *ct_button;
        gboolean hide_dlg;
 
        if (dir_fd && strcmp(dir_fd->path, G_DIR_SEPARATOR_S) == 0)
                {
-               pan_warning_folder(dir_fd->path, NULL);
+               pan_warning_folder(dir_fd->path, nullptr);
                return TRUE;
                }
 
@@ -2065,13 +2065,13 @@ static gboolean pan_warning(FileData *dir_fd)
        if (!pref_list_int_get(PAN_PREF_GROUP, PAN_PREF_HIDE_WARNING, &hide_dlg)) hide_dlg = FALSE;
        if (hide_dlg) return FALSE;
 
-       gd = generic_dialog_new(_("Pan View Performance"), "pan_view_warning", NULL, FALSE,
-                               NULL, NULL);
+       gd = generic_dialog_new(_("Pan View Performance"), "pan_view_warning", nullptr, FALSE,
+                               nullptr, nullptr);
        gd->data = file_data_ref(dir_fd);
-       generic_dialog_add_button(gd, GTK_STOCK_OK, NULL,
+       generic_dialog_add_button(gd, GQ_ICON_OK, "OK",
                                  pan_warning_ok_cb, TRUE);
 
-       box = generic_dialog_add_message(gd, GTK_STOCK_DIALOG_INFO,
+       box = generic_dialog_add_message(gd, GQ_ICON_DIALOG_INFO,
                                         _("Pan view performance may be poor."),
                                         _("To improve the performance of thumbnails in\npan view the following options can be enabled.\n\nNote that both options must be enabled to\nnotice a change in performance."), TRUE);
 
@@ -2081,14 +2081,14 @@ static gboolean pan_warning(FileData *dir_fd)
 
        ct_button = pref_checkbox_new_int(group, _("Cache thumbnails"),
                                          options->thumbnails.enable_caching, &options->thumbnails.enable_caching);
-       button = pref_checkbox_new_int(group, _("Use shared thumbnail cache"),
+       checkbox = pref_checkbox_new_int(group, _("Use shared thumbnail cache"),
                                       options->thumbnails.spec_standard, &options->thumbnails.spec_standard);
-       pref_checkbox_link_sensitivity(ct_button, button);
+       pref_checkbox_link_sensitivity(ct_button, checkbox);
 
        pref_line(box, 0);
 
        pref_checkbox_new(box, _("Do not show this dialog again"), hide_dlg,
-                         G_CALLBACK(pan_warning_hide_cb), NULL);
+                         G_CALLBACK(pan_warning_hide_cb), nullptr);
 
        gtk_widget_show(gd->dialog);
 
@@ -2119,7 +2119,7 @@ void pan_window_new(FileData *dir_fd)
 #define INFO_IMAGE_SIZE_KEY "image_size_data"
 
 
-static void pan_new_window_cb(GtkWidget *UNUSED(widget), gpointer data)
+static void pan_new_window_cb(GtkWidget *, gpointer data)
 {
        auto pw = static_cast<PanWindow *>(data);
        FileData *fd;
@@ -2132,9 +2132,9 @@ static void pan_new_window_cb(GtkWidget *UNUSED(widget), gpointer data)
                }
 }
 
-static void pan_go_to_original_cb(GtkWidget *UNUSED(widget), gpointer data)
+static void pan_go_to_original_cb(GtkWidget *, gpointer data)
 {
-       LayoutWindow *lw = NULL;
+       LayoutWindow *lw = nullptr;
        auto pw = static_cast<PanWindow *>(data);
        FileData *fd;
 
@@ -2167,55 +2167,55 @@ static void pan_edit_cb(GtkWidget *widget, gpointer data)
                }
 }
 
-static void pan_zoom_in_cb(GtkWidget *UNUSED(widget), gpointer data)
+static void pan_zoom_in_cb(GtkWidget *, gpointer data)
 {
        auto pw = static_cast<PanWindow *>(data);
 
        image_zoom_adjust(pw->imd, ZOOM_INCREMENT);
 }
 
-static void pan_zoom_out_cb(GtkWidget *UNUSED(widget), gpointer data)
+static void pan_zoom_out_cb(GtkWidget *, gpointer data)
 {
        auto pw = static_cast<PanWindow *>(data);
 
        image_zoom_adjust(pw->imd, -ZOOM_INCREMENT);
 }
 
-static void pan_zoom_1_1_cb(GtkWidget *UNUSED(widget), gpointer data)
+static void pan_zoom_1_1_cb(GtkWidget *, gpointer data)
 {
        auto pw = static_cast<PanWindow *>(data);
 
        image_zoom_set(pw->imd, 1.0);
 }
 
-static void pan_copy_cb(GtkWidget *UNUSED(widget), gpointer data)
+static void pan_copy_cb(GtkWidget *, gpointer data)
 {
        auto pw = static_cast<PanWindow *>(data);
        FileData *fd;
 
        fd = pan_menu_click_fd(pw);
-       if (fd) file_util_copy(fd, NULL, NULL, pw->imd->widget);
+       if (fd) file_util_copy(fd, nullptr, nullptr, pw->imd->widget);
 }
 
-static void pan_move_cb(GtkWidget *UNUSED(widget), gpointer data)
+static void pan_move_cb(GtkWidget *, gpointer data)
 {
        auto pw = static_cast<PanWindow *>(data);
        FileData *fd;
 
        fd = pan_menu_click_fd(pw);
-       if (fd) file_util_move(fd, NULL, NULL, pw->imd->widget);
+       if (fd) file_util_move(fd, nullptr, nullptr, pw->imd->widget);
 }
 
-static void pan_rename_cb(GtkWidget *UNUSED(widget), gpointer data)
+static void pan_rename_cb(GtkWidget *, gpointer data)
 {
        auto pw = static_cast<PanWindow *>(data);
        FileData *fd;
 
        fd = pan_menu_click_fd(pw);
-       if (fd) file_util_rename(fd, NULL, pw->imd->widget);
+       if (fd) file_util_rename(fd, nullptr, pw->imd->widget);
 }
 
-static void pan_delete_cb(GtkWidget *UNUSED(widget), gpointer data)
+static void pan_delete_cb(GtkWidget *, gpointer data)
 {
        auto pw = static_cast<PanWindow *>(data);
        FileData *fd;
@@ -2224,11 +2224,11 @@ static void pan_delete_cb(GtkWidget *UNUSED(widget), gpointer data)
        if (fd)
                {
                options->file_ops.safe_delete_enable = FALSE;
-               file_util_delete(fd, NULL, pw->imd->widget);
+               file_util_delete(fd, nullptr, pw->imd->widget);
                }
 }
 
-static void pan_move_to_trash_cb(GtkWidget *UNUSED(widget), gpointer data)
+static void pan_move_to_trash_cb(GtkWidget *, gpointer data)
 {
        auto pw = static_cast<PanWindow *>(data);
        FileData *fd;
@@ -2237,11 +2237,11 @@ static void pan_move_to_trash_cb(GtkWidget *UNUSED(widget), gpointer data)
        if (fd)
                {
                options->file_ops.safe_delete_enable = TRUE;
-               file_util_delete(fd, NULL, pw->imd->widget);
+               file_util_delete(fd, nullptr, pw->imd->widget);
                }
 }
 
-static void pan_copy_path_cb(GtkWidget *UNUSED(widget), gpointer data)
+static void pan_copy_path_cb(GtkWidget *, gpointer data)
 {
        auto pw = static_cast<PanWindow *>(data);
        FileData *fd;
@@ -2250,7 +2250,7 @@ static void pan_copy_path_cb(GtkWidget *UNUSED(widget), gpointer data)
        if (fd) file_util_copy_path_to_clipboard(fd, TRUE);
 }
 
-static void pan_copy_path_unquoted_cb(GtkWidget *UNUSED(widget), gpointer data)
+static void pan_copy_path_unquoted_cb(GtkWidget *, gpointer data)
 {
        auto pw = static_cast<PanWindow *>(data);
        FileData *fd;
@@ -2283,28 +2283,28 @@ static void pan_info_toggle_image_cb(GtkWidget *widget, gpointer data)
        /** @FIXME sync info now */
 }
 
-static void pan_fullscreen_cb(GtkWidget *UNUSED(widget), gpointer data)
+static void pan_fullscreen_cb(GtkWidget *, gpointer data)
 {
        auto pw = static_cast<PanWindow *>(data);
 
        pan_fullscreen_toggle(pw, FALSE);
 }
 
-static void pan_close_cb(GtkWidget *UNUSED(widget), gpointer data)
+static void pan_close_cb(GtkWidget *, gpointer data)
 {
        auto pw = static_cast<PanWindow *>(data);
 
        pan_window_close(pw);
 }
 
-static void pan_popup_menu_destroy_cb(GtkWidget *UNUSED(widget), gpointer data)
+static void pan_popup_menu_destroy_cb(GtkWidget *, gpointer data)
 {
        auto editmenu_fd_list = static_cast<GList *>(data);
 
        filelist_free(editmenu_fd_list);
 }
 
-static void pan_play_cb(GtkWidget *UNUSED(widget), gpointer data)
+static void pan_play_cb(GtkWidget *, gpointer data)
 {
        auto pw = static_cast<PanWindow *>(data);
 
@@ -2313,7 +2313,7 @@ static void pan_play_cb(GtkWidget *UNUSED(widget), gpointer data)
 
 static GList *pan_view_get_fd_list(PanWindow *pw)
 {
-       GList *list = NULL;
+       GList *list = nullptr;
        FileData *fd = pan_menu_click_fd(pw);
 
        if (fd) list = g_list_prepend(filelist_copy(fd->sidecar_files), file_data_ref(fd));
@@ -2331,7 +2331,7 @@ static GList *pan_view_get_fd_list(PanWindow *pw)
 static void pan_pop_menu_collections_cb(GtkWidget *widget, gpointer data)
 {
        PanWindow *pw;
-       GList *selection_list = NULL;
+       GList *selection_list = nullptr;
 
        pw = static_cast<PanWindow *>(submenu_item_get_data(widget));
        selection_list = g_list_append(selection_list, pan_menu_click_fd(pw));
@@ -2345,11 +2345,12 @@ static GtkWidget *pan_popup_menu(PanWindow *pw)
        GtkWidget *menu;
        GtkWidget *submenu;
        GtkWidget *item;
-       gboolean active, video;
+       gboolean active;
+       gboolean video;
        GList *editmenu_fd_list;
        GtkAccelGroup *accel_group;
 
-       active = (pw->click_pi != NULL);
+       active = (pw->click_pi != nullptr);
        video = (active && pw->click_pi->fd && pw->click_pi->fd->format_class == FORMAT_CLASS_VIDEO);
 
        menu = popup_menu_short_lived();
@@ -2359,15 +2360,15 @@ static GtkWidget *pan_popup_menu(PanWindow *pw)
        g_object_set_data(G_OBJECT(menu), "window_keys", pan_view_window_keys);
        g_object_set_data(G_OBJECT(menu), "accel_group", accel_group);
 
-       menu_item_add_stock_sensitive(menu, _("_Play"), GTK_STOCK_MEDIA_PLAY, video,
+       menu_item_add_icon_sensitive(menu, _("_Play"), GQ_ICON_PLAY, video,
                            G_CALLBACK(pan_play_cb), pw);
        menu_item_add_divider(menu);
 
-       menu_item_add_stock(menu, _("Zoom _in"), GTK_STOCK_ZOOM_IN,
+       menu_item_add_icon(menu, _("Zoom _in"), GQ_ICON_ZOOM_IN,
                            G_CALLBACK(pan_zoom_in_cb), pw);
-       menu_item_add_stock(menu, _("Zoom _out"), GTK_STOCK_ZOOM_OUT,
+       menu_item_add_icon(menu, _("Zoom _out"), GQ_ICON_ZOOM_OUT,
                            G_CALLBACK(pan_zoom_out_cb), pw);
-       menu_item_add_stock(menu, _("Zoom _1:1"), GTK_STOCK_ZOOM_100,
+       menu_item_add_icon(menu, _("Zoom _1:1"), GQ_ICON_ZOOM_100,
                            G_CALLBACK(pan_zoom_1_1_cb), pw);
        menu_item_add_divider(menu);
 
@@ -2378,13 +2379,13 @@ static GtkWidget *pan_popup_menu(PanWindow *pw)
        submenu_add_edit(menu, &item, G_CALLBACK(pan_edit_cb), pw, editmenu_fd_list);
        gtk_widget_set_sensitive(item, active);
 
-       menu_item_add_stock_sensitive(menu, _("View in _new window"), GTK_STOCK_NEW, active,
+       menu_item_add_icon_sensitive(menu, _("View in _new window"), GQ_ICON_NEW, active,
                                      G_CALLBACK(pan_new_window_cb), pw);
-       menu_item_add_stock(menu, _("Go to original"), GTK_STOCK_FIND,
+       menu_item_add_icon(menu, _("Go to original"), GQ_ICON_FIND,
                        G_CALLBACK(pan_go_to_original_cb), pw);
 
        menu_item_add_divider(menu);
-       menu_item_add_stock_sensitive(menu, _("_Copy..."), GTK_STOCK_COPY, active,
+       menu_item_add_icon_sensitive(menu, _("_Copy..."), GQ_ICON_COPY, active,
                                      G_CALLBACK(pan_copy_cb), pw);
        menu_item_add_sensitive(menu, _("_Move..."), active,
                                G_CALLBACK(pan_move_cb), pw);
@@ -2396,13 +2397,13 @@ static GtkWidget *pan_popup_menu(PanWindow *pw)
                                G_CALLBACK(pan_copy_path_unquoted_cb), pw);
 
        menu_item_add_divider(menu);
-       menu_item_add_stock_sensitive(menu,
+       menu_item_add_icon_sensitive(menu,
                                options->file_ops.confirm_move_to_trash ? _("Move to Trash...") :
-                                       _("Move to Trash"), PIXBUF_INLINE_ICON_TRASH, active,
+                                       _("Move to Trash"), GQ_ICON_DELETE, active,
                                                G_CALLBACK(pan_move_to_trash_cb), pw);
-       menu_item_add_stock_sensitive(menu,
+       menu_item_add_icon_sensitive(menu,
                                options->file_ops.confirm_delete ? _("_Delete...") :
-                                       _("_Delete"), GTK_STOCK_DELETE, active,
+                                       _("_Delete"), GQ_ICON_DELETE_SHRED, active,
                                                G_CALLBACK(pan_delete_cb), pw);
 
        menu_item_add_divider(menu);
@@ -2421,7 +2422,7 @@ static GtkWidget *pan_popup_menu(PanWindow *pw)
 
        menu_item_add_check(menu, _("_Show Exif information"), pw->info_includes_exif,
                            G_CALLBACK(pan_info_toggle_exif_cb), pw);
-       item = menu_item_add(menu, _("Show im_age"), NULL, NULL);
+       item = menu_item_add(menu, _("Show im_age"), nullptr, nullptr);
        submenu = gtk_menu_new();
        gtk_menu_item_set_submenu(GTK_MENU_ITEM(item), submenu);
 
@@ -2455,15 +2456,15 @@ static GtkWidget *pan_popup_menu(PanWindow *pw)
 
        if (pw->fs)
                {
-               menu_item_add(menu, _("Exit _full screen"), G_CALLBACK(pan_fullscreen_cb), pw);
+               menu_item_add_icon(menu, _("Exit _full screen"), GQ_ICON_LEAVE_FULLSCREEN, G_CALLBACK(pan_fullscreen_cb), pw);
                }
        else
                {
-               menu_item_add(menu, _("_Full screen"), G_CALLBACK(pan_fullscreen_cb), pw);
+               menu_item_add_icon(menu, _("_Full screen"), GQ_ICON_FULLSCREEN, G_CALLBACK(pan_fullscreen_cb), pw);
                }
 
        menu_item_add_divider(menu);
-       menu_item_add_stock(menu, _("C_lose window"), GTK_STOCK_CLOSE, G_CALLBACK(pan_close_cb), pw);
+       menu_item_add_icon(menu, _("C_lose window"), GQ_ICON_CLOSE, G_CALLBACK(pan_close_cb), pw);
 
        return menu;
 }
@@ -2475,10 +2476,10 @@ static GtkWidget *pan_popup_menu(PanWindow *pw)
  *-----------------------------------------------------------------------------
  */
 
-static void pan_window_get_dnd_data(GtkWidget *UNUSED(widget), GdkDragContext *context,
-                                   gint UNUSED(x), gint UNUSED(y),
+static void pan_window_get_dnd_data(GtkWidget *, GdkDragContext *context,
+                                   gint, gint,
                                    GtkSelectionData *selection_data, guint info,
-                                   guint UNUSED(time), gpointer data)
+                                   guint, gpointer data)
 {
        auto pw = static_cast<PanWindow *>(data);
 
@@ -2500,9 +2501,9 @@ static void pan_window_get_dnd_data(GtkWidget *UNUSED(widget), GdkDragContext *c
                }
 }
 
-static void pan_window_set_dnd_data(GtkWidget *UNUSED(widget), GdkDragContext *UNUSED(context),
-                                   GtkSelectionData *selection_data, guint UNUSED(info),
-                                   guint UNUSED(time), gpointer data)
+static void pan_window_set_dnd_data(GtkWidget *, GdkDragContext *,
+                                   GtkSelectionData *selection_data, guint,
+                                   guint, gpointer data)
 {
        auto pw = static_cast<PanWindow *>(data);
        FileData *fd;
@@ -2512,14 +2513,14 @@ static void pan_window_set_dnd_data(GtkWidget *UNUSED(widget), GdkDragContext *U
                {
                GList *list;
 
-               list = g_list_append(NULL, fd);
+               list = g_list_append(nullptr, fd);
                uri_selection_data_set_uris_from_filelist(selection_data, list);
                g_list_free(list);
                }
        else
                {
                gtk_selection_data_set(selection_data, gtk_selection_data_get_target(selection_data),
-                                      8, NULL, 0);
+                                      8, nullptr, 0);
                }
 }