Fix #381: Feature-Request: Make JPEG comment available for overlays
[geeqie.git] / src / exiv2.cc
index 17cd839..0ad5c61 100644 (file)
@@ -1,23 +1,31 @@
 /*
- * Geeqie
- * Copyright (C) 2008 The Geeqie Team
+ * Copyright (C) 2008 - 2016 The Geeqie Team
  *
  * Author: Vladimir Nadvornik
  *
- * 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.
  */
 
-#ifdef HAVE_CONFIG_H
-#  include "config.h"
-#endif
+#include "config.h"
 
 #ifdef HAVE_EXIV2
 
 #include <exiv2/image.hpp>
 #include <exiv2/exif.hpp>
 #include <iostream>
+#include <string>
 
 // EXIV2_TEST_VERSION is defined in Exiv2 0.15 and newer.
 #ifndef EXIV2_TEST_VERSION
@@ -32,6 +40,7 @@
 #include <fcntl.h>
 #include <sys/mman.h>
 
+#if !EXIV2_TEST_VERSION(0,17,90)
 #include <exiv2/tiffparser.hpp>
 #include <exiv2/tiffcomposite.hpp>
 #include <exiv2/tiffvisitor.hpp>
 #include <exiv2/rafimage.hpp>
 #endif
 #include <exiv2/futils.hpp>
+#else
+#include <exiv2/preview.hpp>
+#endif
 
-
+#if EXIV2_TEST_VERSION(0,17,0)
+#include <exiv2/convert.hpp>
+#include <exiv2/xmpsidecar.hpp>
+#endif
 
 extern "C" {
-#include <glib.h> 
+#include <glib.h>
 
 #include "main.h"
 #include "exif.h"
 
 #include "filefilter.h"
 #include "ui_fileops.h"
+
+#include "misc.h"
+}
+
+typedef struct _AltKey AltKey;
+
+struct _AltKey
+{
+       const gchar *xmp_key;
+       const gchar *exif_key;
+       const gchar *iptc_key;
+};
+
+/* this is a list of keys that should be converted, even with the older Exiv2 which does not support it directly */
+static const AltKey alt_keys[] = {
+       {"Xmp.tiff.Orientation",                "Exif.Image.Orientation",       NULL},
+       {"Xmp.dc.title",                        NULL,                           "Iptc.Application2.ObjectName"          },
+       {"Xmp.photoshop.Urgency",               NULL,                           "Iptc.Application2.Urgency"             },
+       {"Xmp.photoshop.Category",              NULL,                           "Iptc.Application2.Category"            },
+       {"Xmp.photoshop.SupplementalCategory",  NULL,                           "Iptc.Application2.SuppCategory"        },
+       {"Xmp.dc.subject",                      NULL,                           "Iptc.Application2.Keywords"            },
+       {"Xmp.iptc.Location",                   NULL,                           "Iptc.Application2.LocationName"        },
+       {"Xmp.photoshop.Instruction",           NULL,                           "Iptc.Application2.SpecialInstructions" },
+       {"Xmp.photoshop.DateCreated",           NULL,                           "Iptc.Application2.DateCreated"         },
+       {"Xmp.dc.creator",                      NULL,                           "Iptc.Application2.Byline"              },
+       {"Xmp.photoshop.AuthorsPosition",       NULL,                           "Iptc.Application2.BylineTitle"         },
+       {"Xmp.photoshop.City",                  NULL,                           "Iptc.Application2.City"                },
+       {"Xmp.photoshop.State",                 NULL,                           "Iptc.Application2.ProvinceState"       },
+       {"Xmp.iptc.CountryCode",                NULL,                           "Iptc.Application2.CountryCode"         },
+       {"Xmp.photoshop.Country",               NULL,                           "Iptc.Application2.CountryName"         },
+       {"Xmp.photoshop.TransmissionReference", NULL,                           "Iptc.Application2.TransmissionReference"},
+       {"Xmp.photoshop.Headline",              NULL,                           "Iptc.Application2.Headline"            },
+       {"Xmp.photoshop.Credit",                NULL,                           "Iptc.Application2.Credit"              },
+       {"Xmp.photoshop.Source",                NULL,                           "Iptc.Application2.Source"              },
+       {"Xmp.dc.rights",                       NULL,                           "Iptc.Application2.Copyright"           },
+       {"Xmp.dc.description",                  NULL,                           "Iptc.Application2.Caption"             },
+       {"Xmp.photoshop.CaptionWriter",         NULL,                           "Iptc.Application2.Writer"              },
+       {NULL, NULL, NULL}
+       };
+
+static void _debug_exception(const char* file,
+                             int line,
+                             const char* func,
+                             Exiv2::AnyError& e)
+{
+       gchar *str = g_locale_from_utf8(e.what(), -1, NULL, NULL, NULL);
+       DEBUG_1("%s:%d:%s:Exiv2: %s", file, line, func, str);
+       g_free(str);
 }
 
