Remove commented out code.
[geeqie.git] / src / thumb.c
index 8993dd7..f80eaff 100644 (file)
@@ -1,7 +1,7 @@
 /*
  * Geeqie
  * (C) 2004 John Ellis
- * Copyright (C) 2008 The Geeqie Team
+ * Copyright (C) 2008 - 2012 The Geeqie Team
  *
  * Author: John Ellis
  *
 #include "thumb_standard.h"
 #include "ui_fileops.h"
 #include "exif.h"
+#include "metadata.h"
 
 #include <utime.h>
 
 
 static void thumb_loader_error_cb(ImageLoader *il, gpointer data);
-static void thumb_loader_setup(ThumbLoader *tl, const gchar *path);
+static void thumb_loader_setup(ThumbLoader *tl, FileData *fd);
 
 static GdkPixbuf *get_xv_thumbnail(gchar *thumb_filename, gint max_w, gint max_h);
 
@@ -37,17 +38,20 @@ static GdkPixbuf *get_xv_thumbnail(gchar *thumb_filename, gint max_w, gint max_h
  *-----------------------------------------------------------------------------
  */
 
-static gint thumb_loader_save_to_cache(ThumbLoader *tl)
+/* Save thumbnail to disk
+ * or just mark failed thumbnail with 0 byte file (mark_failure = TRUE) */
+static gboolean thumb_loader_save_thumbnail(ThumbLoader *tl, gboolean mark_failure)
 {
        gchar *cache_dir;
-       gint success = FALSE;
+       gboolean success = FALSE;
        mode_t mode = 0755;
 
-       if (!tl || !tl->fd || !tl->fd->thumb_pixbuf) return FALSE;
+       if (!tl || !tl->fd) return FALSE;
+       if (!mark_failure && !tl->fd->thumb_pixbuf) return FALSE;
 
        cache_dir = cache_get_location(CACHE_TYPE_THUMB, tl->fd->path, FALSE, &mode);
 
-       if (cache_ensure_dir_exists(cache_dir, mode))
+       if (recursive_mkdir_if_not_exists(cache_dir, mode))
                {
                gchar *cache_path;
                gchar *pathl;
@@ -56,72 +60,39 @@ static gint thumb_loader_save_to_cache(ThumbLoader *tl)
                cache_path = g_build_filename(cache_dir, name, NULL);
                g_free(name);
 
-               DEBUG_1("Saving thumb: %s", cache_path);
-
                pathl = path_from_utf8(cache_path);
-               success = pixbuf_to_file_as_png(tl->fd->thumb_pixbuf, pathl);
-               if (success)
+
+               if (mark_failure)
                        {
-                       struct utimbuf ut;
-                       /* set thumb time to that of source file */
+                       FILE *f = fopen(pathl, "w"); ;
 
-                       ut.actime = ut.modtime = filetime(tl->fd->path);
-                       if (ut.modtime > 0)
+                       DEBUG_1("Marking thumb failure: %s", cache_path);
+                       if (f)
                                {
-                               utime(pathl, &ut);
+                               fclose(f);
+                               success = TRUE;
                                }
                        }
                else
                        {
-                       DEBUG_1("Saving failed: %s", pathl);
+                       DEBUG_1("Saving thumb: %s", cache_path);
+                       success = pixbuf_to_file_as_png(tl->fd->thumb_pixbuf, pathl);
                        }
 
-               g_free(pathl);
-               g_free(cache_path);
-               }
-
-       g_free(cache_dir);
-
-       return success;
-}
-
-static gint thumb_loader_mark_failure(ThumbLoader *tl)
-{
-       gchar *cache_dir;
-       gint success = FALSE;
-       mode_t mode = 0755;
-
-       if (!tl) return FALSE;
-
-       cache_dir = cache_get_location(CACHE_TYPE_THUMB, tl->fd->path, FALSE, &mode);
-
-       if (cache_ensure_dir_exists(cache_dir, mode))
-               {
-               gchar *cache_path;
-               gchar *pathl;
-               FILE *f;
-               gchar *name = g_strconcat(filename_from_path(tl->fd->path), GQ_CACHE_EXT_THUMB, NULL);
-
-               cache_path = g_build_filename(cache_dir, name, NULL);
-               g_free(name);
-
-               DEBUG_1("marking thumb failure: %s", cache_path);
-
-               pathl = path_from_utf8(cache_path);
-               f = fopen(pathl, "w");
-               if (f)
+               if (success)
                        {
                        struct utimbuf ut;
-
-                       fclose(f);
+                       /* set thumb time to that of source file */
 
                        ut.actime = ut.modtime = filetime(tl->fd->path);
                        if (ut.modtime > 0)
                                {
                                utime(pathl, &ut);
                                }
-
-                       success = TRUE;
+                       }
+               else
+                       {
+                       DEBUG_1("Saving failed: %s", pathl);
                        }
 
                g_free(pathl);
@@ -129,6 +100,7 @@ static gint thumb_loader_mark_failure(ThumbLoader *tl)
                }
 
        g_free(cache_dir);
+
        return success;
 }
 
