Geocode image by drag-drop
[geeqie.git] / src / search.c
index a8c5b21..f48fef7 100644 (file)
@@ -1,35 +1,43 @@
 /*
- * Geeqie
- * (C) 2005 John Ellis
- * Copyright (C) 2008 The Geeqie Team
+ * Copyright (C) 2005 John Ellis
+ * Copyright (C) 2008 - 2016 The Geeqie Team
  *
  * Author: John Ellis
  *
- * This software is released under the GNU General Public License (GNU GPL).
- * Please read the included file COPYING for more information.
- * This software comes with no warranty of any kind, use at your own risk!
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License along
+ * with this program; if not, write to the Free Software Foundation, Inc.,
+ * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
  */
 
-
 #include "main.h"
 #include "search.h"
 
-#include "bar_info.h"
 #include "cache.h"
 #include "collect.h"
 #include "collect-table.h"
 #include "dnd.h"
 #include "dupe.h"
-#include "image-load.h"
-#include "info.h"
 #include "editors.h"
-#include "img-view.h"
 #include "filedata.h"
-#include "layout_image.h"
+#include "image-load.h"
+#include "img-view.h"
+#include "layout.h"
+#include "math.h"
 #include "menu.h"
+#include "metadata.h"
+#include "misc.h"
 #include "print.h"
 #include "thumb.h"
-#include "utilops.h"
 #include "ui_bookmark.h"
 #include "ui_fileops.h"
 #include "ui_menu.h"
 #include "ui_spinner.h"
 #include "ui_tabcomp.h"
 #include "ui_tree_edit.h"
+#include "uri_utils.h"
+#include "utilops.h"
 #include "window.h"
+#include "bar_keywords.h"
 
 #include <gdk/gdkkeysyms.h> /* for keyboard values */
 
@@ -50,7 +61,6 @@
 #define SEARCH_BUFFER_MATCH_MISS 1
 #define SEARCH_BUFFER_FLUSH_SIZE 99
 
-
 typedef enum {
        SEARCH_MATCH_NONE,
        SEARCH_MATCH_EQUAL,
@@ -124,10 +134,14 @@ struct _SearchData
        GtkWidget *menu_keywords;
        GtkWidget *entry_keywords;
 
-       gchar *search_path;
-       gint   search_path_recurse;
+       GtkWidget *check_comment;
+       GtkWidget *menu_comment;
+       GtkWidget *entry_comment;
+
+       FileData *search_dir_fd;
+       gboolean   search_path_recurse;
        gchar *search_name;
-       gint   search_name_match_case;
+       gboolean   search_name_match_case;
        gint64 search_size;
        gint64 search_size_end;
        gint   search_date_y;
@@ -144,6 +158,9 @@ struct _SearchData
        gchar *search_similarity_path;
        CacheData *search_similarity_cd;
        GList *search_keyword_list;
+       gchar *search_comment;
+       gboolean   search_comment_match_case;
+       gboolean   search_date_exif;
 
        MatchType search_type;
 
@@ -152,6 +169,8 @@ struct _SearchData
        MatchType match_date;
        MatchType match_dimensions;
        MatchType match_keywords;
+       MatchType match_comment;
+       MatchType match_gps;
 
        gboolean match_name_enable;
        gboolean match_size_enable;
@@ -159,6 +178,7 @@ struct _SearchData
        gboolean match_dimensions_enable;
        gboolean match_similarity_enable;
        gboolean match_keywords_enable;
+       gboolean match_comment_enable;
 
        GList *search_folder_list;
        GList *search_done_list;
@@ -169,8 +189,8 @@ struct _SearchData
        gint search_total;
        gint search_buffer_count;
 
-       gint search_idle_id;
-       gint update_idle_id;
+       guint search_idle_id; /* event source id */
+       guint update_idle_id; /* event source id */
 
        ImageLoader *img_loader;
        CacheData   *img_cd;
@@ -178,8 +198,20 @@ struct _SearchData
        FileData *click_fd;
 
        ThumbLoader *thumb_loader;
-       gint thumb_enable;
+       gboolean thumb_enable;
        FileData *thumb_fd;
+
+       /* Used for lat/long coordinate search
+       */
+       gint search_gps;
+       gdouble search_lat, search_lon;
+       GtkWidget *entry_gps_coord;
+       GtkWidget *check_gps;
+       GtkWidget *spin_gps;
+       GtkWidget *units_gps;
+       GtkWidget *menu_gps;
+       gboolean match_gps_enable;
+
 };
 
 typedef struct _MatchFileData MatchFileData;
@@ -229,6 +261,17 @@ static const MatchList text_search_menu_keyword[] = {
        { N_("exclude"),        SEARCH_MATCH_NONE }
 };
 
+static const MatchList text_search_menu_comment[] = {
+       { N_("contains"),       SEARCH_MATCH_CONTAINS },
+       { N_("miss"),           SEARCH_MATCH_NONE }
+};
+
+static const MatchList text_search_menu_gps[] = {
+       { N_("not geocoded"),   SEARCH_MATCH_NONE },
+       { N_("less than"),      SEARCH_MATCH_UNDER },
+       { N_("greater than"),   SEARCH_MATCH_OVER }
+};
+
 static GList *search_window_list = NULL;
 
 
@@ -237,6 +280,7 @@ static gint search_result_count(SearchData *sd, gint64 *bytes);
 
 static void search_window_close(SearchData *sd);
 