+#define debug_exception(e) _debug_exception(__FILE__, __LINE__, __func__, e)
+
 struct _ExifData
 {
-       Exiv2::Image::AutoPtr image;
-       Exiv2::Image::AutoPtr sidecar;
        Exiv2::ExifData::const_iterator exifIter; /* for exif_get_next_item */
        Exiv2::IptcData::const_iterator iptcIter; /* for exif_get_next_item */
 #if EXIV2_TEST_VERSION(0,16,0)
        Exiv2::XmpData::const_iterator xmpIter; /* for exif_get_next_item */
 #endif
-       bool have_sidecar;
+
+       virtual ~_ExifData()
+       {
+       }
+
+       virtual void writeMetadata(gchar *path = NULL)
+       {
+               g_critical("Unsupported method of writing metadata");
+       }
+
+       virtual ExifData *original()
+       {
+               return NULL;
+       }
+
+       virtual Exiv2::Image *image() = 0;
+
+       virtual Exiv2::ExifData &exifData() = 0;
+
+       virtual Exiv2::IptcData &iptcData() = 0;
+
+#if EXIV2_TEST_VERSION(0,16,0)
+       virtual Exiv2::XmpData &xmpData() = 0;
+#endif
+
+       virtual void add_jpeg_color_profile(unsigned char *cp_data, guint cp_length) = 0;
+
+       virtual guchar *get_jpeg_color_profile(guint *data_len) = 0;
+
+       virtual std::string image_comment() const = 0;
+
+       virtual void set_image_comment(const std::string& comment) = 0;
+};
+
+// This allows read-only access to the original metadata
+struct _ExifDataOriginal : public _ExifData
+{
+protected:
+       Exiv2::Image::AutoPtr image_;
 
        /* the icc profile in jpeg is not technically exif - store it here */
-       unsigned char *cp_data;
-       guint cp_length;
+       unsigned char *cp_data_;
+       guint cp_length_;
+       gboolean valid_;
+       gchar *pathl_;
 
-       _ExifData(gchar *path, gchar *sidecar_path)
+       Exiv2::ExifData emptyExifData_;
+       Exiv2::IptcData emptyIptcData_;
+#if EXIV2_TEST_VERSION(0,16,0)
+       Exiv2::XmpData emptyXmpData_;
+#endif
+
+public:
+       _ExifDataOriginal(Exiv2::Image::AutoPtr image)
        {
-               have_sidecar = false;
-               cp_data = NULL;
-               cp_length = 0;
-               gchar *pathl = path_from_utf8(path);
-               image = Exiv2::ImageFactory::open(pathl);
-               g_free(pathl);
-//             g_assert (image.get() != 0);
-               image->readMetadata();
+               cp_data_ = NULL;
+               cp_length_ = 0;
+               image_ = image;
+               valid_ = TRUE;
+       }
+
+       _ExifDataOriginal(gchar *path)
+       {
+               cp_data_ = NULL;
+               cp_length_ = 0;
+               valid_ = TRUE;
+
+               pathl_ = path_from_utf8(path);
+               try
+                       {
+                       image_ = Exiv2::ImageFactory::open(pathl_);
+//                     g_assert (image.get() != 0);
+                       image_->readMetadata();
 
 #if EXIV2_TEST_VERSION(0,16,0)
-               DEBUG_2("xmp count %li", image->xmpData().count());
-               if (sidecar_path && image->xmpData().empty())
+                       if (image_->mimeType() == "application/rdf+xml")
+                               {
+                               //Exiv2 sidecar converts xmp to exif and iptc, we don't want it.
+                               image_->clearExifData();
+                               image_->clearIptcData();
+                               }
+#endif
+
+#if EXIV2_TEST_VERSION(0,14,0)
+                       if (image_->mimeType() == "image/jpeg")
+                               {
+                               /* try to get jpeg color profile */
+                               Exiv2::BasicIo &io = image_->io();
+                               gint open = io.isopen();
+                               if (!open) io.open();
+                               if (io.isopen())
+                                       {
+                                       unsigned char *mapped = (unsigned char*)io.mmap();
+                                       if (mapped) exif_jpeg_parse_color(this, mapped, io.size());
+                                       io.munmap();
+                                       }
+                               if (!open) io.close();
+                               }
+#endif
+                       }
+               catch (Exiv2::AnyError& e)
                        {
-                       gchar *sidecar_pathl = path_from_utf8(sidecar_path);
-                       sidecar = Exiv2::ImageFactory::open(sidecar_pathl);
-                       g_free(sidecar_pathl);
-                       sidecar->readMetadata();
-                       have_sidecar = sidecar->good();
-                       DEBUG_2("sidecar xmp count %li", sidecar->xmpData().count());
+                       valid_ = FALSE;
                        }
+       }
+
+       virtual ~_ExifDataOriginal()
+       {
+               if (cp_data_) g_free(cp_data_);
+               if (pathl_) g_free(pathl_);
+       }
+
+       virtual Exiv2::Image *image()
+       {
+               if (!valid_) return NULL;
+               return image_.get();
+       }
+
+       virtual Exiv2::ExifData &exifData ()
+       {
+               if (!valid_) return emptyExifData_;
+               return image_->exifData();
+       }
 
+       virtual Exiv2::IptcData &iptcData ()
+       {
+               if (!valid_) return emptyIptcData_;
+               return image_->iptcData();
+       }
+
+#if EXIV2_TEST_VERSION(0,16,0)
+       virtual Exiv2::XmpData &xmpData ()
+       {
+               if (!valid_) return emptyXmpData_;
+               return image_->xmpData();
+       }
 #endif
-#if EXIV2_TEST_VERSION(0,14,0)
-               if (image->mimeType() == std::string("image/jpeg"))
+
+       virtual void add_jpeg_color_profile(unsigned char *cp_data, guint cp_length)
+       {
+               if (cp_data_) g_free(cp_data_);
+               cp_data_ = cp_data;
+               cp_length_ = cp_length;
+       }
+
+       virtual guchar *get_jpeg_color_profile(guint *data_len)
+       {
+               if (cp_data_)
+               {
+                       if (data_len) *data_len = cp_length_;
+                       return (unsigned char *) g_memdup(cp_data_, cp_length_);
+               }
+               return NULL;
+       }
+
+       virtual std::string image_comment() const
+       {
+               return image_.get() ? image_->comment() : "";
+       }
+
+       virtual void set_image_comment(const std::string& comment)
+       {
+               if (image_.get())
+                       image_->setComment(comment);
+       }
+};
+
+extern "C" {
+static void _ExifDataProcessed_update_xmp(gpointer key, gpointer value, gpointer data);
+}
+
+// This allows read-write access to the metadata
+struct _ExifDataProcessed : public _ExifData
+{
+protected:
+       _ExifDataOriginal *imageData_;
+       _ExifDataOriginal *sidecarData_;
+
+       Exiv2::ExifData exifData_;
+       Exiv2::IptcData iptcData_;
+#if EXIV2_TEST_VERSION(0,16,0)
+       Exiv2::XmpData xmpData_;
+#endif
+
+public:
+       _ExifDataProcessed(gchar *path, gchar *sidecar_path, GHashTable *modified_xmp)
+       {
+               imageData_ = new _ExifDataOriginal(path);
+               sidecarData_ = NULL;
+#if EXIV2_TEST_VERSION(0,16,0)
+               if (sidecar_path)
+                       {
+                       sidecarData_ = new _ExifDataOriginal(sidecar_path);
+                       xmpData_ = sidecarData_->xmpData();
+                       }
+               else
                        {
-                       /* try to get jpeg color profile */
-                       Exiv2::BasicIo &io = image->io();
-                       gint open = io.isopen();
-                       if (!open) io.open();
-                       unsigned char *mapped = (unsigned char*)io.mmap();
-                       if (mapped) exif_jpeg_parse_color(this, mapped, io.size());
-                       io.munmap();
-                       if (!open) io.close();
-                       }
-#endif         
-       }
-       
-       ~_ExifData()
+                       xmpData_ = imageData_->xmpData();
+                       }
+
+#endif
+               exifData_ = imageData_->exifData();
+               iptcData_ = imageData_->iptcData();
+#if EXIV2_TEST_VERSION(0,17,0)
+               try
+                       {
+                       syncExifWithXmp(exifData_, xmpData_);
+                       }
+               catch (...)
+                       {
+                       DEBUG_1("Exiv2: Catching bug\n");
+                       }
+#endif
+               if (modified_xmp)
+                       {
+                       g_hash_table_foreach(modified_xmp, _ExifDataProcessed_update_xmp, this);
+                       }
+       }
+
+       virtual ~_ExifDataProcessed()
        {
-               if (cp_data) g_free(cp_data);
+               if (imageData_) delete imageData_;
+               if (sidecarData_) delete sidecarData_;
        }