@@ -141,6 +113,12 @@ static void thumb_loader_percent_cb(ImageLoader *il, gdouble percent, gpointer d
        if (tl->func_progress) tl->func_progress(tl, tl->data);
 }
 
+static void thumb_loader_set_fallback(ThumbLoader *tl)
+{
+       if (tl->fd->thumb_pixbuf) g_object_unref(tl->fd->thumb_pixbuf);
+       tl->fd->thumb_pixbuf = pixbuf_fallback(tl->fd, tl->max_w, tl->max_h);
+}
+
 static void thumb_loader_done_cb(ImageLoader *il, gpointer data)
 {
        ThumbLoader *tl = data;
@@ -164,14 +142,7 @@ static void thumb_loader_done_cb(ImageLoader *il, gpointer data)
                {
                if (!tl->fd->exif_orientation)
                        {
-                       ExifData *exif = exif_read_fd(tl->fd);
-                       gint orientation;
-
-                       if (exif && exif_get_integer(exif, "Exif.Image.Orientation", &orientation))
-                               tl->fd->exif_orientation = orientation;
-                       else
-                               tl->fd->exif_orientation = EXIF_ORIENTATION_TOP_LEFT;
-                       exif_free_fd(tl->fd, exif);
+                       tl->fd->exif_orientation = metadata_read_int(tl->fd, ORIENTATION_KEY, EXIF_ORIENTATION_TOP_LEFT);
                        }
                
                if (tl->fd->exif_orientation != EXIF_ORIENTATION_TOP_LEFT)
@@ -190,9 +161,11 @@ static void thumb_loader_done_cb(ImageLoader *il, gpointer data)
                DEBUG_1("thumbnail size mismatch, regenerating: %s", tl->fd->path);
                tl->cache_hit = FALSE;
 
-               thumb_loader_setup(tl, tl->fd->path);
+               thumb_loader_setup(tl, tl->fd);
+       
+               g_signal_connect(G_OBJECT(tl->il), "done", (GCallback)thumb_loader_done_cb, tl);
 
-               if (!image_loader_start(tl->il, thumb_loader_done_cb, tl))
+               if (!image_loader_start(tl->il))
                        {
                        image_loader_free(tl->il);
                        tl->il = NULL;
@@ -209,16 +182,16 @@ static void thumb_loader_done_cb(ImageLoader *il, gpointer data)
                {
                gint w, h;
 
-               if (((double)tl->max_w / pw) < ((double)tl->max_h / ph))
+               if (((gdouble)tl->max_w / pw) < ((gdouble)tl->max_h / ph))
                        {
                        w = tl->max_w;
-                       h = (double)w / pw * ph;
+                       h = (gdouble)w / pw * ph;
                        if (h < 1) h = 1;
                        }
                else
                        {
                        h = tl->max_h;
-                       w = (double)h / ph * pw;
+                       w = (gdouble)h / ph * pw;
                        if (w < 1) w = 1;
                        }
                
@@ -235,17 +208,18 @@ static void thumb_loader_done_cb(ImageLoader *il, gpointer data)
                        {
                        if (tl->fd->thumb_pixbuf) g_object_unref(tl->fd->thumb_pixbuf);
                        tl->fd->thumb_pixbuf = pixbuf;
-                       gdk_pixbuf_ref(tl->fd->thumb_pixbuf);
+
+                       g_object_ref(tl->fd->thumb_pixbuf);
                        }
-               save = il->shrunk;
+               save = image_loader_get_shrunk(il);
                }
 
-       if (rotated) gdk_pixbuf_unref(rotated);
+       if (rotated) g_object_unref(rotated);
        
        /* save it ? */
        if (tl->cache_enable && save)
                {
-               thumb_loader_save_to_cache(tl);
+               thumb_loader_save_thumbnail(tl, FALSE);
                }
 
        if (tl->func_done) tl->func_done(tl, tl->data);
@@ -267,14 +241,16 @@ static void thumb_loader_error_cb(ImageLoader *il, gpointer data)
        image_loader_free(tl->il);
        tl->il = NULL;
 
+       thumb_loader_set_fallback(tl);
+       
        if (tl->func_error) tl->func_error(tl, tl->data);
 }
 
-static gint thumb_loader_done_delay_cb(gpointer data)
+static gboolean thumb_loader_done_delay_cb(gpointer data)
 {
        ThumbLoader *tl = data;
 
-       tl->idle_done_id = -1;
+       tl->idle_done_id = 0;
 
        if (tl->func_done) tl->func_done(tl, tl->data);
 
@@ -283,24 +259,20 @@ static gint thumb_loader_done_delay_cb(gpointer data)
 
 static void thumb_loader_delay_done(ThumbLoader *tl)
 {
-       if (tl->idle_done_id == -1) tl->idle_done_id = g_idle_add(thumb_loader_done_delay_cb, tl);
+       if (!tl->idle_done_id) tl->idle_done_id = g_idle_add(thumb_loader_done_delay_cb, tl);
 }
 
-static void thumb_loader_setup(ThumbLoader *tl, const gchar *path)
+static void thumb_loader_setup(ThumbLoader *tl, FileData *fd)
 {
-       FileData *fd = file_data_new_simple(path);
        image_loader_free(tl->il);
        tl->il = image_loader_new(fd);
-       file_data_unref(fd);
+       image_loader_set_priority(tl->il, G_PRIORITY_LOW);
 
-       if (options->thumbnails.fast)
-               {
-               /* this will speed up jpegs by up to 3x in some cases */
-               image_loader_set_requested_size(tl->il, tl->max_w, tl->max_h);
-               }
+       /* this will speed up jpegs by up to 3x in some cases */
+       image_loader_set_requested_size(tl->il, tl->max_w, tl->max_h);
 
-       image_loader_set_error_func(tl->il, thumb_loader_error_cb, tl);
-       if (tl->func_progress) image_loader_set_percent_func(tl->il, thumb_loader_percent_cb, tl);
+       g_signal_connect(G_OBJECT(tl->il), "error", (GCallback)thumb_loader_error_cb, tl);
+       if (tl->func_progress) g_signal_connect(G_OBJECT(tl->il), "percent", (GCallback)thumb_loader_percent_cb, tl);
 }
 
 void thumb_loader_set_callbacks(ThumbLoader *tl,
@@ -328,7 +300,7 @@ void thumb_loader_set_callbacks(ThumbLoader *tl,
        tl->data = data;
 }
 
-void thumb_loader_set_cache(ThumbLoader *tl, gint enable_cache, gint local, gint retry_failed)
+void thumb_loader_set_cache(ThumbLoader *tl, gboolean enable_cache, gboolean local, gboolean retry_failed)
 {
        if (!tl) return;
 
@@ -339,14 +311,10 @@ void thumb_loader_set_cache(ThumbLoader *tl, gint enable_cache, gint local, gint
                }
 
        tl->cache_enable = enable_cache;
-#if 0
-       tl->cache_local = local;
-       tl->cache_retry = retry_failed;
-#endif
 }
 
 
-gint thumb_loader_start(ThumbLoader *tl, FileData *fd)
+gboolean thumb_loader_start(ThumbLoader *tl, FileData *fd)
 {
        gchar *cache_path = NULL;
 
@@ -361,8 +329,6 @@ gint thumb_loader_start(ThumbLoader *tl, FileData *fd)
 
        if (!tl->fd) tl->fd = file_data_ref(fd);
 
-       if (tl->fd->thumb_pixbuf) g_object_unref(tl->fd->thumb_pixbuf);
-       tl->fd->thumb_pixbuf = pixbuf_fallback(tl->fd, tl->max_w, tl->max_h);
 
        if (tl->cache_enable)
                {
@@ -378,6 +344,7 @@ gint thumb_loader_start(ThumbLoader *tl, FileData *fd)
                                        {
                                        DEBUG_1("Broken image mark found:%s", cache_path);
                                        g_free(cache_path);
+                                       thumb_loader_set_fallback(tl);
                                        return FALSE;
                                        }
 
@@ -404,16 +371,19 @@ gint thumb_loader_start(ThumbLoader *tl, FileData *fd)
 
        if (cache_path)
                {
-               thumb_loader_setup(tl, cache_path);
+               FileData *fd = file_data_new_no_grouping(cache_path);
+               thumb_loader_setup(tl, fd);
+               file_data_unref(fd);
                g_free(cache_path);
                tl->cache_hit = TRUE;
                }
        else
                {
-               thumb_loader_setup(tl, tl->fd->path);
+               thumb_loader_setup(tl, tl->fd);
                }
 
-       if (!image_loader_start(tl->il, thumb_loader_done_cb, tl))
+       g_signal_connect(G_OBJECT(tl->il), "done", (GCallback)thumb_loader_done_cb, tl);
+       if (!image_loader_start(tl->il))
                {
                /* try from original if cache attempt */
                if (tl->cache_hit)
@@ -421,34 +391,25 @@ gint thumb_loader_start(ThumbLoader *tl, FileData *fd)
                        tl->cache_hit = FALSE;
                        log_printf("%s", _("Thumbnail image in cache failed to load, trying to recreate.\n"));
 
-                       thumb_loader_setup(tl, tl->fd->path);
-                       if (image_loader_start(tl->il, thumb_loader_done_cb, tl)) return TRUE;
+                       thumb_loader_setup(tl, tl->fd);
+                       g_signal_connect(G_OBJECT(tl->il), "done", (GCallback)thumb_loader_done_cb, tl);
+                       if (image_loader_start(tl->il)) return TRUE;
                        }
                /* mark failed thumbnail in cache with 0 byte file */
                if (tl->cache_enable)
                        {
-                       thumb_loader_mark_failure(tl);
+                       thumb_loader_save_thumbnail(tl, TRUE);
                        }
 
                image_loader_free(tl->il);
                tl->il = NULL;
+               thumb_loader_set_fallback(tl);
                return FALSE;
                }
 
        return TRUE;
 }
 
-#if 0
-gint thumb_loader_to_pixmap(ThumbLoader *tl, GdkPixmap **pixmap, GdkBitmap **mask)
-{
-       if (!tl || !tl->pixbuf) return -1;
-
-       gdk_pixbuf_render_pixmap_and_mask(tl->pixbuf, pixmap, mask, 128);
-
-       return thumb_loader_get_space(tl);
-}
-#endif
-
 GdkPixbuf *thumb_loader_get_pixbuf(ThumbLoader *tl)
 {
        GdkPixbuf *pixbuf;
@@ -471,39 +432,26 @@ GdkPixbuf *thumb_loader_get_pixbuf(ThumbLoader *tl)
        return pixbuf;
 }
 
-#if 0
-gint thumb_loader_get_space(ThumbLoader *tl)
-{
-       if (!tl) return 0;
-
-       if (tl->pixbuf) return (tl->max_w - gdk_pixbuf_get_width(tl->pixbuf));
-
-       return tl->max_w;
-}
-#endif
-
 ThumbLoader *thumb_loader_new(gint width, gint height)
 {
        ThumbLoader *tl;
 
-       if (options->thumbnails.spec_standard)
+       /* non-std thumb loader is more effective for configurations with disabled caching
+          because it loads the thumbnails at the required size. loader_std loads
+          the thumbnails at the sizes appropriate for standard cache (typically 256x256 pixels)
+          and then performs one additional scaling */
+       if (options->thumbnails.spec_standard && options->thumbnails.enable_caching)
                {
                return (ThumbLoader *)thumb_loader_std_new(width, height);
                }
 
        tl = g_new0(ThumbLoader, 1);
-       tl->standard_loader = FALSE;
-       tl->fd = NULL;
+       
        tl->cache_enable = options->thumbnails.enable_caching;
-       tl->cache_hit = FALSE;
        tl->percent_done = 0.0;
        tl->max_w = width;
        tl->max_h = height;
 
-       tl->il = NULL;
-
-       tl->idle_done_id = -1;
-
        return tl;
 }
 
@@ -520,37 +468,22 @@ void thumb_loader_free(ThumbLoader *tl)
        image_loader_free(tl->il);
        file_data_unref(tl->fd);
 
-       if (tl->idle_done_id != -1) g_source_remove(tl->idle_done_id);
+       if (tl->idle_done_id) g_source_remove(tl->idle_done_id);
 
        g_free(tl);
 }
 
-#if 0
-gint thumb_from_xpm_d(const char **data, gint max_w, gint max_h, GdkPixmap **pixmap, GdkBitmap **mask)
+/* release thumb_pixbuf on file change - this forces reload. */
+void thumb_notify_cb(FileData *fd, NotifyType type, gpointer data)
 {
-       GdkPixbuf *pixbuf;
-       gint w, h;
-
-       pixbuf = gdk_pixbuf_new_from_xpm_data(data);
-       w = gdk_pixbuf_get_width(pixbuf);
-       h = gdk_pixbuf_get_height(pixbuf);
-
-       if (pixbuf_scale_aspect(w, h, max_w, max_h, &w, &h))
+       if ((type & (NOTIFY_REREAD | NOTIFY_CHANGE)) && fd->thumb_pixbuf)
                {
-               /* scale */
-               GdkPixbuf *tmp;
-
-               tmp = pixbuf;
-               pixbuf = gdk_pixbuf_scale_simple(tmp, w, h, GDK_INTERP_NEAREST);
-               gdk_pixbuf_unref(tmp);
+               DEBUG_1("Notify thumb: %s %04x", fd->path, type);
+               g_object_unref(fd->thumb_pixbuf);
+               fd->thumb_pixbuf = NULL;
                }
-
-       gdk_pixbuf_render_pixmap_and_mask(pixbuf, pixmap, mask, 128);
-       gdk_pixbuf_unref(pixbuf);
-
-       return w;
 }
-#endif
+
 
 /*
  *-----------------------------------------------------------------------------
@@ -572,33 +505,32 @@ static guchar *load_xv_thumbnail(gchar *filename, gint *widthp, gint *heightp)
 {
        FILE *file;
        gchar buffer[XV_BUFFER];
-       guchar *data;
-       gint width, height, depth;
+       guchar *data = NULL;
 
        file = fopen(filename, "rt");
        if (!file) return NULL;
 
-       fgets(buffer, XV_BUFFER, file);
-       if (strncmp(buffer, "P7 332", 6) != 0)
+       if (fgets(buffer, XV_BUFFER, file) != NULL
+           && strncmp(buffer, "P7 332", 6) == 0)
                {
-               fclose(file);
-               return NULL;
-               }
+               gint width, height, depth;
 
-       while (fgets(buffer, XV_BUFFER, file) && buffer[0] == '#') /* do_nothing() */;
+               while (fgets(buffer, XV_BUFFER, file) && buffer[0] == '#') /* do_nothing() */;
 
-       if (sscanf(buffer, "%d %d %d", &width, &height, &depth) != 3)
-               {
-               fclose(file);
-               return NULL;
+               if (sscanf(buffer, "%d %d %d", &width, &height, &depth) == 3)
+                       {
+                       gsize size = width * height;
+                       
+                       data = g_new(guchar, size);
+                       if (data && fread(data, 1, size, file) == size)
+                               {
+                               *widthp = width;
+                               *heightp = height;
+                               }
+                       }
                }
 
-       data = g_new(guchar, width * height);
-       fread(data, 1, width * height, file);
-
        fclose(file);
-       *widthp = width;
-       *heightp = height;
        return data;
 }
 #undef XV_BUFFER
@@ -655,7 +587,7 @@ static GdkPixbuf *get_xv_thumbnail(gchar *thumb_filename, gint max_w, gint max_h
 
                        tmp = pixbuf;
                        pixbuf = gdk_pixbuf_scale_simple(tmp, width, height, GDK_INTERP_NEAREST);
-                       gdk_pixbuf_unref(tmp);
+                       g_object_unref(tmp);
                        }
 
                return pixbuf;
@@ -663,3 +595,4 @@ static GdkPixbuf *get_xv_thumbnail(gchar *thumb_filename, gint max_w, gint max_h
 
        return NULL;
 }
+/* vim: set shiftwidth=8 softtabstop=0 cindent cinoptions={1s: */