+static void search_notify_cb(FileData *fd, NotifyType type, gpointer data);
 
 /*
  *-------------------------------------------------------------------
@@ -270,7 +314,7 @@ static void search_status_update(SearchData *sd)
 
        t = search_result_count(sd, &t_bytes);
        s = search_result_selection_count(sd, &s_bytes);
-       
+
        tt = text_from_size_abrev(t_bytes);
 
        if (s > 0)
@@ -290,7 +334,7 @@ static void search_status_update(SearchData *sd)
        g_free(buf);
 }
 
-static void search_progress_update(SearchData *sd, gint search, gdouble thumbs)
+static void search_progress_update(SearchData *sd, gboolean search, gdouble thumbs)
 {
 
        if (search || thumbs >= 0.0)
@@ -327,7 +371,7 @@ static void search_progress_update(SearchData *sd, gint search, gdouble thumbs)
 static gint search_result_find_row(SearchData *sd, FileData *fd, GtkTreeIter *iter)
 {
        GtkTreeModel *store;
-       gint valid;
+       gboolean valid;
        gint n = 0;
 
        store = gtk_tree_view_get_model(GTK_TREE_VIEW(sd->result_view));
@@ -345,13 +389,13 @@ static gint search_result_find_row(SearchData *sd, FileData *fd, GtkTreeIter *it
        return -1;
 }
 
-static gint search_result_row_selected(SearchData *sd, FileData *fd)
+static gboolean search_result_row_selected(SearchData *sd, FileData *fd)
 {
        GtkTreeModel *store;
        GtkTreeSelection *selection;
        GList *slist;
        GList *work;
-       gint found = FALSE;
+       gboolean found = FALSE;
 
        selection = gtk_tree_view_get_selection(GTK_TREE_VIEW(sd->result_view));
        slist = gtk_tree_selection_get_selected_rows(selection, &store);
@@ -432,7 +476,7 @@ static gint search_result_util(SearchData *sd, gint64 *bytes, GList **list)
 {
        GtkTreeModel *store;
        GtkTreeIter iter;
-       gint valid;
+       gboolean valid;
        gint n = 0;
        gint64 total = 0;
        GList *plist = NULL;
@@ -494,7 +538,7 @@ static void search_result_append(SearchData *sd, MatchFileData *mfd)
        gtk_list_store_set(store, &iter,
                                SEARCH_COLUMN_POINTER, mfd,
                                SEARCH_COLUMN_RANK, mfd->rank,
-                               SEARCH_COLUMN_THUMB, fd->pixbuf,
+                               SEARCH_COLUMN_THUMB, fd->thumb_pixbuf,
                                SEARCH_COLUMN_NAME, fd->name,
                                SEARCH_COLUMN_SIZE, text_size,
                                SEARCH_COLUMN_DATE, text_from_time(fd->date),
@@ -511,7 +555,7 @@ static GList *search_result_refine_list(SearchData *sd)
        GList *list = NULL;
        GtkTreeModel *store;
        GtkTreeIter iter;
-       gint valid;
+       gboolean valid;
 
        store = gtk_tree_view_get_model(GTK_TREE_VIEW(sd->result_view));
 
@@ -583,7 +627,7 @@ static void search_result_remove(SearchData *sd, FileData *fd)
 {
        GtkTreeModel *store;
        GtkTreeIter iter;
-       gint valid;
+       gboolean valid;
 
        store = gtk_tree_view_get_model(GTK_TREE_VIEW(sd->result_view));
        valid = gtk_tree_model_get_iter_first(store, &iter);
@@ -640,13 +684,9 @@ static void search_result_remove_selection(SearchData *sd)
        search_status_update(sd);
 }
 
-static void search_result_edit_selected(SearchData *sd, gint n)
+static void search_result_edit_selected(SearchData *sd, const gchar *key)
 {
-       GList *list;
-
-       list = search_result_selection_list(sd);
-       start_editor_from_filelist(n, list);
-       filelist_free(list);
+       file_util_start_editor_from_filelist(key, search_result_selection_list(sd), NULL, sd->window);
 }
 
 static void search_result_collection_from_selection(SearchData *sd)
@@ -660,20 +700,23 @@ static void search_result_collection_from_selection(SearchData *sd)
        filelist_free(list);
 }
 
-static gint search_result_update_idle_cb(gpointer data)
+static gboolean search_result_update_idle_cb(gpointer data)
 {
        SearchData *sd = data;
 
        search_status_update(sd);
 
-       sd->update_idle_id = -1;
+       sd->update_idle_id = 0;
        return FALSE;
 }
 
 static void search_result_update_idle_cancel(SearchData *sd)
 {
-       if (sd->update_idle_id != -1) g_source_remove(sd->update_idle_id);
-       sd->update_idle_id = -1;
+       if (sd->update_idle_id)
+               {
+               g_source_remove(sd->update_idle_id);
+               sd->update_idle_id = 0;
+               }
 }
 
 static gboolean search_result_select_cb(GtkTreeSelection *selection, GtkTreeModel *store,
@@ -681,7 +724,7 @@ static gboolean search_result_select_cb(GtkTreeSelection *selection, GtkTreeMode
 {
        SearchData *sd = data;
 
-       if (sd->update_idle_id == -1)
+       if (!sd->update_idle_id)
                {
                sd->update_idle_id = g_idle_add(search_result_update_idle_cb, sd);
                }
@@ -709,7 +752,7 @@ static void search_result_thumb_set(SearchData *sd, FileData *fd, GtkTreeIter *i
                if (search_result_find_row(sd, fd, &iter_n) >= 0) iter = &iter_n;
                }
 
-       if (iter) gtk_list_store_set(store, iter, SEARCH_COLUMN_THUMB, fd->pixbuf, -1);
+       if (iter) gtk_list_store_set(store, iter, SEARCH_COLUMN_THUMB, fd->thumb_pixbuf, -1);
 }
 
 static void search_result_thumb_do(SearchData *sd)
@@ -719,9 +762,6 @@ static void search_result_thumb_do(SearchData *sd)
        if (!sd->thumb_loader || !sd->thumb_fd) return;
        fd = sd->thumb_fd;
 
-       if (fd->pixbuf) g_object_unref(fd->pixbuf);
-       fd->pixbuf = thumb_loader_get_pixbuf(sd->thumb_loader, TRUE);
-
        search_result_thumb_set(sd, fd, NULL);
 }
 
@@ -738,7 +778,7 @@ static void search_result_thumb_step(SearchData *sd)
        GtkTreeModel *store;
        GtkTreeIter iter;
        MatchFileData *mfd = NULL;
-       gint valid;
+       gboolean valid;
        gint row = 0;
        gint length = 0;
 
@@ -760,9 +800,9 @@ static void search_result_thumb_step(SearchData *sd)
 
                length++;
                gtk_tree_model_get(store, &iter, SEARCH_COLUMN_POINTER, &mfd, SEARCH_COLUMN_THUMB, &pixbuf, -1);
-               if (pixbuf || mfd->fd->pixbuf)
+               if (pixbuf || mfd->fd->thumb_pixbuf)
                        {
-                       if (!pixbuf) gtk_list_store_set(GTK_LIST_STORE(store), &iter, SEARCH_COLUMN_THUMB, mfd->fd->pixbuf, -1);
+                       if (!pixbuf) gtk_list_store_set(GTK_LIST_STORE(store), &iter, SEARCH_COLUMN_THUMB, mfd->fd->thumb_pixbuf, -1);
                        row++;
                        mfd = NULL;
                        }
@@ -794,7 +834,7 @@ static void search_result_thumb_step(SearchData *sd)
                                   search_result_thumb_done_cb,
                                   NULL,
                                   sd);
-       if (!thumb_loader_start(sd->thumb_loader, mfd->fd->path))
+       if (!thumb_loader_start(sd->thumb_loader, mfd->fd))
                {
                search_result_thumb_do(sd);
                search_result_thumb_step(sd);
@@ -812,7 +852,7 @@ static void search_result_thumb_height(SearchData *sd)
 
        gtk_tree_view_column_set_fixed_width(column, (sd->thumb_enable) ? options->thumbnails.max_width : 4);
 
-       list = gtk_tree_view_column_get_cell_renderers(column);
+       list = gtk_cell_layout_get_cells(GTK_CELL_LAYOUT(column));
        if (!list) return;
        cell = list->data;
        g_list_free(list);
@@ -821,7 +861,7 @@ static void search_result_thumb_height(SearchData *sd)
        gtk_tree_view_columns_autosize(GTK_TREE_VIEW(sd->result_view));
 }
 
-static void search_result_thumb_enable(SearchData *sd, gint enable)
+static void search_result_thumb_enable(SearchData *sd, gboolean enable)
 {
        if (sd->thumb_enable == enable) return;
 
@@ -829,7 +869,7 @@ static void search_result_thumb_enable(SearchData *sd, gint enable)
                {
                GtkTreeModel *store;
                GtkTreeIter iter;
-               gint valid;
+               gboolean valid;
 
                thumb_loader_free(sd->thumb_loader);
                sd->thumb_loader = NULL;
@@ -860,7 +900,7 @@ static void sr_menu_view_cb(GtkWidget *widget, gpointer data)
 {
        SearchData *sd = data;
 
-       if (sd->click_fd) layout_image_set_fd(NULL, sd->click_fd);
+       if (sd->click_fd) layout_set_fd(NULL, sd->click_fd);
 }
 
 static void sr_menu_viewnew_cb(GtkWidget *widget, gpointer data)
@@ -894,20 +934,12 @@ static void sr_menu_select_none_cb(GtkWidget *widget, gpointer data)
 static void sr_menu_edit_cb(GtkWidget *widget, gpointer data)
 {
        SearchData *sd;
-       gint n;
+       const gchar *key = data;
 
        sd = submenu_item_get_data(widget);
-       n = GPOINTER_TO_INT(data);
        if (!sd) return;
 
-       search_result_edit_selected(sd, n);
-}
-
-static void sr_menu_info_cb(GtkWidget *widget, gpointer data)
-{
-       SearchData *sd = data;
-
-       info_window_new(NULL, search_result_selection_list(sd), NULL);
+       search_result_edit_selected(sd, key);
 }
 
 static void sr_menu_collection_cb(GtkWidget *widget, gpointer data)
@@ -974,12 +1006,21 @@ static void sr_menu_clear_cb(GtkWidget *widget, gpointer data)
        search_result_clear(sd);
 }
 
-static GtkWidget *search_result_menu(SearchData *sd, gint on_row, gint empty)
+static void search_result_menu_destroy_cb(GtkWidget *widget, gpointer data)
+{
+       GList *editmenu_fd_list = data;
+
+       filelist_free(editmenu_fd_list);
+}
+
+static GtkWidget *search_result_menu(SearchData *sd, gboolean on_row, gboolean empty)
 {
        GtkWidget *menu;
        GtkWidget *item;
+       GList *editmenu_fd_list;
 
        menu = popup_menu_short_lived();
+
        menu_item_add_sensitive(menu, _("_View"), on_row,
                                G_CALLBACK(sr_menu_view_cb), sd);
        menu_item_add_stock_sensitive(menu, _("View in _new window"), GTK_STOCK_NEW, on_row,
@@ -990,10 +1031,12 @@ static GtkWidget *search_result_menu(SearchData *sd, gint on_row, gint empty)
        menu_item_add_sensitive(menu, _("Select none"), !empty,
                                G_CALLBACK(sr_menu_select_none_cb), sd);
        menu_item_add_divider(menu);
-       submenu_add_edit(menu, &item, G_CALLBACK(sr_menu_edit_cb), sd);
+
+       editmenu_fd_list = search_result_selection_list(sd);
+       g_signal_connect(G_OBJECT(menu), "destroy",
+                        G_CALLBACK(search_result_menu_destroy_cb), editmenu_fd_list);
+       submenu_add_edit(menu, &item, G_CALLBACK(sr_menu_edit_cb), sd, editmenu_fd_list);
        if (!on_row) gtk_widget_set_sensitive(item, FALSE);
-       menu_item_add_stock_sensitive(menu, _("_Properties"), GTK_STOCK_PROPERTIES, on_row,
-                                     G_CALLBACK(sr_menu_info_cb), sd);
        menu_item_add_stock_sensitive(menu, _("Add to new collection"), GTK_STOCK_INDEX, on_row,
                                      G_CALLBACK(sr_menu_collection_cb), sd);
        menu_item_add_stock_sensitive(menu, _("Print..."), GTK_STOCK_PRINT, on_row,
@@ -1007,9 +1050,8 @@ static GtkWidget *search_result_menu(SearchData *sd, gint on_row, gint empty)
                                G_CALLBACK(sr_menu_rename_cb), sd);
        menu_item_add_stock_sensitive(menu, _("_Delete..."), GTK_STOCK_DELETE, on_row,
                                      G_CALLBACK(sr_menu_delete_cb), sd);
-       if (options->show_copy_path)
-               menu_item_add_sensitive(menu, _("_Copy path"), on_row,
-                                       G_CALLBACK(sr_menu_copy_path_cb), sd);
+       menu_item_add_sensitive(menu, _("_Copy path"), on_row,
+                               G_CALLBACK(sr_menu_copy_path_cb), sd);
        menu_item_add_divider(menu);
        menu_item_add_stock_sensitive(menu, _("Rem_ove"), GTK_STOCK_REMOVE, on_row,
                                      G_CALLBACK(sr_menu_remove_cb), sd);
@@ -1043,7 +1085,7 @@ static void search_result_menu_pos_cb(GtkMenu *menu, gint *x, gint *y, gboolean
  *-------------------------------------------------------------------
  */
 