-       
-       void writeMetadata()
+
+       virtual ExifData *original()
        {
-               if (have_sidecar) sidecar->writeMetadata();
-               image->writeMetadata();
+               return imageData_;
        }
-       
-       Exiv2::ExifData &exifData ()
+
+       virtual void writeMetadata(gchar *path = NULL)
        {
-               return image->exifData();
+               if (!path)
+                       {
+#if EXIV2_TEST_VERSION(0,17,0)
+                       if (options->metadata.save_legacy_IPTC)
+                               copyXmpToIptc(xmpData_, iptcData_);
+                       else
+                               iptcData_.clear();
+
+                       copyXmpToExif(xmpData_, exifData_);
+#endif
+                       Exiv2::Image *image = imageData_->image();
+
+                       if (!image) Exiv2::Error(21);
+                       image->setExifData(exifData_);
+                       image->setIptcData(iptcData_);
+#if EXIV2_TEST_VERSION(0,16,0)
+                       image->setXmpData(xmpData_);
+#endif
+                       image->writeMetadata();
+                       }
+               else
+                       {
+#if EXIV2_TEST_VERSION(0,17,0)
+                       gchar *pathl = path_from_utf8(path);;
+
+                       Exiv2::Image::AutoPtr sidecar = Exiv2::ImageFactory::create(Exiv2::ImageType::xmp, pathl);
+
+                       g_free(pathl);
+
+                       sidecar->setXmpData(xmpData_);
+                       sidecar->writeMetadata();
+#else
+                       throw Exiv2::Error(3, "xmp");
+#endif
+                       }
        }
 
-       Exiv2::IptcData &iptcData ()
+       virtual Exiv2::Image *image()
        {
-               return image->iptcData();
+               return imageData_->image();
+       }
+
+       virtual Exiv2::ExifData &exifData ()
+       {
+               return exifData_;
+       }
+
+       virtual Exiv2::IptcData &iptcData ()
+       {
+               return iptcData_;
        }
 
 #if EXIV2_TEST_VERSION(0,16,0)
-       Exiv2::XmpData &xmpData ()
+       virtual Exiv2::XmpData &xmpData ()
        {
-               return have_sidecar ? sidecar->xmpData() : image->xmpData();
+               return xmpData_;
        }
 #endif
 
+       virtual void add_jpeg_color_profile(unsigned char *cp_data, guint cp_length)
+       {
+               imageData_->add_jpeg_color_profile(cp_data, cp_length);
+       }
+
+       virtual guchar *get_jpeg_color_profile(guint *data_len)
+       {
+               return imageData_->get_jpeg_color_profile(data_len);
+       }
+
+       virtual std::string image_comment() const
+       {
+               return imageData_->image_comment();
+       }
+
+       virtual void set_image_comment(const std::string& comment)
+       {
+               imageData_->set_image_comment(comment);
+       }
 };
 
+
+
+
 extern "C" {
 
-ExifData *exif_read(gchar *path, gchar *sidecar_path)
+
+void exif_init(void)
+{
+#ifdef EXV_ENABLE_NLS
+       bind_textdomain_codeset (EXV_PACKAGE, "UTF-8");
+#endif
+}
+
+
+
+static void _ExifDataProcessed_update_xmp(gpointer key, gpointer value, gpointer data)
+{
+       exif_update_metadata((ExifData *)data, (gchar *)key, (GList *)value);
+}
+
+ExifData *exif_read(gchar *path, gchar *sidecar_path, GHashTable *modified_xmp)
 {
        DEBUG_1("exif read %s, sidecar: %s", path, sidecar_path ? sidecar_path : "-");
        try {
-               return new ExifData(path, sidecar_path);
+               return new _ExifDataProcessed(path, sidecar_path, modified_xmp);
        }
        catch (Exiv2::AnyError& e) {
-               std::cout << "Caught Exiv2 exception '" << e << "'\n";
+               debug_exception(e);
                return NULL;
        }
-       
+
 }
 
-int exif_write(ExifData *exif)
+gboolean exif_write(ExifData *exif)
 {
        try {
                exif->writeMetadata();
-               return 1;
+               return TRUE;
        }
        catch (Exiv2::AnyError& e) {
-               std::cout << "Caught Exiv2 exception '" << e << "'\n";
-               return 0;
+               debug_exception(e);
+               return FALSE;
+       }
+}
+
+gboolean exif_write_sidecar(ExifData *exif, gchar *path)
+{
+       try {
+               exif->writeMetadata(path);
+               return TRUE;
        }
-       
+       catch (Exiv2::AnyError& e) {
+               debug_exception(e);
+               return FALSE;
+       }
+
 }
 
 
 void exif_free(ExifData *exif)
 {
-       
+       if (!exif) return;
+       g_assert(dynamic_cast<_ExifDataProcessed *>(exif)); // this should not be called on ExifDataOriginal
        delete exif;
 }
 
+ExifData *exif_get_original(ExifData *exif)
+{
+       return exif->original();
+}
+
+
 ExifItem *exif_get_item(ExifData *exif, const gchar *key)
 {
        try {
@@ -206,7 +543,7 @@ ExifItem *exif_get_item(ExifData *exif, const gchar *key)
                return (ExifItem *)item;
        }
        catch (Exiv2::AnyError& e) {
-               std::cout << "Caught Exiv2 exception '" << e << "'\n";
+               debug_exception(e);
                return NULL;
        }
 }
@@ -243,7 +580,7 @@ ExifItem *exif_add_item(ExifData *exif, const gchar *key)
                return (ExifItem *)item;
        }
        catch (Exiv2::AnyError& e) {
-               std::cout << "Caught Exiv2 exception '" << e << "'\n";
+               debug_exception(e);
                return NULL;
        }
 }
@@ -257,20 +594,20 @@ ExifItem *exif_get_first_item(ExifData *exif)
 #if EXIV2_TEST_VERSION(0,16,0)
                exif->xmpIter = exif->xmpData().begin();
 #endif
-               if (exif->exifIter != exif->exifData().end()) 
+               if (exif->exifIter != exif->exifData().end())
                        {
                        const Exiv2::Metadatum *item = &*exif->exifIter;
                        exif->exifIter++;
                        return (ExifItem *)item;
                        }
-               if (exif->iptcIter != exif->iptcData().end()) 
+               if (exif->iptcIter != exif->iptcData().end())
                        {
                        const Exiv2::Metadatum *item = &*exif->iptcIter;
                        exif->iptcIter++;
                        return (ExifItem *)item;
                        }
 #if EXIV2_TEST_VERSION(0,16,0)
-               if (exif->xmpIter != exif->xmpData().end()) 
+               if (exif->xmpIter != exif->xmpData().end())
                        {
                        const Exiv2::Metadatum *item = &*exif->xmpIter;
                        exif->xmpIter++;
@@ -278,10 +615,10 @@ ExifItem *exif_get_first_item(ExifData *exif)
                        }
 #endif
                return NULL;
-                       
+
        }
        catch (Exiv2::AnyError& e) {
-               std::cout << "Caught Exiv2 exception '" << e << "'\n";
+               debug_exception(e);
                return NULL;
        }
 }
@@ -312,7 +649,7 @@ ExifItem *exif_get_next_item(ExifData *exif)
                return NULL;
        }
        catch (Exiv2::AnyError& e) {
-               std::cout << "Caught Exiv2 exception '" << e << "'\n";
+               debug_exception(e);
                return NULL;
        }
 }
@@ -324,7 +661,7 @@ char *exif_item_get_tag_name(ExifItem *item)
                return g_strdup(((Exiv2::Metadatum *)item)->key().c_str());
        }
        catch (Exiv2::AnyError& e) {
-               std::cout << "Caught Exiv2 exception '" << e << "'\n";
+               debug_exception(e);
                return NULL;
        }
 }
@@ -336,7 +673,7 @@ guint exif_item_get_tag_id(ExifItem *item)
                return ((Exiv2::Metadatum *)item)->tag();
        }
        catch (Exiv2::AnyError& e) {
-               std::cout << "Caught Exiv2 exception '" << e << "'\n";
+               debug_exception(e);
                return 0;
        }
 }
@@ -348,7 +685,7 @@ guint exif_item_get_elements(ExifItem *item)
                return ((Exiv2::Metadatum *)item)->count();
        }
        catch (Exiv2::AnyError& e) {
-               std::cout << "Caught Exiv2 exception '" << e << "'\n";
+               debug_exception(e);
                return 0;
        }
 }
@@ -358,14 +695,14 @@ char *exif_item_get_data(ExifItem *item, guint *data_len)
        try {
                if (!item) return 0;
                Exiv2::Metadatum *md = (Exiv2::Metadatum *)item;
-               if(data_len) *data_len = md->size();
+               if (data_len) *data_len = md->size();
                char *data = (char *)g_malloc(md->size());
                long res = md->copy((Exiv2::byte *)data, Exiv2::littleEndian /* should not matter */);
                g_assert(res == md->size());
                return data;
        }
        catch (Exiv2::AnyError& e) {
-               std::cout << "Caught Exiv2 exception '" << e << "'\n";
+               debug_exception(e);
                return NULL;
        }
 }
@@ -374,10 +711,10 @@ char *exif_item_get_description(ExifItem *item)
 {
        try {
                if (!item) return NULL;
-               return g_strdup(((Exiv2::Metadatum *)item)->tagLabel().c_str());
+               return utf8_validate_or_convert(((Exiv2::Metadatum *)item)->tagLabel().c_str());
        }
        catch (std::exception& e) {
-//             std::cout << "Caught Exiv2 exception '" << e << "'\n";
+//             debug_exception(e);
                return NULL;
        }
 }
@@ -388,7 +725,7 @@ invalidTypeId, unsignedByte, asciiString, unsignedShort,
   signedShort, signedLong, signedRational, string,
   date, time, comment, directory,
   xmpText, xmpAlt, xmpBag, xmpSeq,
-  langAlt, lastTypeId 
+  langAlt, lastTypeId
 */
 
 static guint format_id_trans_tbl [] = {
@@ -412,8 +749,8 @@ static guint format_id_trans_tbl [] = {
        EXIF_FORMAT_STRING,
        EXIF_FORMAT_STRING
        };
-       
-       
+
+
 
 guint exif_item_get_format_id(ExifItem *item)
 {
@@ -424,19 +761,19 @@ guint exif_item_get_format_id(ExifItem *item)
                return format_id_trans_tbl[id];
        }
        catch (Exiv2::AnyError& e) {
-               std::cout << "Caught Exiv2 exception '" << e << "'\n";
+               debug_exception(e);
                return EXIF_FORMAT_UNKNOWN;
        }
 }
 
-const char *exif_item_get_format_name(ExifItem *item, gint brief)
+const char *exif_item_get_format_name(ExifItem *item, gboolean brief)
 {
        try {
                if (!item) return NULL;
                return ((Exiv2::Metadatum *)item)->typeName();
        }
        catch (Exiv2::AnyError& e) {
-               std::cout << "Caught Exiv2 exception '" << e << "'\n";
+               debug_exception(e);
                return NULL;
        }
 }
@@ -448,12 +785,14 @@ gchar *exif_item_get_data_as_text(ExifItem *item)
                if (!item) return NULL;
                Exiv2::Metadatum *metadatum = (Exiv2::Metadatum *)item;
 #if EXIV2_TEST_VERSION(0,17,0)
-               return g_strdup(metadatum->print().c_str());
+               return utf8_validate_or_convert(metadatum->print().c_str());
 #else
                std::stringstream str;
                Exiv2::Exifdatum *exifdatum;
                Exiv2::Iptcdatum *iptcdatum;
+#if EXIV2_TEST_VERSION(0,16,0)
                Exiv2::Xmpdatum *xmpdatum;
+#endif
                if ((exifdatum = dynamic_cast<Exiv2::Exifdatum *>(metadatum)))
                        str << *exifdatum;
                else if ((iptcdatum = dynamic_cast<Exiv2::Iptcdatum *>(metadatum)))
@@ -463,7 +802,7 @@ gchar *exif_item_get_data_as_text(ExifItem *item)
                        str << *xmpdatum;
 #endif
 