-static gint search_result_press_cb(GtkWidget *widget, GdkEventButton *bevent, gpointer data)
+static gboolean search_result_press_cb(GtkWidget *widget, GdkEventButton *bevent, gpointer data)
 {
        SearchData *sd = data;
        GtkTreeModel *store;
@@ -1075,7 +1117,7 @@ static gint search_result_press_cb(GtkWidget *widget, GdkEventButton *bevent, gp
 
        if (bevent->button == MOUSE_BUTTON_LEFT && bevent->type == GDK_2BUTTON_PRESS)
                {
-               layout_image_set_fd(NULL, mfd->fd);
+               layout_set_fd(NULL, mfd->fd);
                }
 
        if (bevent->button == MOUSE_BUTTON_MIDDLE) return TRUE;
@@ -1110,7 +1152,7 @@ static gint search_result_press_cb(GtkWidget *widget, GdkEventButton *bevent, gp
        return FALSE;
 }
 
-static gint search_result_release_cb(GtkWidget *widget, GdkEventButton *bevent, gpointer data)
+static gboolean search_result_release_cb(GtkWidget *widget, GdkEventButton *bevent, gpointer data)
 {
        SearchData *sd = data;
        GtkTreeModel *store;
@@ -1172,10 +1214,10 @@ static gint search_result_release_cb(GtkWidget *widget, GdkEventButton *bevent,
        return FALSE;
 }
 
-static gint search_result_keypress_cb(GtkWidget *widget, GdkEventKey *event, gpointer data)
+static gboolean search_result_keypress_cb(GtkWidget *widget, GdkEventKey *event, gpointer data)
 {
        SearchData *sd = data;
-       gint stop_signal = FALSE;
+       gboolean stop_signal = FALSE;
        GtkTreeModel *store;
        GtkTreeSelection *selection;
        GList *slist;
@@ -1201,40 +1243,19 @@ static gint search_result_keypress_cb(GtkWidget *widget, GdkEventKey *event, gpo
 
        if (event->state & GDK_CONTROL_MASK)
                {
-               gint edit_val = -1;
-
                stop_signal = TRUE;
                switch (event->keyval)
                        {
                        case '1':
-                               edit_val = 0;
-                               break;
                        case '2':
-                               edit_val = 1;
-                               break;
                        case '3':
-                               edit_val = 2;
-                               break;
                        case '4':
-                               edit_val = 3;
-                               break;
                        case '5':
-                               edit_val = 4;
-                               break;
                        case '6':
-                               edit_val = 5;
-                               break;
                        case '7':
-                               edit_val = 6;
-                               break;
                        case '8':
-                               edit_val = 7;
-                               break;
                        case '9':
-                               edit_val = 8;
-                               break;
                        case '0':
-                               edit_val = 9;
                                break;
                        case 'C': case 'c':
                                file_util_copy(NULL, search_result_selection_list(sd), NULL, widget);
@@ -1248,9 +1269,6 @@ static gint search_result_keypress_cb(GtkWidget *widget, GdkEventKey *event, gpo
                        case 'D': case 'd':
                                file_util_delete(NULL, search_result_selection_list(sd), widget);
                                break;
-                       case 'P': case 'p':
-                               info_window_new(NULL,  search_result_selection_list(sd), NULL);
-                               break;
                        case 'A': case 'a':
                                if (event->state & GDK_SHIFT_MASK)
                                        {
@@ -1261,26 +1279,21 @@ static gint search_result_keypress_cb(GtkWidget *widget, GdkEventKey *event, gpo
                                        gtk_tree_selection_select_all(selection);
                                        }
                                break;
-                       case GDK_Delete: case GDK_KP_Delete:
+                       case GDK_KEY_Delete: case GDK_KEY_KP_Delete:
                                search_result_clear(sd);
                                break;
                        default:
                                stop_signal = FALSE;
                                break;
                        }
-
-               if (edit_val >= 0)
-                       {
-                       search_result_edit_selected(sd, edit_val);
-                       }
                }
        else
                {
                stop_signal = TRUE;
                switch (event->keyval)
                        {
-                       case GDK_Return: case GDK_KP_Enter:
-                               if (mfd) layout_image_set_fd(NULL, mfd->fd);
+                       case GDK_KEY_Return: case GDK_KEY_KP_Enter:
+                               if (mfd) layout_set_fd(NULL, mfd->fd);
                                break;
                        case 'V': case 'v':
                                {
@@ -1291,23 +1304,30 @@ static gint search_result_keypress_cb(GtkWidget *widget, GdkEventKey *event, gpo
                                filelist_free(list);
                                }
                                break;
-                       case GDK_Delete: case GDK_KP_Delete:
+                       case GDK_KEY_Delete: case GDK_KEY_KP_Delete:
                                search_result_remove_selection(sd);
                                break;
                        case 'C': case 'c':
                                search_result_collection_from_selection(sd);
                                break;
-                       case GDK_Menu:
-                       case GDK_F10:
+                       case GDK_KEY_Menu:
+                       case GDK_KEY_F10:
                                {
                                GtkWidget *menu;
 
-                               sd->click_fd = mfd->fd;
+                               if (mfd)
+                                       sd->click_fd = mfd->fd;
+                               else
+                                       sd->click_fd = NULL;
+
                                menu = search_result_menu(sd, (mfd != NULL), (search_result_count(sd, NULL) > 0));
                                gtk_menu_popup(GTK_MENU(menu), NULL, NULL,
                                               search_result_menu_pos_cb, sd, 0, GDK_CURRENT_TIME);
                                }
                                break;
+                       case GDK_KEY_F1:
+                               help_window_show("GuideReferenceKeyboardShortcuts.html#SearchKeyboardShortcuts");
+                               break;
                        default:
                                stop_signal = FALSE;
                                break;
@@ -1317,10 +1337,10 @@ static gint search_result_keypress_cb(GtkWidget *widget, GdkEventKey *event, gpo
        return stop_signal;
 }
 
-static gint search_window_keypress_cb(GtkWidget *widget, GdkEventKey *event, gpointer data)
+static gboolean search_window_keypress_cb(GtkWidget *widget, GdkEventKey *event, gpointer data)
 {
        SearchData *sd = data;
-       gint stop_signal = FALSE;
+       gboolean stop_signal = FALSE;
 
        if (event->state & GDK_CONTROL_MASK)
                {
@@ -1355,32 +1375,24 @@ static GtkTargetEntry result_drag_types[] = {
 };
 static gint n_result_drag_types = 2;
 
+static GtkTargetEntry result_drop_types[] = {
+       { "text/uri-list", 0, TARGET_URI_LIST },
+       { "text/plain", 0, TARGET_TEXT_PLAIN }
+};
+static gint n_result_drop_types = 2;
+
 static void search_dnd_data_set(GtkWidget *widget, GdkDragContext *context,
                                GtkSelectionData *selection_data, guint info,
                                guint time, gpointer data)
 {
        SearchData *sd = data;
-       gchar *uri_text;
-       gint length;
        GList *list;
 
-       switch (info)
-               {
-               case TARGET_URI_LIST:
-               case TARGET_TEXT_PLAIN:
-                       list = search_result_selection_list(sd);
-                       if (!list) return;
-                       uri_text = uri_text_from_filelist(list, &length, (info == TARGET_TEXT_PLAIN));
-                       filelist_free(list);
-                       break;
-               default:
-                       uri_text = NULL;
-                       break;
-               }
+       list = search_result_selection_list(sd);
+       if (!list) return;
 
-       if (uri_text) gtk_selection_data_set(selection_data, selection_data->target,
-                                            8, (guchar *)uri_text, length);
-       g_free(uri_text);
+       uri_selection_data_set_uris_from_filelist(selection_data, list);
+       filelist_free(list);
 }
 
 static void search_dnd_begin(GtkWidget *widget, GdkDragContext *context, gpointer data)
@@ -1409,9 +1421,48 @@ static void search_dnd_begin(GtkWidget *widget, GdkDragContext *context, gpointe
                }
 
        if (sd->thumb_enable &&
-           sd->click_fd && sd->click_fd->pixbuf)
+           sd->click_fd && sd->click_fd->thumb_pixbuf)
                {
-               dnd_set_drag_icon(widget, context, sd->click_fd->pixbuf, search_result_selection_count(sd, NULL));
+               dnd_set_drag_icon(widget, context, sd->click_fd->thumb_pixbuf, search_result_selection_count(sd, NULL));
+               }
+}
+
+static void search_gps_dnd_received_cb(GtkWidget *pane, GdkDragContext *context,
+                                                                               gint x, gint y,
+                                                                               GtkSelectionData *selection_data, guint info,
+                                                                               guint time, gpointer data)
+{
+       SearchData *sd = data;
+       GList *list;
+       gdouble latitude, longitude;
+       FileData *fd;
+
+       if (info == TARGET_URI_LIST)
+               {
+               list = uri_filelist_from_gtk_selection_data(selection_data);
+
+               /* If more than one file, use only the first file in a list.
+               */
+               if (list != NULL)
+                       {
+                       fd = list->data;
+                       latitude = metadata_read_GPS_coord(fd, "Xmp.exif.GPSLatitude", 1000);
+                       longitude = metadata_read_GPS_coord(fd, "Xmp.exif.GPSLongitude", 1000);
+                       if (latitude != 1000 && longitude != 1000)
+                               {
+                               gtk_entry_set_text(GTK_ENTRY(sd->entry_gps_coord),
+                                                       g_strdup_printf("%lf %lf", latitude, longitude));
+                               }
+                       else
+                               {
+                               gtk_entry_set_text(GTK_ENTRY(sd->entry_gps_coord), "Image is not geocoded");
+                               }
+                       }
+               }
+
+       if (info == TARGET_TEXT_PLAIN)
+               {
+               gtk_entry_set_text(GTK_ENTRY(sd->entry_gps_coord),"");
                }
 }
 
@@ -1424,11 +1475,14 @@ static void search_dnd_init(SearchData *sd)
                         G_CALLBACK(search_dnd_data_set), sd);
        g_signal_connect(G_OBJECT(sd->result_view), "drag_begin",
                         G_CALLBACK(search_dnd_begin), sd);
-#if 0
-       g_signal_connect(G_OBJECT(sd->result_view), "drag_end",
-                        G_CALLBACK(search_dnd_end), sd);
-#endif
 
+       gtk_drag_dest_set(GTK_WIDGET(sd->entry_gps_coord),
+                                        GTK_DEST_DEFAULT_ALL,
+                                         result_drop_types, n_result_drop_types,
+                                        GDK_ACTION_COPY);
+
+       g_signal_connect(G_OBJECT(sd->entry_gps_coord), "drag_data_received",
+                                       G_CALLBACK(search_gps_dnd_received_cb), sd);
 }
 
 /*
@@ -1439,7 +1493,7 @@ static void search_dnd_init(SearchData *sd)
 
 #define MATCH_IS_BETWEEN(val, a, b)  (b > a ? (val >= a && val <= b) : (val >= b && val <= a))
 
-static gint search_step_cb(gpointer data);
+static gboolean search_step_cb(gpointer data);
 
 
 static void search_buffer_flush(SearchData *sd)
@@ -1462,10 +1516,10 @@ static void search_buffer_flush(SearchData *sd)
 
 static void search_stop(SearchData *sd)
 {
-       if (sd->search_idle_id != -1)
+       if (sd->search_idle_id)
                {
                g_source_remove(sd->search_idle_id);
-               sd->search_idle_id = -1;
+               sd->search_idle_id = 0;
                }
 
        image_loader_free(sd->img_loader);
@@ -1519,21 +1573,21 @@ static void search_file_load_process(SearchData *sd, CacheData *cd)
                        }
 
                if (options->thumbnails.enable_caching &&
-                   sd->img_loader && sd->img_loader->fd)
+                   sd->img_loader && image_loader_get_fd(sd->img_loader))
                        {
                        gchar *base;
                        const gchar *path;
                        mode_t mode = 0755;
 
-                       path = sd->img_loader->fd->path;
+                       path = image_loader_get_fd(sd->img_loader)->path;
                        base = cache_get_location(CACHE_TYPE_SIM, path, FALSE, &mode);
-                       if (cache_ensure_dir_exists(base, mode))
+                       if (recursive_mkdir_if_not_exists(base, mode))
                                {
                                g_free(cd->path);
                                cd->path = cache_get_location(CACHE_TYPE_SIM, path, TRUE, NULL);
                                if (cache_sim_data_save(cd))
                                        {
-                                       filetime_set(cd->path, filetime(sd->img_loader->fd->path));
+                                       filetime_set(cd->path, filetime(image_loader_get_fd(sd->img_loader)->path));
                                        }
                                }
                        g_free(base);
@@ -1552,12 +1606,12 @@ static void search_file_load_done_cb(ImageLoader *il, gpointer data)
        search_file_load_process(sd, sd->img_cd);
 }
 
-static gint search_file_do_extra(SearchData *sd, FileData *fd, gint *match,
-                                gint *width, gint *height, gint *simval)
+static gboolean search_file_do_extra(SearchData *sd, FileData *fd, gint *match,
+                                    gint *width, gint *height, gint *simval)
 {
-       gint new_data = FALSE;
-       gint tmatch = TRUE;
-       gint tested = FALSE;
+       gboolean new_data = FALSE;
+       gboolean tmatch = TRUE;
+       gboolean tested = FALSE;
 
        if (!sd->img_cd)
                {
@@ -1584,8 +1638,9 @@ static gint search_file_do_extra(SearchData *sd, FileData *fd, gint *match,
                    (sd->match_similarity_enable && !sd->img_cd->similarity))
                        {
                        sd->img_loader = image_loader_new(fd);
-                       image_loader_set_error_func(sd->img_loader, search_file_load_done_cb, sd);
-                       if (image_loader_start(sd->img_loader, search_file_load_done_cb, sd))
+                       g_signal_connect(G_OBJECT(sd->img_loader), "error", (GCallback)search_file_load_done_cb, sd);
+                       g_signal_connect(G_OBJECT(sd->img_loader), "done", (GCallback)search_file_load_done_cb, sd);
+                       if (image_loader_start(sd->img_loader))
                                {
                                return TRUE;
                                }
@@ -1662,15 +1717,16 @@ static gint search_file_do_extra(SearchData *sd, FileData *fd, gint *match,
        return FALSE;
 }
 
-static gint search_file_next(SearchData *sd)
+static gboolean search_file_next(SearchData *sd)
 {
        FileData *fd;
-       gint match = TRUE;
-       gint tested = FALSE;
-       gint extra_only = FALSE;
+       gboolean match = TRUE;
+       gboolean tested = FALSE;
+       gboolean extra_only = FALSE;
        gint width = 0;
        gint height = 0;
        gint sim = 0;
+       time_t file_date;
 
        if (!sd->search_file_list) return FALSE;
 
@@ -1700,7 +1756,7 @@ static gint search_file_next(SearchData *sd)
                                }
                        else
                                {
-                               match = (strcasecmp(fd->name, sd->search_name) == 0);
+                               match = (g_ascii_strcasecmp(fd->name, sd->search_name) == 0);
                                }
                        }
                else if (sd->match_name == SEARCH_MATCH_CONTAINS)
@@ -1747,11 +1803,21 @@ static gint search_file_next(SearchData *sd)
                tested = TRUE;
                match = FALSE;
 
+               if (sd->search_date_exif)
+                       {
+                       read_exif_time_data(fd);
+                       file_date = fd->exifdate;
+                       }
+               else
+                       {
+                       file_date = fd->date;
+                       }
+
                if (sd->match_date == SEARCH_MATCH_EQUAL)
                        {
                        struct tm *lt;
 
-                       lt = localtime(&fd->date);
+                       lt = localtime(&file_date);
                        match = (lt &&
                                 lt->tm_year == sd->search_date_y - 1900 &&
                                 lt->tm_mon == sd->search_date_m - 1 &&
@@ -1759,11 +1825,11 @@ static gint search_file_next(SearchData *sd)
                        }
                else if (sd->match_date == SEARCH_MATCH_UNDER)
                        {
-                       match = (fd->date < convert_dmy_to_time(sd->search_date_d, sd->search_date_m, sd->search_date_y));
+                       match = (file_date < convert_dmy_to_time(sd->search_date_d, sd->search_date_m, sd->search_date_y));
                        }
                else if (sd->match_date == SEARCH_MATCH_OVER)
                        {
-                       match = (fd->date > convert_dmy_to_time(sd->search_date_d, sd->search_date_m, sd->search_date_y) + 60 * 60 * 24 - 1);
+                       match = (file_date > convert_dmy_to_time(sd->search_date_d, sd->search_date_m, sd->search_date_y) + 60 * 60 * 24 - 1);
                        }
                else if (sd->match_date == SEARCH_MATCH_BETWEEN)
                        {
@@ -1778,7 +1844,7 @@ static gint search_file_next(SearchData *sd)
                                {
                                a += 60 * 60 * 24 - 1;
                                }
-                       match = MATCH_IS_BETWEEN(fd->date, a, b);
+                       match = MATCH_IS_BETWEEN(file_date, a, b);
                        }
                }
 
@@ -1789,14 +1855,16 @@ static gint search_file_next(SearchData *sd)
                tested = TRUE;
                match = FALSE;
 
-               if (comment_read(fd, &list, NULL))
+               list = metadata_read_list(fd, KEYWORD_KEY, METADATA_PLAIN);
+
+               if (list)
                        {
                        GList *needle;
                        GList *haystack;
 
                        if (sd->match_keywords == SEARCH_MATCH_ALL)
                                {
-                               gint found = TRUE;
+                               gboolean found = TRUE;
 
                                needle = sd->search_keyword_list;
                                while (needle && found)
@@ -1805,7 +1873,7 @@ static gint search_file_next(SearchData *sd)
                                        haystack = list;
                                        while (haystack && !found)
                                                {
-                                               found = (strcasecmp((gchar *)needle->data,
+                                               found = (g_ascii_strcasecmp((gchar *)needle->data,
                                                                    (gchar *)haystack->data) == 0);
                                                haystack = haystack->next;
                                                }
@@ -1816,7 +1884,7 @@ static gint search_file_next(SearchData *sd)
                                }
                        else if (sd->match_keywords == SEARCH_MATCH_ANY)
                                {
-                               gint found = FALSE;
+                               gboolean found = FALSE;
 
                                needle = sd->search_keyword_list;
                                while (needle && !found)
@@ -1824,7 +1892,7 @@ static gint search_file_next(SearchData *sd)
                                        haystack = list;
                                        while (haystack && !found)
                                                {
-                                               found = (strcasecmp((gchar *)needle->data,
+                                               found = (g_ascii_strcasecmp((gchar *)needle->data,
                                                                    (gchar *)haystack->data) == 0);
                                                haystack = haystack->next;
                                                }
@@ -1835,7 +1903,7 @@ static gint search_file_next(SearchData *sd)
                                }
                        else if (sd->match_keywords == SEARCH_MATCH_NONE)
                                {
-                               gint found = FALSE;
+                               gboolean found = FALSE;
 
                                needle = sd->search_keyword_list;
                                while (needle && !found)
@@ -1843,7 +1911,7 @@ static gint search_file_next(SearchData *sd)
                                        haystack = list;
                                        while (haystack && !found)
                                                {
-                                               found = (strcasecmp((gchar *)needle->data,
+                                               found = (g_ascii_strcasecmp((gchar *)needle->data,
                                                                    (gchar *)haystack->data) == 0);
                                                haystack = haystack->next;
                                                }
@@ -1860,8 +1928,97 @@ static gint search_file_next(SearchData *sd)
                        }
                }
 
-       if ((match || extra_only) &&
-           (sd->match_dimensions_enable || sd->match_similarity_enable))
+       if (match && sd->match_comment_enable && sd->search_comment && strlen(sd->search_comment))
+               {
+               gchar *comment;
+
+               tested = TRUE;
+               match = FALSE;
+
+               comment = metadata_read_string(fd, COMMENT_KEY, METADATA_PLAIN);
+
+               if (comment)
+                       {
+                       if (!sd->search_comment_match_case)
+                               {
+                               gchar *tmp = g_utf8_strdown(comment, -1);
+                               g_free(comment);
+                               comment = tmp;
+                               }
+
+                       if (sd->match_comment == SEARCH_MATCH_CONTAINS)
+                               {
+                               match = (strstr(comment, sd->search_comment) != NULL);
+                               }
+                       else if (sd->match_comment == SEARCH_MATCH_NONE)
+                               {
+                               match = (strstr(comment, sd->search_comment) == NULL);
+                               }
+                       g_free(comment);
+                       }
+               else
+                       {
+                       match = (sd->match_comment == SEARCH_MATCH_NONE);
+                       }
+               }
+
+       if (match && sd->match_gps_enable)
+               {
+               /* Calculate the distance the image is from the specified origin.
+               * This is a standard algorithm. A simplified one may be faster.
+               */
+               #define RADIANS  0.0174532925
+               #define KM_EARTH_RADIUS 6371
+               #define MILES_EARTH_RADIUS 3959
+               #define NAUTICAL_MILES_EARTH_RADIUS 3440
+
+               gdouble latitude, longitude, range, conversion;
+
+               if (g_strcmp0(gtk_combo_box_text_get_active_text(
+                                               GTK_COMBO_BOX_TEXT(sd->units_gps)), _("km")) == 0)
+                       {
+                       conversion = KM_EARTH_RADIUS;
+                       }
+               else if (g_strcmp0(gtk_combo_box_text_get_active_text(
+                                               GTK_COMBO_BOX_TEXT(sd->units_gps)), _("miles")) == 0)
+                       {
+                       conversion = MILES_EARTH_RADIUS;
+                       }
+               else
+                       {
+                       conversion = NAUTICAL_MILES_EARTH_RADIUS;
+                       }
+
+               tested = TRUE;
+               match = FALSE;
+
+               latitude = metadata_read_GPS_coord(fd, "Xmp.exif.GPSLatitude", 1000);
+               longitude = metadata_read_GPS_coord(fd, "Xmp.exif.GPSLongitude", 1000);
+               if (latitude != 1000 && longitude != 1000)
+                       {
+                       range = conversion * acos(sin(latitude * RADIANS) *
+                                               sin(sd->search_lat * RADIANS) + cos(latitude * RADIANS) *
+                                               cos(sd->search_lat * RADIANS) * cos((sd->search_lon -
+                                               longitude) * RADIANS));
+                       if (sd->match_gps == SEARCH_MATCH_UNDER)
+                               {
+                               if (sd->search_gps >= range)
+                                       match = TRUE;
+                               }
+                       else if (sd->match_gps == SEARCH_MATCH_OVER)
+                               {
+                               if (sd->search_gps < range)
+                                       match = TRUE;
+                               }
+                       }
+               else if (sd->match_gps == SEARCH_MATCH_NONE)
+                       {
+                       match = TRUE;
+                       }
+               }
+
+       if ((match || extra_only) && (sd->match_dimensions_enable ||
+                                                               sd ->match_similarity_enable))
                {
                tested = TRUE;
 
@@ -1899,7 +2056,7 @@ static gint search_file_next(SearchData *sd)
        return FALSE;
 }
 
-static gint search_step_cb(gpointer data)
+static gboolean search_step_cb(gpointer data)
 {
        SearchData *sd = data;
        FileData *fd;
@@ -1914,7 +2071,7 @@ static gint search_step_cb(gpointer data)
                {
                if (search_file_next(sd))
                        {
-                       sd->search_idle_id = -1;
+                       sd->search_idle_id = 0;
                        return FALSE;
                        }
                return TRUE;
@@ -1922,7 +2079,7 @@ static gint search_step_cb(gpointer data)
 
        if (!sd->search_file_list && !sd->search_folder_list)
                {
-               sd->search_idle_id = -1;
+               sd->search_idle_id = 0;
 
                search_stop(sd);
                search_result_thumb_step(sd);
@@ -1936,23 +2093,28 @@ static gint search_step_cb(gpointer data)
                {
                GList *list = NULL;
                GList *dlist = NULL;
-               gint success = FALSE;
+               gboolean success = FALSE;
 
                sd->search_done_list = g_list_prepend(sd->search_done_list, fd);
 
                if (sd->search_type == SEARCH_MATCH_NONE)
                        {
-                       success = filelist_read(fd->path, &list, &dlist);
+                       success = filelist_read(fd, &list, &dlist);
                        }
                else if (sd->search_type == SEARCH_MATCH_ALL &&
-                        sd->search_path &&
-                        strlen(fd->path) >= strlen(sd->search_path))
+                        sd->search_dir_fd &&
+                        strlen(fd->path) >= strlen(sd->search_dir_fd->path))
                        {
                        const gchar *path;
 
-                       path = fd->path + strlen(sd->search_path);
-                       if (path != fd->path) success = filelist_read(path, &list, NULL);
-                       success |= filelist_read(fd->path, NULL, &dlist);
+                       path = fd->path + strlen(sd->search_dir_fd->path);
+                       if (path != fd->path)
+                               {
+                               FileData *dir_fd = file_data_new_dir(path);
+                               success = filelist_read(dir_fd, &list, NULL);
+                               file_data_unref(dir_fd);
+                               }
+                       success |= filelist_read(fd, NULL, &dlist);
                        if (success)
                                {
                                GList *work;
@@ -2016,10 +2178,9 @@ static void search_start(SearchData *sd)
        search_stop(sd);
        search_result_clear(sd);
 
-       if (sd->search_path)
+       if (sd->search_dir_fd)
                {
-               sd->search_folder_list = g_list_prepend(sd->search_folder_list,
-                                                       file_data_new_simple(sd->search_path));
+               sd->search_folder_list = g_list_prepend(sd->search_folder_list, file_data_ref(sd->search_dir_fd));
                }
 
        if (!sd->search_name_match_case)
@@ -2030,6 +2191,14 @@ static void search_start(SearchData *sd)
                sd->search_name = tmp;
                }
 
+       if (!sd->search_comment_match_case)
+               {
+               /* convert to lowercase here, so that this is only done once per search */
+               gchar *tmp = g_utf8_strdown(sd->search_comment, -1);
+               g_free(sd->search_comment);
+               sd->search_comment = tmp;
+               }
+
        sd->search_count = 0;
        sd->search_total = 0;
 
@@ -2059,9 +2228,10 @@ static void search_start(SearchData *sd)
                                sd->search_similarity_cd = cache_sim_data_new();
                                }
 
-                       sd->img_loader = image_loader_new(file_data_new_simple(sd->search_similarity_path));
-                       image_loader_set_error_func(sd->img_loader, search_similarity_load_done_cb, sd);
-                       if (image_loader_start(sd->img_loader, search_similarity_load_done_cb, sd))
+                       sd->img_loader = image_loader_new(file_data_new_group(sd->search_similarity_path));
+                       g_signal_connect(G_OBJECT(sd->img_loader), "error", (GCallback)search_similarity_load_done_cb, sd);
+                       g_signal_connect(G_OBJECT(sd->img_loader), "done", (GCallback)search_similarity_load_done_cb, sd);
+                       if (image_loader_start(sd->img_loader))
                                {
                                return;
                                }
@@ -2079,6 +2249,7 @@ static void search_start_cb(GtkWidget *widget, gpointer data)
        SearchData *sd = data;
        GtkTreeViewColumn *column;
        gchar *path;
+       gchar *entry_text;
 
        if (sd->search_folder_list)
                {
@@ -2091,6 +2262,10 @@ static void search_start_cb(GtkWidget *widget, gpointer data)
        g_free(sd->search_name);
        sd->search_name = g_strdup(gtk_entry_get_text(GTK_ENTRY(sd->entry_name)));
 
+       /* XXX */
+       g_free(sd->search_comment);
+       sd->search_comment = g_strdup(gtk_entry_get_text(GTK_ENTRY(sd->entry_comment)));
+
        g_free(sd->search_similarity_path);
        sd->search_similarity_path = g_strdup(gtk_entry_get_text(GTK_ENTRY(sd->entry_similarity)));
        if (sd->match_similarity_enable)
@@ -2105,6 +2280,32 @@ static void search_start_cb(GtkWidget *widget, gpointer data)
                tab_completion_append_to_history(sd->entry_similarity, sd->search_similarity_path);
                }
 
+       /* Check the coordinate entry.
+       * If the result is not sensible, it should get blocked.
+       */
+       if (sd->match_gps_enable)
+               {
+               if (sd->match_gps != SEARCH_MATCH_NONE)
+                       {
+                       entry_text = decode_geo_parameters(gtk_entry_get_text(
+                                                                               GTK_ENTRY(sd->entry_gps_coord)));
+
+                       sd->search_lat = 1000;
+                       sd->search_lon = 1000;
+                       sscanf(entry_text," %lf  %lf ", &sd->search_lat, &sd->search_lon );
+                       if (!(entry_text != NULL && !g_strstr_len(entry_text, -1, "Error") &&
+                                               sd->search_lat >= -90 && sd->search_lat <= 90 &&
+                                               sd->search_lon >= -180 && sd->search_lon <= 180))
+                               {
+                               file_util_warning_dialog(_(
+                                               "Entry does not contain a valid lat/long value"),
+                                                       entry_text, GTK_STOCK_DIALOG_WARNING, sd->window);
+                               return;
+                               }
+                       g_free(entry_text);
+                       }
+               }
+
        string_list_free(sd->search_keyword_list);
        sd->search_keyword_list = keyword_list_pull(sd->entry_keywords);
 
@@ -2137,11 +2338,10 @@ static void search_start_cb(GtkWidget *widget, gpointer data)
                path = remove_trailing_slash(gtk_entry_get_text(GTK_ENTRY(sd->path_entry)));
                if (isdir(path))
                        {
-                       g_free(sd->search_path);
-                       sd->search_path = path;
-                       path = NULL;
+                       file_data_unref(sd->search_dir_fd);
+                       sd->search_dir_fd = file_data_new_dir(path);
 
-                       tab_completion_append_to_history(sd->path_entry, sd->search_path);
+                       tab_completion_append_to_history(sd->path_entry, sd->search_dir_fd->path);
 
                        search_start(sd);
                        }
@@ -2157,10 +2357,8 @@ static void search_start_cb(GtkWidget *widget, gpointer data)
        else if (sd->search_type == SEARCH_MATCH_ALL)
                {
                /* search metadata */
-
-               g_free(sd->search_path);
-               sd->search_path = g_strconcat(homedir(), "/", GQ_CACHE_RC_METADATA, NULL);
-
+               file_data_unref(sd->search_dir_fd);
+               sd->search_dir_fd = file_data_new_dir(get_metadata_cache_dir());
                search_start(sd);
                }
        else if (sd->search_type == SEARCH_MATCH_CONTAINS)
@@ -2170,8 +2368,8 @@ static void search_start_cb(GtkWidget *widget, gpointer data)
 
                list = search_result_refine_list(sd);
 
-               g_free(sd->search_path);
-               sd->search_path = NULL;
+               file_data_unref(sd->search_dir_fd);
+               sd->search_dir_fd = NULL;
 
                search_start(sd);
 
@@ -2229,7 +2427,10 @@ static gint search_result_sort_cb(GtkTreeModel *model, GtkTreeIter *a, GtkTreeIt
                        return 0;
                        break;
                case SEARCH_COLUMN_NAME:
-                       return CASE_SORT(fda->fd->name, fdb->fd->name);
+                       if (options->file_sort.case_sensitive)
+                               return strcmp(fda->fd->collate_key_name, fdb->fd->collate_key_name);
+                       else
+                               return strcmp(fda->fd->collate_key_name_nocase, fdb->fd->collate_key_name_nocase);
                        break;
                case SEARCH_COLUMN_SIZE:
                        if (fda->fd->size > fdb->fd->size) return 1;
@@ -2245,7 +2446,7 @@ static gint search_result_sort_cb(GtkTreeModel *model, GtkTreeIter *a, GtkTreeIt
                        return sort_matchdata_dimensions(fda, fdb);
                        break;
                case SEARCH_COLUMN_PATH:
-                       return CASE_SORT(fda->fd->path, fdb->fd->path);
+                       return utf8_compare(fda->fd->path, fdb->fd->path, options->file_sort.case_sensitive);
                        break;
                default:
                        break;
@@ -2254,7 +2455,7 @@ static gint search_result_sort_cb(GtkTreeModel *model, GtkTreeIter *a, GtkTreeIt
        return 0;
 }
 
-static void search_result_add_column(SearchData * sd, gint n, const gchar *title, gint image, gint right_justify)
+static void search_result_add_column(SearchData * sd, gint n, const gchar *title, gboolean image, gboolean right_justify)
 {
        GtkTreeViewColumn *column;
        GtkCellRenderer *renderer;
@@ -2287,27 +2488,34 @@ static void search_result_add_column(SearchData * sd, gint n, const gchar *title
        gtk_tree_view_append_column(GTK_TREE_VIEW(sd->result_view), column);
 }
 
-static void menu_choice_set_visible(GtkWidget *widget, gint visible)
+static void menu_choice_set_visible(GtkWidget *widget, gboolean visible)
 {
        if (visible)
                {
-               if (!GTK_WIDGET_VISIBLE(widget)) gtk_widget_show(widget);
+               if (!gtk_widget_get_visible(widget)) gtk_widget_show(widget);
                }
        else
                {
-               if (GTK_WIDGET_VISIBLE(widget)) gtk_widget_hide(widget);
+               if (gtk_widget_get_visible(widget)) gtk_widget_hide(widget);
                }
 }
 
-static void menu_choice_path_cb(GtkWidget *combo, gpointer data)
+static gboolean menu_choice_get_match_type(GtkWidget *combo, MatchType *type)
 {
-       SearchData *sd = data;
        GtkTreeModel *store;
        GtkTreeIter iter;
 
        store = gtk_combo_box_get_model(GTK_COMBO_BOX(combo));
-       if (!gtk_combo_box_get_active_iter(GTK_COMBO_BOX(combo), &iter)) return;
-       gtk_tree_model_get(store, &iter, MENU_CHOICE_COLUMN_VALUE, &sd->search_type, -1);
+       if (!gtk_combo_box_get_active_iter(GTK_COMBO_BOX(combo), &iter)) return FALSE;
+       gtk_tree_model_get(store, &iter, MENU_CHOICE_COLUMN_VALUE, type, -1);
+       return TRUE;
+}
+
+static void menu_choice_path_cb(GtkWidget *combo, gpointer data)
+{
+       SearchData *sd = data;
+
+       if (!menu_choice_get_match_type(combo, &sd->search_type)) return;
 
        menu_choice_set_visible(gtk_widget_get_parent(sd->check_recurse),
                                (sd->search_type == SEARCH_MATCH_NONE));
@@ -2316,23 +2524,15 @@ static void menu_choice_path_cb(GtkWidget *combo, gpointer data)
 static void menu_choice_name_cb(GtkWidget *combo, gpointer data)
 {
        SearchData *sd = data;
-       GtkTreeModel *store;
-       GtkTreeIter iter;
 
-       store = gtk_combo_box_get_model(GTK_COMBO_BOX(combo));
-       if (!gtk_combo_box_get_active_iter(GTK_COMBO_BOX(combo), &iter)) return;
-       gtk_tree_model_get(store, &iter, MENU_CHOICE_COLUMN_VALUE, &sd->match_name, -1);
+       if (!menu_choice_get_match_type(combo, &sd->match_name)) return;
 }
 
 static void menu_choice_size_cb(GtkWidget *combo, gpointer data)
 {
        SearchData *sd = data;
-       GtkTreeModel *store;
-       GtkTreeIter iter;
 
-       store = gtk_combo_box_get_model(GTK_COMBO_BOX(combo));
-       if (!gtk_combo_box_get_active_iter(GTK_COMBO_BOX(combo), &iter)) return;
-       gtk_tree_model_get(store, &iter, MENU_CHOICE_COLUMN_VALUE, &sd->match_size, -1);
+       if (!menu_choice_get_match_type(combo, &sd->match_size)) return;
 
        menu_choice_set_visible(gtk_widget_get_parent(sd->spin_size_end),
                                (sd->match_size == SEARCH_MATCH_BETWEEN));
@@ -2341,12 +2541,8 @@ static void menu_choice_size_cb(GtkWidget *combo, gpointer data)
 static void menu_choice_date_cb(GtkWidget *combo, gpointer data)
 {
        SearchData *sd = data;
-       GtkTreeModel *store;
-       GtkTreeIter iter;
 
-       store = gtk_combo_box_get_model(GTK_COMBO_BOX(combo));
-       if (!gtk_combo_box_get_active_iter(GTK_COMBO_BOX(combo), &iter)) return;
-       gtk_tree_model_get(store, &iter, MENU_CHOICE_COLUMN_VALUE, &sd->match_date, -1);
+       if (!menu_choice_get_match_type(combo, &sd->match_date)) return;
 
        menu_choice_set_visible(gtk_widget_get_parent(sd->date_sel_end),
                                (sd->match_date == SEARCH_MATCH_BETWEEN));
@@ -2355,12 +2551,8 @@ static void menu_choice_date_cb(GtkWidget *combo, gpointer data)
 static void menu_choice_dimensions_cb(GtkWidget *combo, gpointer data)
 {
        SearchData *sd = data;
-       GtkTreeModel *store;
-       GtkTreeIter iter;
 
-       store = gtk_combo_box_get_model(GTK_COMBO_BOX(combo));
-       if (!gtk_combo_box_get_active_iter(GTK_COMBO_BOX(combo), &iter)) return;
-       gtk_tree_model_get(store, &iter, MENU_CHOICE_COLUMN_VALUE, &sd->match_dimensions, -1);
+       if (!menu_choice_get_match_type(combo, &sd->match_dimensions)) return;
 
        menu_choice_set_visible(gtk_widget_get_parent(sd->spin_width_end),
                                (sd->match_dimensions == SEARCH_MATCH_BETWEEN));
@@ -2369,12 +2561,15 @@ static void menu_choice_dimensions_cb(GtkWidget *combo, gpointer data)
 static void menu_choice_keyword_cb(GtkWidget *combo, gpointer data)
 {
        SearchData *sd = data;
-       GtkTreeModel *store;
-       GtkTreeIter iter;
 
-       store = gtk_combo_box_get_model(GTK_COMBO_BOX(combo));
-       if (!gtk_combo_box_get_active_iter(GTK_COMBO_BOX(combo), &iter)) return;
-       gtk_tree_model_get(store, &iter, MENU_CHOICE_COLUMN_VALUE, &sd->match_keywords, -1);
+       if (!menu_choice_get_match_type(combo, &sd->match_keywords)) return;
+}
+
+static void menu_choice_comment_cb(GtkWidget *combo, gpointer data)
+{
+       SearchData *sd = data;
+
+       if (!menu_choice_get_match_type(combo, &sd->match_comment)) return;
 }
 
 static void menu_choice_spin_cb(GtkAdjustment *adjustment, gpointer data)
@@ -2384,6 +2579,16 @@ static void menu_choice_spin_cb(GtkAdjustment *adjustment, gpointer data)
        *value = (gint)gtk_adjustment_get_value(adjustment);
 }
 
+static void menu_choice_gps_cb(GtkWidget *combo, gpointer data)
+{
+       SearchData *sd = data;
+
+       if (!menu_choice_get_match_type(combo, &sd->match_gps)) return;
+
+       menu_choice_set_visible(gtk_widget_get_parent(sd->spin_gps),
+                                       (sd->match_gps != SEARCH_MATCH_NONE));
+}
+
 static GtkWidget *menu_spin(GtkWidget *box, gdouble min, gdouble max, gint value,
                            GCallback func, gpointer data)
 {
@@ -2494,7 +2699,7 @@ static void search_window_close(SearchData *sd)
        gtk_widget_destroy(sd->window);
 }
 
-static gint search_window_delete_cb(GtkWidget *widget, GdkEventAny *event, gpointer data)
+static gboolean search_window_delete_cb(GtkWidget *widget, GdkEventAny *event, gpointer data)
 {
        SearchData *sd = data;
 
@@ -2516,15 +2721,19 @@ static void search_window_destroy_cb(GtkWidget *widget, gpointer data)
        search_stop(sd);
        search_result_clear(sd);
 
-       g_free(sd->search_path);
+       file_data_unref(sd->search_dir_fd);
+
        g_free(sd->search_name);
+       g_free(sd->search_comment);
        g_free(sd->search_similarity_path);
        string_list_free(sd->search_keyword_list);
 
+       file_data_unregister_notify_func(search_notify_cb, sd);
+
        g_free(sd);
 }
 
-void search_new(const gchar *path, const gchar *example_file)
+void search_new(FileData *dir_fd, FileData *example_file)
 {
        SearchData *sd;
        GtkWidget *vbox;
@@ -2541,15 +2750,13 @@ void search_new(const gchar *path, const gchar *example_file)
 
        sd = g_new0(SearchData, 1);
 
-       sd->search_path = g_strdup(path);
+       sd->search_dir_fd = file_data_ref(dir_fd);
        sd->search_path_recurse = TRUE;
        sd->search_size = 0;
        sd->search_width = 640;
        sd->search_height = 480;
        sd->search_width_end = 1024;
        sd->search_height_end = 768;
-       sd->search_name = NULL;
-       sd->search_name_match_case = FALSE;
 
        sd->search_type = SEARCH_MATCH_NONE;
 
@@ -2558,27 +2765,26 @@ void search_new(const gchar *path, const gchar *example_file)
        sd->match_date = SEARCH_MATCH_EQUAL;
        sd->match_dimensions = SEARCH_MATCH_EQUAL;
        sd->match_keywords = SEARCH_MATCH_ALL;
+       sd->match_comment = SEARCH_MATCH_CONTAINS;
 
        sd->match_name_enable = TRUE;
-       sd->match_size_enable = FALSE;
-       sd->match_date_enable = FALSE;
-       sd->match_dimensions_enable = FALSE;
-       sd->match_similarity_enable = FALSE;
-       sd->match_keywords_enable = FALSE;
 
        sd->search_similarity = 95;
-       sd->search_similarity_path = g_strdup(example_file);
-       sd->search_similarity_cd = NULL;
 
-       sd->search_idle_id = -1;
-       sd->update_idle_id = -1;
+       sd->search_gps = 1;
+       sd->match_gps = SEARCH_MATCH_NONE;
+
+       if (example_file)
+               {
+               sd->search_similarity_path = g_strdup(example_file->path);
+               }
 
        sd->window = window_new(GTK_WINDOW_TOPLEVEL, "search", NULL, NULL, _("Image search"));
 
        gtk_window_set_resizable(GTK_WINDOW(sd->window), TRUE);
 
-       geometry.min_width = 32;
-       geometry.min_height = 32;
+       geometry.min_width = DEFAULT_MINIMAL_WINDOW_SIZE;
+       geometry.min_height = DEFAULT_MINIMAL_WINDOW_SIZE;
        geometry.base_width = DEF_SEARCH_WIDTH;
        geometry.base_height = DEF_SEARCH_HEIGHT;
        gtk_window_set_geometry_hints(GTK_WINDOW(sd->window), NULL, &geometry,
@@ -2611,7 +2817,7 @@ void search_new(const gchar *path, const gchar *example_file)
        gtk_widget_show(sd->menu_path);
 
        hbox2 = pref_box_new(hbox, TRUE, GTK_ORIENTATION_HORIZONTAL, PREF_PAD_SPACE);
-       combo = tab_completion_new_with_history(&sd->path_entry, sd->search_path,
+       combo = tab_completion_new_with_history(&sd->path_entry, sd->search_dir_fd->path,
                                                "search_path", -1,
                                                NULL, NULL);
        tab_completion_add_select_button(sd->path_entry, NULL, TRUE);
@@ -2620,6 +2826,7 @@ void search_new(const gchar *path, const gchar *example_file)
        sd->check_recurse = pref_checkbox_new_int(hbox2, _("Recurse"),
                                                  sd->search_path_recurse, &sd->search_path_recurse);
 
+       /* Search for file name */
        hbox = menu_choice(sd->box_search, &sd->check_name, &sd->menu_name,
                           _("File name"), &sd->match_name_enable,
                           text_search_menu_name, sizeof(text_search_menu_name) / sizeof(MatchList),
@@ -2630,6 +2837,7 @@ void search_new(const gchar *path, const gchar *example_file)
        pref_checkbox_new_int(hbox, _("Match case"),
                              sd->search_name_match_case, &sd->search_name_match_case);
 
+       /* Search for file size */
        hbox = menu_choice(sd->box_search, &sd->check_size, &sd->menu_size,
                           _("File size is"), &sd->match_size_enable,
                           text_search_menu_size, sizeof(text_search_menu_size) / sizeof(MatchList),
@@ -2642,6 +2850,7 @@ void search_new(const gchar *path, const gchar *example_file)
        sd->spin_size_end = menu_spin(hbox2, 0, 1024*1024*1024, sd->search_size_end,
                                      G_CALLBACK(menu_choice_spin_cb), &sd->search_size_end);
 
+       /* Search for file date */
        hbox = menu_choice(sd->box_search, &sd->check_date, &sd->menu_date,
                           _("File date is"), &sd->match_date_enable,
                           text_search_menu_date, sizeof(text_search_menu_date) / sizeof(MatchList),
@@ -2658,7 +2867,10 @@ void search_new(const gchar *path, const gchar *example_file)
        date_selection_time_set(sd->date_sel_end, time(NULL));
        gtk_box_pack_start(GTK_BOX(hbox2), sd->date_sel_end, FALSE, FALSE, 0);
        gtk_widget_show(sd->date_sel_end);
+       pref_checkbox_new_int(hbox, _("Exif date"),
+                               sd->search_date_exif, &sd->search_date_exif);
 
+       /* Search for image dimensions */
        hbox = menu_choice(sd->box_search, &sd->check_dimensions, &sd->menu_dimensions,
                           _("Image dimensions are"), &sd->match_dimensions_enable,
                           text_search_menu_size, sizeof(text_search_menu_size) / sizeof(MatchList),
@@ -2679,6 +2891,7 @@ void search_new(const gchar *path, const gchar *example_file)
        sd->spin_height_end = menu_spin(hbox2, 0, 1000000, sd->search_height_end,
                                        G_CALLBACK(menu_choice_spin_cb), &sd->search_height_end);
 
+       /* Search for image similarity */
        hbox = menu_choice(sd->box_search, &sd->check_similarity, NULL,
                           _("Image content is"), &sd->match_similarity_enable,
                           NULL, 0, NULL, sd);
@@ -2695,6 +2908,7 @@ void search_new(const gchar *path, const gchar *example_file)
        gtk_box_pack_start(GTK_BOX(hbox), combo, TRUE, TRUE, 0);
        gtk_widget_show(combo);
 
+       /* Search for image keywords */
        hbox = menu_choice(sd->box_search, &sd->check_keywords, &sd->menu_keywords,
                           _("Keywords"), &sd->match_keywords_enable,
                           text_search_menu_keyword, sizeof(text_search_menu_keyword) / sizeof(MatchList),
@@ -2706,6 +2920,54 @@ void search_new(const gchar *path, const gchar *example_file)
                         G_CALLBACK(menu_choice_check_cb), sd->entry_keywords);
        gtk_widget_show(sd->entry_keywords);
 
+       /* Search for image comment */
+       hbox = menu_choice(sd->box_search, &sd->check_comment, &sd->menu_comment,
+                       _("Comment"), &sd->match_comment_enable,
+                       text_search_menu_comment, sizeof(text_search_menu_comment) / sizeof(MatchList),
+                       G_CALLBACK(menu_choice_comment_cb), sd);
+       sd->entry_comment = gtk_entry_new();
+       gtk_box_pack_start(GTK_BOX(hbox), sd->entry_comment, TRUE, TRUE, 0);
+       gtk_widget_set_sensitive(sd->entry_comment, sd->match_comment_enable);
+       g_signal_connect(G_OBJECT(sd->check_comment), "toggled",
+                       G_CALLBACK(menu_choice_check_cb), sd->entry_comment);
+       gtk_widget_show(sd->entry_comment);
+       pref_checkbox_new_int(hbox, _("Match case"),
+                             sd->search_comment_match_case, &sd->search_comment_match_case);
+
+       /* Search for images within a specified range of a lat/long coordinate
+       */
+       hbox = menu_choice(sd->box_search, &sd->check_gps, &sd->menu_gps,
+                          _("Image is"), &sd->match_gps_enable,
+                          text_search_menu_gps, sizeof(text_search_menu_gps) / sizeof(MatchList),
+                          G_CALLBACK(menu_choice_gps_cb), sd);
+
+       hbox2 = gtk_hbox_new(FALSE, PREF_PAD_SPACE);
+       gtk_box_pack_start(GTK_BOX(hbox), hbox2, FALSE, FALSE, 0);
+       sd->spin_gps = menu_spin(hbox2, 1, 9999, sd->search_gps,
+                                                                  G_CALLBACK(menu_choice_spin_cb), &sd->search_gps);
+
+       sd->units_gps = gtk_combo_box_text_new();
+       gtk_combo_box_text_append_text(GTK_COMBO_BOX_TEXT(sd->units_gps), _("km"));
+       gtk_combo_box_text_append_text(GTK_COMBO_BOX_TEXT(sd->units_gps), _("miles"));
+       gtk_combo_box_text_append_text(GTK_COMBO_BOX_TEXT(sd->units_gps), _("n.m."));
+       gtk_box_pack_start(GTK_BOX(hbox2), sd->units_gps, FALSE, FALSE, 0);
+       gtk_combo_box_set_active(GTK_COMBO_BOX(sd->units_gps), 0);
+       gtk_widget_set_tooltip_text(sd->units_gps, "kilometres, miles or nautical miles");
+       gtk_widget_show(sd->units_gps);
+
+       pref_label_new(hbox2, _("from"));
+
+       sd->entry_gps_coord = gtk_entry_new();
+       gtk_editable_set_editable(GTK_EDITABLE(sd->entry_gps_coord), TRUE);
+       gtk_widget_set_has_tooltip(sd->entry_gps_coord, TRUE);
+       gtk_widget_set_tooltip_text(sd->entry_gps_coord, _("Enter a coordinate in the form:\n89.123 179.456\nor drag-and-drop a geo-coded image\nor left-click on the map and paste\nor cut-and-paste or drag-and-drop\nan internet search URL\nSee the Help file"));
+       gtk_box_pack_start(GTK_BOX(hbox2), sd->entry_gps_coord, TRUE, TRUE, 0);
+       gtk_widget_set_sensitive(sd->entry_gps_coord, TRUE);
+
+       gtk_widget_show(sd->entry_gps_coord);
+
+       /* Done the types of searches */
+
        scrolled = gtk_scrolled_window_new(NULL, NULL);
        gtk_scrolled_window_set_shadow_type(GTK_SCROLLED_WINDOW(scrolled), GTK_SHADOW_IN);
        gtk_scrolled_window_set_policy(GTK_SCROLLED_WINDOW(scrolled),
@@ -2751,10 +3013,6 @@ void search_new(const gchar *path, const gchar *example_file)
        gtk_tree_view_set_headers_visible(GTK_TREE_VIEW(sd->result_view), TRUE);
        gtk_tree_view_set_enable_search(GTK_TREE_VIEW(sd->result_view), FALSE);
 
-#if 0
-       gtk_tree_view_set_search_column(GTK_TREE_VIEW(sd->result_view), SEARCH_COLUMN_NAME);
-#endif
-
        search_result_add_column(sd, SEARCH_COLUMN_RANK, _("Rank"), FALSE, FALSE);
        search_result_add_column(sd, SEARCH_COLUMN_THUMB, "", TRUE, FALSE);
        search_result_add_column(sd, SEARCH_COLUMN_NAME, _("Name"), FALSE, FALSE);
@@ -2789,6 +3047,10 @@ void search_new(const gchar *path, const gchar *example_file)
 
        sd->label_progress = gtk_progress_bar_new();
        gtk_widget_set_size_request(sd->label_progress, 50, -1);
+#if GTK_CHECK_VERSION(3,0,0)
+       gtk_progress_bar_set_text(GTK_PROGRESS_BAR(sd->label_progress), "");
+       gtk_progress_bar_set_show_text(GTK_PROGRESS_BAR(sd->label_progress), TRUE);
+#endif
        gtk_box_pack_start(GTK_BOX(hbox), sd->label_progress, TRUE, TRUE, 0);
        gtk_widget_show(sd->label_progress);
 
@@ -2808,6 +3070,8 @@ void search_new(const gchar *path, const gchar *example_file)
 
        search_window_list = g_list_append(search_window_list, sd);
 
+       file_data_register_notify_func(search_notify_cb, sd, NOTIFY_PRIORITY_MEDIUM);
+
        gtk_widget_show(sd->window);
 }
 
@@ -2821,7 +3085,7 @@ static void search_result_change_path(SearchData *sd, FileData *fd)
 {
        GtkTreeModel *store;
        GtkTreeIter iter;
-       gint valid;
+       gboolean valid;
 
        store = gtk_tree_view_get_model(GTK_TREE_VIEW(sd->result_view));
        valid = gtk_tree_model_get_iter_first(store, &iter);
@@ -2850,21 +3114,25 @@ static void search_result_change_path(SearchData *sd, FileData *fd)
                }
 }
 
-void search_maint_renamed(FileData *fd)
+static void search_notify_cb(FileData *fd, NotifyType type, gpointer data)
 {
-       GList *work;
+       SearchData *sd = data;
 
-       work = search_window_list;
-       while (work)
-               {
-               SearchData *sd = work->data;
-               work = work->next;
+       if (!(type & NOTIFY_CHANGE) || !fd->change) return;
 
-               search_result_change_path(sd, fd);
-               }
-}
+       DEBUG_1("Notify search: %s %04x", fd->path, type);
 
-void search_maint_removed(FileData *fd)
-{
-       search_maint_renamed(fd);
+       switch (fd->change->type)
+               {
+               case FILEDATA_CHANGE_MOVE:
+               case FILEDATA_CHANGE_RENAME:
+               case FILEDATA_CHANGE_DELETE:
+                       search_result_change_path(sd, fd);
+                       break;
+               case FILEDATA_CHANGE_COPY:
+               case FILEDATA_CHANGE_UNSPECIFIED:
+               case FILEDATA_CHANGE_WRITE_METADATA:
+                       break;
+               }
 }
+/* vim: set shiftwidth=8 softtabstop=0 cindent cinoptions={1s: */