-               return g_strdup(str.str().c_str());
+               return utf8_validate_or_convert(str.str().c_str());
 #endif
        }
        catch (Exiv2::AnyError& e) {
@@ -482,13 +821,13 @@ gchar *exif_item_get_string(ExifItem *item, int idx)
                std::string str = em->toString(); // FIXME
 #endif
                if (idx == 0 && str == "") str = em->toString();
-               if (str.length() > 5 && str.substr(0, 5) == "lang=") 
+               if (str.length() > 5 && str.substr(0, 5) == "lang=")
                        {
-                       std::string::size_type pos = str.find_first_of(' ');
-                       if (pos != std::string::npos) str = str.substr(pos+1);
+                       std::string::size_type pos = str.find_first_of(' ');
+                       if (pos != std::string::npos) str = str.substr(pos+1);
                        }
 
-               return g_strdup(str.c_str());
+               return utf8_validate_or_convert(str.c_str());
        }
        catch (Exiv2::AnyError& e) {
                return NULL;
@@ -499,21 +838,22 @@ gchar *exif_item_get_string(ExifItem *item, int idx)
 gint exif_item_get_integer(ExifItem *item, gint *value)
 {
        try {
-               if (!item) return 0;
+               if (!item || exif_item_get_elements(item) == 0) return 0;
                *value = ((Exiv2::Metadatum *)item)->toLong();
                return 1;
        }
        catch (Exiv2::AnyError& e) {
-               std::cout << "Caught Exiv2 exception '" << e << "'\n";
+               debug_exception(e);
                return 0;
        }
 }
 
-ExifRational *exif_item_get_rational(ExifItem *item, gint *sign)
+ExifRational *exif_item_get_rational(ExifItem *item, gint *sign, guint n)
 {
        try {
                if (!item) return NULL;
-               Exiv2::Rational v = ((Exiv2::Metadatum *)item)->toRational();
+               if (n >= exif_item_get_elements(item)) return NULL;
+               Exiv2::Rational v = ((Exiv2::Metadatum *)item)->toRational(n);
                static ExifRational ret;
                ret.num = v.first;
                ret.den = v.second;
@@ -521,115 +861,505 @@ ExifRational *exif_item_get_rational(ExifItem *item, gint *sign)
                return &ret;
        }
        catch (Exiv2::AnyError& e) {
-               std::cout << "Caught Exiv2 exception '" << e << "'\n";
+               debug_exception(e);
                return NULL;
        }
 }
 
-const gchar *exif_get_tag_description_by_key(const gchar *key)
+gchar *exif_get_tag_description_by_key(const gchar *key)
 {
        try {
                Exiv2::ExifKey ekey(key);
-               return Exiv2::ExifTags::tagLabel(ekey.tag(), ekey.ifdId ());
+               return utf8_validate_or_convert(ekey.tagLabel().c_str());
        }
        catch (Exiv2::AnyError& e) {
-               std::cout << "Caught Exiv2 exception '" << e << "'\n";
-               return NULL;
+               try {
+                       Exiv2::IptcKey ikey(key);
+                       return utf8_validate_or_convert(ikey.tagLabel().c_str());
+               }
+               catch (Exiv2::AnyError& e) {
+                       try {
+#if EXIV2_TEST_VERSION(0,16,0)
+                               Exiv2::XmpKey xkey(key);
+                               return utf8_validate_or_convert(xkey.tagLabel().c_str());
+#endif
+                       }
+                       catch (Exiv2::AnyError& e) {
+                               debug_exception(e);
+                               return NULL;
+                       }
+               }
        }
+       return NULL;
+}
+
+static const AltKey *find_alt_key(const gchar *xmp_key)
+{
+       gint i = 0;
+
+       while (alt_keys[i].xmp_key)
+               {
+               if (strcmp(xmp_key, alt_keys[i].xmp_key) == 0) return &alt_keys[i];
+               i++;
+               }
+       return NULL;
 }
 
-int exif_item_set_string(ExifItem *item, const char *str)
+static gint exif_update_metadata_simple(ExifData *exif, const gchar *key, const GList *values)
 {
        try {
-               if (!item) return 0;
-               ((Exiv2::Metadatum *)item)->setValue(std::string(str));
+               const GList *work = values;
+
+               try {
+                       Exiv2::ExifKey ekey(key);
+
+                       Exiv2::ExifData::iterator pos = exif->exifData().findKey(ekey);
+                       while (pos != exif->exifData().end())
+                               {
+                               exif->exifData().erase(pos);
+                               pos = exif->exifData().findKey(ekey);
+                               }
+
+                       while (work)
+                               {
+                               exif->exifData()[key] = (gchar *)work->data;
+                               work = work->next;
+                               }
+               }
+               catch (Exiv2::AnyError& e) {
+#if EXIV2_TEST_VERSION(0,16,0)
+                       try
+#endif
+                       {
+                               Exiv2::IptcKey ekey(key);
+                               Exiv2::IptcData::iterator pos = exif->iptcData().findKey(ekey);
+                               while (pos != exif->iptcData().end())
+                                       {
+                                       exif->iptcData().erase(pos);
+                                       pos = exif->iptcData().findKey(ekey);
+                                       }
+
+                               while (work)
+                                       {
+                                       exif->iptcData()[key] = (gchar *)work->data;
+                                       work = work->next;
+                                       }
+                       }
+#if EXIV2_TEST_VERSION(0,16,0)
+                       catch (Exiv2::AnyError& e) {
+                               Exiv2::XmpKey ekey(key);
+                               Exiv2::XmpData::iterator pos = exif->xmpData().findKey(ekey);
+                               while (pos != exif->xmpData().end())
+                                       {
+                                       exif->xmpData().erase(pos);
+                                       pos = exif->xmpData().findKey(ekey);
+                                       }
+
+                               while (work)
+                                       {
+                                       exif->xmpData()[key] = (gchar *)work->data;
+                                       work = work->next;
+                                       }
+                       }
+#endif
+               }
                return 1;
        }
        catch (Exiv2::AnyError& e) {
+               debug_exception(e);
                return 0;
        }
 }
 
-int exif_item_delete(ExifData *exif, ExifItem *item)
+gint exif_update_metadata(ExifData *exif, const gchar *key, const GList *values)
 {
-       try {
-               if (!item) return 0;
-               for (Exiv2::ExifData::iterator i = exif->exifData().begin(); i != exif->exifData().end(); ++i) {
-                       if (((Exiv2::Metadatum *)item) == &*i) {
-                               i = exif->exifData().erase(i);
-                               return 1;
-                       }
+       gint ret = exif_update_metadata_simple(exif, key, values);
+
+       if (
+#if !EXIV2_TEST_VERSION(0,17,0)
+           TRUE || /* no conversion support */
+#endif
+           !values || /* deleting item */
+           !ret  /* writing to the explicitely given xmp tag failed */
+           )
+               {
+               /* deleted xmp metadatum can't be converted, we have to delete also the corresponding legacy tag */
+               /* if we can't write xmp, update at least the legacy tag */
+               const AltKey *alt_key = find_alt_key(key);
+               if (alt_key && alt_key->iptc_key)
+                       ret = exif_update_metadata_simple(exif, alt_key->iptc_key, values);
+
+               if (alt_key && alt_key->exif_key)
+                       ret = exif_update_metadata_simple(exif, alt_key->exif_key, values);
                }
-               for (Exiv2::IptcData::iterator i = exif->iptcData().begin(); i != exif->iptcData().end(); ++i) {
-                       if (((Exiv2::Metadatum *)item) == &*i) {
-                               i = exif->iptcData().erase(i);
-                               return 1;
+       return ret;
+}
+
+
+static GList *exif_add_value_to_glist(GList *list, Exiv2::Metadatum &item, MetadataFormat format, const Exiv2::ExifData *metadata)
+{
+#if EXIV2_TEST_VERSION(0,16,0)
+       Exiv2::TypeId id = item.typeId();
+       if (format == METADATA_FORMATTED ||
+           id == Exiv2::asciiString ||
+           id == Exiv2::undefined ||
+           id == Exiv2::string ||
+           id == Exiv2::date ||
+           id == Exiv2::time ||
+           id == Exiv2::xmpText ||
+           id == Exiv2::langAlt ||
+           id == Exiv2::comment
+           )
+               {
+#endif
+               /* read as a single entry */
+               std::string str;
+
+               if (format == METADATA_FORMATTED)
+                       {
+#if EXIV2_TEST_VERSION(0,17,0)
+                       str = item.print(
+#if EXIV2_TEST_VERSION(0,18,0)
+                                       metadata
+#endif
+                                       );
+#else
+                       std::stringstream stream;
+                       Exiv2::Exifdatum *exifdatum;
+                       Exiv2::Iptcdatum *iptcdatum;
+#if EXIV2_TEST_VERSION(0,16,0)
+                       Exiv2::Xmpdatum *xmpdatum;
+#endif
+                       if ((exifdatum = dynamic_cast<Exiv2::Exifdatum *>(&item)))
+                               stream << *exifdatum;
+                       else if ((iptcdatum = dynamic_cast<Exiv2::Iptcdatum *>(&item)))
+                               stream << *iptcdatum;
+#if EXIV2_TEST_VERSION(0,16,0)
+                       else if ((xmpdatum = dynamic_cast<Exiv2::Xmpdatum *>(&item)))
+                               stream << *xmpdatum;
+#endif
+                       str = stream.str();
+#endif
+                       if (str.length() > 1024)
+                               {
+                               /* truncate very long strings, they cause problems in gui */
+                               str.erase(1024);
+                               str.append("...");
+                               }
+                       }
+               else
+                       {
+                       str = item.toString();
+                       }
+               if (str.length() > 5 && str.substr(0, 5) == "lang=")
+                       {
+                       std::string::size_type pos = str.find_first_of(' ');
+                       if (pos != std::string::npos) str = str.substr(pos+1);
                        }
+               list = g_list_append(list, utf8_validate_or_convert(str.c_str()));
+#if EXIV2_TEST_VERSION(0,16,0)
+               }
+       else
+               {
+               /* read as a list */
+               gint i;
+               for (i = 0; i < item.count(); i++)
+                       list = g_list_append(list, utf8_validate_or_convert(item.toString(i).c_str()));
                }
+#endif
+       return list;
+}
+
+static GList *exif_get_metadata_simple(ExifData *exif, const gchar *key, MetadataFormat format)
+{
+       GList *list = NULL;
+       try {
+               try {
+                       Exiv2::ExifKey ekey(key);
+                       Exiv2::ExifData::iterator pos = exif->exifData().findKey(ekey);
+                       if (pos != exif->exifData().end())
+                               list = exif_add_value_to_glist(list, *pos, format, &exif->exifData());
+
+               }
+               catch (Exiv2::AnyError& e) {
+                       try {
+                               Exiv2::IptcKey ekey(key);
+                               Exiv2::IptcData::iterator pos = exif->iptcData().begin();
+                               while (pos != exif->iptcData().end())
+                                       {
+                                       if (pos->key() == key)
+                                               list = exif_add_value_to_glist(list, *pos, format, NULL);
+                                       ++pos;
+                                       }
+
+                       }
+                       catch (Exiv2::AnyError& e) {
 #if EXIV2_TEST_VERSION(0,16,0)
-               for (Exiv2::XmpData::iterator i = exif->xmpData().begin(); i != exif->xmpData().end(); ++i) {
-                       if (((Exiv2::Metadatum *)item) == &*i) {
-                               i = exif->xmpData().erase(i);
-                               return 1;
+                               Exiv2::XmpKey ekey(key);
+                               Exiv2::XmpData::iterator pos = exif->xmpData().findKey(ekey);
+                               if (pos != exif->xmpData().end())
+                                       list = exif_add_value_to_glist(list, *pos, format, NULL);
+#endif
                        }
                }
-#endif         
-               return 0;
        }
        catch (Exiv2::AnyError& e) {
-               return 0;
+               debug_exception(e);
        }
+       return list;
 }
 
+GList *exif_get_metadata(ExifData *exif, const gchar *key, MetadataFormat format)
+{
+       GList *list = NULL;
+
+       if (!key) return NULL;
+
+       if (format == METADATA_FORMATTED)
+               {
+               gchar *text;
+               gint key_valid;
+               text = exif_get_formatted_by_key(exif, key, &key_valid);
+               if (key_valid) return g_list_append(NULL, text);
+               }
+
+       list = exif_get_metadata_simple(exif, key, format);
+
+       /* the following code can be ifdefed out as soon as Exiv2 supports it */
+       if (!list)
+               {
+               const AltKey *alt_key = find_alt_key(key);
+               if (alt_key && alt_key->iptc_key)
+                       list = exif_get_metadata_simple(exif, alt_key->iptc_key, format);
+
+#if !EXIV2_TEST_VERSION(0,17,0)
+               /* with older Exiv2 versions exif is not synced */
+               if (!list && alt_key && alt_key->exif_key)
+                       list = exif_get_metadata_simple(exif, alt_key->exif_key, format);
+#endif
+               }
+       return list;
+}
+
+
 void exif_add_jpeg_color_profile(ExifData *exif, unsigned char *cp_data, guint cp_length)
 {
-       if (exif->cp_data) g_free(exif->cp_data);
-       exif->cp_data = cp_data;
-       exif->cp_length =cp_length;
+       exif->add_jpeg_color_profile(cp_data, cp_length);
 }
 
-unsigned char *exif_get_color_profile(ExifData *exif, guint *data_len)
+guchar *exif_get_color_profile(ExifData *exif, guint *data_len)
 {
-       if (exif->cp_data)
-               {
-               if (data_len) *data_len = exif->cp_length;
-               return (unsigned char *) g_memdup(exif->cp_data, exif->cp_length);
-               }
+       guchar *ret = exif->get_jpeg_color_profile(data_len);
+       if (ret) return ret;
+
        ExifItem *prof_item = exif_get_item(exif, "Exif.Image.InterColorProfile");
        if (prof_item && exif_item_get_format_id(prof_item) == EXIF_FORMAT_UNDEFINED)
-               return (unsigned char *) exif_item_get_data(prof_item, data_len);
-       return NULL;
+               ret = (guchar *)exif_item_get_data(prof_item, data_len);
+       return ret;
 }
 
+gchar* exif_get_image_comment(FileData* fd)
+{
+       if (!fd || !fd->exif)
+               return g_strdup("");
+
+       return g_strdup(fd->exif->image_comment().c_str());
+}
+
+void exif_set_image_comment(FileData* fd, const gchar* comment)
+{
+       if (!fd || !fd->exif)
+               return;
+
+       fd->exif->set_image_comment(comment ? comment : "");
+}
+
+
+#if EXIV2_TEST_VERSION(0,17,90)
+
+guchar *exif_get_preview(ExifData *exif, guint *data_len, gint requested_width, gint requested_height)
+{
+       if (!exif) return NULL;
+
+       if (!exif->image()) return NULL;
 
+       std::string const path = exif->image()->io().path();
+       /* given image pathname, first do simple (and fast) file extension test */
+       gboolean is_raw = filter_file_class(path.c_str(), FORMAT_CLASS_RAWIMAGE);
+
+       if (!is_raw && requested_width == 0) return NULL;
+
+       try {
+
+               Exiv2::PreviewManager pm(*exif->image());
+
+               Exiv2::PreviewPropertiesList list = pm.getPreviewProperties();
+
+               if (!list.empty())
+                       {
+                       Exiv2::PreviewPropertiesList::iterator pos;
+                       Exiv2::PreviewPropertiesList::iterator last = --list.end();
+
+                       if (requested_width == 0)
+                               {
+                               pos = last; // the largest
+                               }
+                       else
+                               {
+                               pos = list.begin();
+                               while (pos != last)
+                                       {
+                                       if (pos->width_ >= (uint32_t)requested_width &&
+                                           pos->height_ >= (uint32_t)requested_height) break;
+                                       ++pos;
+                                       }
+
+                               // we are not interested in smaller thumbnails in normal image formats - we can use full image instead
+                               if (!is_raw)
+                                       {
+                                       if (pos->width_ < (uint32_t)requested_width || pos->height_ < (uint32_t)requested_height) return NULL;
+                                       }
+                               }
+
+                       Exiv2::PreviewImage image = pm.getPreviewImage(*pos);
+
+                       Exiv2::DataBuf buf = image.copy();
+                       std::pair<Exiv2::byte*, long> p = buf.release();
+
+                       *data_len = p.second;
+                       return p.first;
+                       }
+               return NULL;
+       }
+       catch (Exiv2::AnyError& e) {
+               debug_exception(e);
+               return NULL;
+       }
+}
 
+void exif_free_preview(guchar *buf)
+{
+       delete[] (Exiv2::byte*)buf;
 }
+#endif
 
-/* This is a dirty hack to support raw file preview, bassed on 
+}
+#if !EXIV2_TEST_VERSION(0,17,90)
+
+/* This is a dirty hack to support raw file preview, bassed on
 tiffparse.cpp from Exiv2 examples */
 
 class RawFile {
        public:
-    
-       RawFile(int fd);
+
+       RawFile(Exiv2::BasicIo &io);
        ~RawFile();
-    
+
        const Exiv2::Value *find(uint16_t tag, uint16_t group);
-    
+
        unsigned long preview_offset();
-    
+
        private:
        int type;
        Exiv2::TiffComponent::AutoPtr rootDir;
-       Exiv2::byte *map_data;
+       Exiv2::BasicIo &io_;
+       const Exiv2::byte *map_data;
        size_t map_len;
        unsigned long offset;
 };
 
+typedef struct _UnmapData UnmapData;
+struct _UnmapData
+{
+       guchar *ptr;
+       guchar *map_data;
+       size_t map_len;
+};
+
+static GList *exif_unmap_list = 0;
+
+extern "C" guchar *exif_get_preview(ExifData *exif, guint *data_len, gint requested_width, gint requested_height)
+{
+       unsigned long offset;
+
+       if (!exif) return NULL;
+       if (!exif->image()) return NULL;
+
+       std::string const path = exif->image()->io().path();
+
+       /* given image pathname, first do simple (and fast) file extension test */
+       if (!filter_file_class(path.c_str(), FORMAT_CLASS_RAWIMAGE)) return NULL;
+
+       try {
+               struct stat st;
+               guchar *map_data;
+               size_t map_len;
+               UnmapData *ud;
+               int fd;
+
+               RawFile rf(exif->image()->io());
+               offset = rf.preview_offset();
+               DEBUG_1("%s: offset %lu", path.c_str(), offset);
+
+               fd = open(path.c_str(), O_RDONLY);
+               if (fd == -1)
+                       {
+                       return NULL;
+                       }
+
+               if (fstat(fd, &st) == -1)
+                       {
+                       close(fd);
+                       return NULL;
+                       }
+               map_len = st.st_size;
+               map_data = (guchar *) mmap(0, map_len, PROT_READ, MAP_PRIVATE, fd, 0);
+               close(fd);
+               if (map_data == MAP_FAILED)
+                       {
+                       return NULL;
+                       }
+               *data_len = map_len - offset;
+               ud = g_new(UnmapData, 1);
+               ud->ptr = map_data + offset;
+               ud->map_data = map_data;
+               ud->map_len = map_len;
+
+               exif_unmap_list = g_list_prepend(exif_unmap_list, ud);
+               return ud->ptr;
+
+       }
+       catch (Exiv2::AnyError& e) {
+               debug_exception(e);
+       }
+       return NULL;
+
+}
+
+void exif_free_preview(guchar *buf)
+{
+       GList *work = exif_unmap_list;
+
+       while (work)
+               {
+               UnmapData *ud = (UnmapData *)work->data;
+               if (ud->ptr == buf)
+                       {
+                       munmap(ud->map_data, ud->map_len);
+                       exif_unmap_list = g_list_remove_link(exif_unmap_list, work);
+                       g_free(ud);
+                       return;
+                       }
+               work = work->next;
+               }
+       g_assert_not_reached();
+}
+
 using namespace Exiv2;
 
-RawFile::RawFile(int fd) : map_data(NULL), map_len(0), offset(0)
+RawFile::RawFile(BasicIo &io) : io_(io), map_data(NULL), map_len(0), offset(0)
 {
+/*
        struct stat st;
        if (fstat(fd, &st) == -1)
                {
@@ -641,6 +1371,15 @@ RawFile::RawFile(int fd) : map_data(NULL), map_len(0), offset(0)
                {
                throw Error(14);
                }
+*/
+        if (io.open() != 0) {
+            throw Error(9, io.path(), strError());
+        }
+
+        map_data = io.mmap();
+        map_len = io.size();
+
+
        type = Exiv2::ImageFactory::getType(map_data, map_len);
 
 #if EXIV2_TEST_VERSION(0,16,0)
@@ -664,8 +1403,8 @@ RawFile::RawFile(int fd) : map_data(NULL), map_len(0), offset(0)
 #endif
 #if EXIV2_TEST_VERSION(0,13,0)
                case Exiv2::ImageType::raf:
-                       if (map_len < 84 + 4) throw Error(14);
-                       offset = getULong(map_data + 84, bigEndian);
+                       if (map_len < 84 + 4) throw Error(14);
+                       offset = getULong(map_data + 84, bigEndian);
                        return;
 #endif
                case Exiv2::ImageType::crw:
@@ -673,7 +1412,7 @@ RawFile::RawFile(int fd) : map_data(NULL), map_len(0), offset(0)
                        // Parse the image, starting with a CIFF header component
                        Exiv2::CiffHeader::AutoPtr parseTree(new Exiv2::CiffHeader);
                        parseTree->read(map_data, map_len);
-                       CiffComponent *entry = parseTree->findComponent(0x2007, 0); 
+                       CiffComponent *entry = parseTree->findComponent(0x2007, 0);
                        if (entry) offset =  entry->pData() - map_data;
                        return;
                        }
@@ -688,9 +1427,9 @@ RawFile::RawFile(int fd) : map_data(NULL), map_len(0), offset(0)
 
        rootDir = createFct(Tag::root, Group::none);
        if (0 == rootDir.get()) {
-               throw Error(1, "No root element defined in TIFF structure");
+               throw Error(1, "No root element defined in TIFF structure");
        }
-       
+
        if (tiffHeader)
                {
                if (!tiffHeader->read(map_data, map_len)) throw Error(3, "TIFF");
@@ -700,33 +1439,31 @@ RawFile::RawFile(int fd) : map_data(NULL), map_len(0), offset(0)
                rootDir->setStart(map_data + tiffHeader->ifdOffset());
 #endif
                }
-               
+
        if (cr2Header)
                {
                rootDir->setStart(map_data + cr2Header->offset());
                }
-       
+
        TiffRwState::AutoPtr state(new TiffRwState(tiffHeader ? tiffHeader->byteOrder() : littleEndian, 0, createFct));
 
        TiffReader reader(map_data,
-                      map_len,
-                      rootDir.get(),
-                      state);
+                         map_len,
+                         rootDir.get(),
+                         state);
 
        rootDir->accept(reader);
-       
-       if (tiffHeader) 
+
+       if (tiffHeader)
                delete tiffHeader;
-       if (cr2Header) 
+       if (cr2Header)
                delete cr2Header;
 }
 
 RawFile::~RawFile(void)
 {
-       if (map_data && munmap(map_data, map_len) == -1)
-               {
-               log_printf("Failed to unmap file \n");
-               }
+       io_.munmap();
+       io_.close();
 }
 
 const Value * RawFile::find(uint16_t tag, uint16_t group)
@@ -747,21 +1484,21 @@ unsigned long RawFile::preview_offset(void)
 {
        const Value *val;
        if (offset) return offset;
-       
+
        if (type == Exiv2::ImageType::cr2)
                {
                val = find(0x111, Group::ifd0);
                if (val) return val->toLong();
-    
+
                return 0;
                }
-       
+
        val = find(0x201, Group::sub0_0);
        if (val) return val->toLong();
 
        val = find(0x201, Group::ifd0);
        if (val) return val->toLong();
-    
+
        val = find(0x201, Group::ignr); // for PEF files, originally it was probably ifd2
        if (val) return val->toLong();
 
@@ -772,42 +1509,9 @@ unsigned long RawFile::preview_offset(void)
 }
 
 
-extern "C" gint format_raw_img_exif_offsets_fd(int fd, const gchar *path,
-                                   unsigned char *header_data, const guint header_len,
-                                   guint *image_offset, guint *exif_offset)
-{
-       int success;
-       unsigned long offset;
-
-       /* given image pathname, first do simple (and fast) file extension test */
-       if (!filter_file_class(path, FORMAT_CLASS_RAWIMAGE)) return 0;
-
-       try {
-               RawFile rf(fd);
-               offset = rf.preview_offset();
-               DEBUG_1("%s: offset %lu", path, offset);
-       }
-       catch (Exiv2::AnyError& e) {
-               std::cout << "Caught Exiv2 exception '" << e << "'\n";
-               return 0;
-       }
-
-       if (image_offset && offset > 0)
-               {
-               *image_offset = offset;
-               if ((unsigned long) lseek(fd, *image_offset, SEEK_SET) != *image_offset)
-                       {
-                       log_printf("Failed to seek to embedded image\n");
-
-                       *image_offset = 0;
-                       if (*exif_offset) *exif_offset = 0;
-                       success = FALSE;
-                       }
-               }
-
-       return offset > 0;
-}
+#endif
 
 
-#endif 
+#endif
 /* HAVE_EXIV2 */
+/* vim: set shiftwidth=8 softtabstop=0 cindent cinoptions={1s: */