* 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
*/
-#include "config.h"
-
-#ifdef HAVE_EXIV2
+#include "exif.h"
-#include <exiv2/image.hpp>
-#include <exiv2/exif.hpp>
-#include <iostream>
+#include <algorithm>
+#include <cstdint>
+#include <cstring>
+#include <exception>
+#include <list>
+#include <memory>
#include <string>
+#include <utility>
+#include <vector>
-// EXIV2_TEST_VERSION is defined in Exiv2 0.15 and newer.
-#ifndef EXIV2_TEST_VERSION
-# define EXIV2_TEST_VERSION(major,minor,patch) \
- ( EXIV2_VERSION >= EXIV2_MAKE_VERSION(major,minor,patch) )
-#endif
-
-
-#include <sys/types.h>
-#include <sys/stat.h>
-#include <unistd.h>
-#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/tiffimage.hpp>
-#include <exiv2/cr2image.hpp>
-#include <exiv2/crwimage.hpp>
-#if EXIV2_TEST_VERSION(0,16,0)
-#include <exiv2/orfimage.hpp>
-#endif
-#if EXIV2_TEST_VERSION(0,13,0)
-#include <exiv2/rafimage.hpp>
-#endif
-#include <exiv2/futils.hpp>
-#else
-#include <exiv2/preview.hpp>
-#endif
+#include <config.h>
-#if EXIV2_TEST_VERSION(0,17,0)
-#include <exiv2/convert.hpp>
-#include <exiv2/xmpsidecar.hpp>
+#include <exiv2/exiv2.hpp>
+#include <glib.h>
+#ifdef ENABLE_NLS
+# include <libintl.h>
#endif
-extern "C" {
-#include <glib.h>
+#include "debug.h"
+#include "filedata.h"
+#include "filefilter.h"
+#include "misc.h"
+#include "options.h"
+#include "typedefs.h"
+#include "ui-fileops.h"
-#include "main.h"
-#include "exif.h"
+struct ExifItem;
-#include "filefilter.h"
-#include "ui_fileops.h"
+#if EXIV2_TEST_VERSION(0,27,0)
+#define HAVE_EXIV2_ERROR_CODE
+#endif
-#include "misc.h"
-}
+#if EXIV2_TEST_VERSION(0,27,0)
+#define EXV_PACKAGE "exiv2"
+#endif
-typedef struct _AltKey AltKey;
+#if EXIV2_TEST_VERSION(0,28,0)
+#define AnyError Error
+#define AutoPtr UniquePtr
+#endif
-struct _AltKey
+struct AltKey
{
const gchar *xmp_key;
const gchar *exif_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 constexpr AltKey alt_keys[] = {
+ {"Xmp.tiff.Orientation", "Exif.Image.Orientation", nullptr},
+ {"Xmp.dc.title", nullptr, "Iptc.Application2.ObjectName" },
+ {"Xmp.photoshop.Urgency", nullptr, "Iptc.Application2.Urgency" },
+ {"Xmp.photoshop.Category", nullptr, "Iptc.Application2.Category" },
+ {"Xmp.photoshop.SupplementalCategory", nullptr, "Iptc.Application2.SuppCategory" },
+ {"Xmp.dc.subject", nullptr, "Iptc.Application2.Keywords" },
+ {"Xmp.iptc.Location", nullptr, "Iptc.Application2.LocationName" },
+ {"Xmp.photoshop.Instruction", nullptr, "Iptc.Application2.SpecialInstructions" },
+ {"Xmp.photoshop.DateCreated", nullptr, "Iptc.Application2.DateCreated" },
+ {"Xmp.dc.creator", nullptr, "Iptc.Application2.Byline" },
+ {"Xmp.photoshop.AuthorsPosition", nullptr, "Iptc.Application2.BylineTitle" },
+ {"Xmp.photoshop.City", nullptr, "Iptc.Application2.City" },
+ {"Xmp.photoshop.State", nullptr, "Iptc.Application2.ProvinceState" },
+ {"Xmp.iptc.CountryCode", nullptr, "Iptc.Application2.CountryCode" },
+ {"Xmp.photoshop.Country", nullptr, "Iptc.Application2.CountryName" },
+ {"Xmp.photoshop.TransmissionReference", nullptr, "Iptc.Application2.TransmissionReference"},
+ {"Xmp.photoshop.Headline", nullptr, "Iptc.Application2.Headline" },
+ {"Xmp.photoshop.Credit", nullptr, "Iptc.Application2.Credit" },
+ {"Xmp.photoshop.Source", nullptr, "Iptc.Application2.Source" },
+ {"Xmp.dc.rights", nullptr, "Iptc.Application2.Copyright" },
+ {"Xmp.dc.description", nullptr, "Iptc.Application2.Caption" },
+ {"Xmp.photoshop.CaptionWriter", nullptr, "Iptc.Application2.Writer" },
};
static void _debug_exception(const char* file,
const char* func,
Exiv2::AnyError& e)
{
- gchar *str = g_locale_from_utf8(e.what(), -1, NULL, NULL, NULL);
+ gchar *str = g_locale_from_utf8(e.what(), -1, nullptr, nullptr, nullptr);
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
+struct ExifData
{
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
- virtual ~_ExifData()
- {
- }
+ virtual ~ExifData() = default;
- virtual void writeMetadata(gchar *path = NULL)
+ virtual void writeMetadata(gchar * = nullptr)
{
g_critical("Unsupported method of writing metadata");
}
virtual ExifData *original()
{
- return NULL;
+ return nullptr;
}
virtual Exiv2::Image *image() = 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
+struct ExifDataOriginal : public ExifData
{
protected:
Exiv2::Image::AutoPtr image_;
Exiv2::ExifData emptyExifData_;
Exiv2::IptcData emptyIptcData_;
-#if EXIV2_TEST_VERSION(0,16,0)
Exiv2::XmpData emptyXmpData_;
-#endif
public:
- _ExifDataOriginal(Exiv2::Image::AutoPtr image)
+ ExifDataOriginal(Exiv2::Image::AutoPtr image)
{
- cp_data_ = NULL;
+ cp_data_ = nullptr;
cp_length_ = 0;
- image_ = image;
+ image_ = std::move(image);
valid_ = TRUE;
}
- _ExifDataOriginal(gchar *path)
+ ExifDataOriginal(gchar *path)
{
- cp_data_ = NULL;
+ cp_data_ = nullptr;
cp_length_ = 0;
valid_ = TRUE;
try
{
image_ = Exiv2::ImageFactory::open(pathl_);
-// g_assert (image.get() != 0);
image_->readMetadata();
-#if EXIV2_TEST_VERSION(0,16,0)
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 */
if (!open) io.open();
if (io.isopen())
{
- unsigned char *mapped = (unsigned char*)io.mmap();
+ auto mapped = static_cast<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)
{
}
}
- virtual ~_ExifDataOriginal()
+ ~ExifDataOriginal() override
{
if (cp_data_) g_free(cp_data_);
if (pathl_) g_free(pathl_);
}
- virtual Exiv2::Image *image()
+ Exiv2::Image *image() override
{
- if (!valid_) return NULL;
+ if (!valid_) return nullptr;
return image_.get();
}
- virtual Exiv2::ExifData &exifData ()
+ Exiv2::ExifData &exifData () override
{
if (!valid_) return emptyExifData_;
return image_->exifData();
}
- virtual Exiv2::IptcData &iptcData ()
+ Exiv2::IptcData &iptcData () override
{
if (!valid_) return emptyIptcData_;
return image_->iptcData();
}
-#if EXIV2_TEST_VERSION(0,16,0)
- virtual Exiv2::XmpData &xmpData ()
+ Exiv2::XmpData &xmpData () override
{
if (!valid_) return emptyXmpData_;
return image_->xmpData();
}
-#endif
- virtual void add_jpeg_color_profile(unsigned char *cp_data, guint cp_length)
+ void add_jpeg_color_profile(unsigned char *cp_data, guint cp_length) override
{
if (cp_data_) g_free(cp_data_);
cp_data_ = cp_data;
cp_length_ = cp_length;
}
- virtual guchar *get_jpeg_color_profile(guint *data_len)
+ guchar *get_jpeg_color_profile(guint *data_len) override
{
if (cp_data_)
{
if (data_len) *data_len = cp_length_;
- return (unsigned char *) g_memdup(cp_data_, cp_length_);
+#if GLIB_CHECK_VERSION(2,68,0)
+ return static_cast<unsigned char *>(g_memdup2(cp_data_, cp_length_));
+#else
+ return static_cast<unsigned char *>(g_memdup(cp_data_, cp_length_));
+#endif
}
- return NULL;
+ return nullptr;
+ }
+
+ std::string image_comment() const override
+ {
+ return image_.get() ? image_->comment() : "";
+ }
+
+ void set_image_comment(const std::string& comment) override
+ {
+ 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
+struct ExifDataProcessed : public ExifData
{
protected:
- _ExifDataOriginal *imageData_;
- _ExifDataOriginal *sidecarData_;
+ std::unique_ptr<ExifDataOriginal> imageData_;
+ std::unique_ptr<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)
+ ExifDataProcessed(gchar *path, gchar *sidecar_path, GHashTable *modified_xmp)
{
- imageData_ = new _ExifDataOriginal(path);
- sidecarData_ = NULL;
-#if EXIV2_TEST_VERSION(0,16,0)
+ imageData_ = std::make_unique<ExifDataOriginal>(path);
+ sidecarData_ = nullptr;
if (sidecar_path)
{
- sidecarData_ = new _ExifDataOriginal(sidecar_path);
+ sidecarData_ = std::make_unique<ExifDataOriginal>(sidecar_path);
xmpData_ = sidecarData_->xmpData();
}
else
xmpData_ = imageData_->xmpData();
}
-#endif
exifData_ = imageData_->exifData();
iptcData_ = imageData_->iptcData();
-#if EXIV2_TEST_VERSION(0,17,0)
try
{
syncExifWithXmp(exifData_, xmpData_);
{
DEBUG_1("Exiv2: Catching bug\n");
}
-#endif
if (modified_xmp)
{
g_hash_table_foreach(modified_xmp, _ExifDataProcessed_update_xmp, this);
}
}
- virtual ~_ExifDataProcessed()
+ ExifData *original() override
{
- if (imageData_) delete imageData_;
- if (sidecarData_) delete sidecarData_;
+ return imageData_.get();
}
- virtual ExifData *original()
- {
- return imageData_;
- }
-
- virtual void writeMetadata(gchar *path = NULL)
+ void writeMetadata(gchar *path = nullptr) override
{
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);
+ if (!image)
+#ifdef HAVE_EXIV2_ERROR_CODE
+# if EXIV2_TEST_VERSION(0,28,0)
+ throw Exiv2::Error(Exiv2::ErrorCode::kerInputDataReadFailed);
+# else
+ throw Exiv2::Error(Exiv2::kerInputDataReadFailed);
+# endif
+#else
+ throw Exiv2::Error(21);
+#endif
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);
+ auto sidecar = Exiv2::ImageFactory::create(Exiv2::ImageType::xmp, pathl);
g_free(pathl);
sidecar->setXmpData(xmpData_);
sidecar->writeMetadata();
-#else
- throw Exiv2::Error(3, "xmp");
-#endif
}
}
- virtual Exiv2::Image *image()
+ Exiv2::Image *image() override
{
return imageData_->image();
}
- virtual Exiv2::ExifData &exifData ()
+ Exiv2::ExifData &exifData () override
{
return exifData_;
}
- virtual Exiv2::IptcData &iptcData ()
+ Exiv2::IptcData &iptcData () override
{
return iptcData_;
}
-#if EXIV2_TEST_VERSION(0,16,0)
- virtual Exiv2::XmpData &xmpData ()
+ Exiv2::XmpData &xmpData () override
{
return xmpData_;
}
-#endif
- virtual void add_jpeg_color_profile(unsigned char *cp_data, guint cp_length)
+ void add_jpeg_color_profile(unsigned char *cp_data, guint cp_length) override
{
imageData_->add_jpeg_color_profile(cp_data, cp_length);
}
- virtual guchar *get_jpeg_color_profile(guint *data_len)
+ guchar *get_jpeg_color_profile(guint *data_len) override
{
return imageData_->get_jpeg_color_profile(data_len);
}
+
+ std::string image_comment() const override
+ {
+ return imageData_->image_comment();
+ }
+
+ void set_image_comment(const std::string& comment) override
+ {
+ imageData_->set_image_comment(comment);
+ }
};
-extern "C" {
-void exif_init(void)
+void exif_init()
{
#ifdef EXV_ENABLE_NLS
bind_textdomain_codeset (EXV_PACKAGE, "UTF-8");
#endif
+
+#ifdef EXV_ENABLE_BMFF
+ Exiv2::enableBMFF(TRUE);
+#endif
}
static void _ExifDataProcessed_update_xmp(gpointer key, gpointer value, gpointer data)
{
- exif_update_metadata((ExifData *)data, (gchar *)key, (GList *)value);
+ exif_update_metadata(static_cast<ExifData *>(data), static_cast<gchar *>(key), static_cast<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 _ExifDataProcessed(path, sidecar_path, modified_xmp);
+ return new ExifDataProcessed(path, sidecar_path, modified_xmp);
}
catch (Exiv2::AnyError& e) {
debug_exception(e);
- return NULL;
+ return nullptr;
}
}
void exif_free(ExifData *exif)
{
if (!exif) return;
- g_assert(dynamic_cast<_ExifDataProcessed *>(exif)); // this should not be called on ExifDataOriginal
+ g_assert(dynamic_cast<ExifDataProcessed *>(exif)); // this should not be called on ExifDataOriginal
delete exif;
}
ExifItem *exif_get_item(ExifData *exif, const gchar *key)
{
try {
- Exiv2::Metadatum *item = NULL;
+ Exiv2::Metadatum *item = nullptr;
try {
Exiv2::ExifKey ekey(key);
- Exiv2::ExifData::iterator pos = exif->exifData().findKey(ekey);
- if (pos == exif->exifData().end()) return NULL;
+ auto pos = exif->exifData().findKey(ekey);
+ if (pos == exif->exifData().end()) return nullptr;
item = &*pos;
}
catch (Exiv2::AnyError& e) {
try {
Exiv2::IptcKey ekey(key);
- Exiv2::IptcData::iterator pos = exif->iptcData().findKey(ekey);
- if (pos == exif->iptcData().end()) return NULL;
+ auto pos = exif->iptcData().findKey(ekey);
+ if (pos == exif->iptcData().end()) return nullptr;
item = &*pos;
}
catch (Exiv2::AnyError& e) {
-#if EXIV2_TEST_VERSION(0,16,0)
Exiv2::XmpKey ekey(key);
- Exiv2::XmpData::iterator pos = exif->xmpData().findKey(ekey);
- if (pos == exif->xmpData().end()) return NULL;
+ auto pos = exif->xmpData().findKey(ekey);
+ if (pos == exif->xmpData().end()) return nullptr;
item = &*pos;
-#endif
}
}
- return (ExifItem *)item;
+ return reinterpret_cast<ExifItem *>(item);
}
catch (Exiv2::AnyError& e) {
debug_exception(e);
- return NULL;
+ return nullptr;
}
}
ExifItem *exif_add_item(ExifData *exif, const gchar *key)
{
try {
- Exiv2::Metadatum *item = NULL;
+ Exiv2::Metadatum *item = nullptr;
try {
Exiv2::ExifKey ekey(key);
- exif->exifData().add(ekey, NULL);
- Exiv2::ExifData::iterator pos = exif->exifData().end(); // a hack, there should be a better way to get the currently added item
+ exif->exifData().add(ekey, nullptr);
+ auto pos = exif->exifData().end(); // a hack, there should be a better way to get the currently added item
pos--;
item = &*pos;
}
catch (Exiv2::AnyError& e) {
try {
Exiv2::IptcKey ekey(key);
- exif->iptcData().add(ekey, NULL);
- Exiv2::IptcData::iterator pos = exif->iptcData().end();
+ exif->iptcData().add(ekey, nullptr);
+ auto pos = exif->iptcData().end();
pos--;
item = &*pos;
}
catch (Exiv2::AnyError& e) {
-#if EXIV2_TEST_VERSION(0,16,0)
Exiv2::XmpKey ekey(key);
- exif->xmpData().add(ekey, NULL);
- Exiv2::XmpData::iterator pos = exif->xmpData().end();
+ exif->xmpData().add(ekey, nullptr);
+ auto pos = exif->xmpData().end();
pos--;
item = &*pos;
-#endif
}
}
- return (ExifItem *)item;
+ return reinterpret_cast<ExifItem *>(item);
}
catch (Exiv2::AnyError& e) {
debug_exception(e);
- return NULL;
+ return nullptr;
}
}
try {
exif->exifIter = exif->exifData().begin();
exif->iptcIter = exif->iptcData().begin();
-#if EXIV2_TEST_VERSION(0,16,0)
exif->xmpIter = exif->xmpData().begin();
-#endif
if (exif->exifIter != exif->exifData().end())
{
const Exiv2::Metadatum *item = &*exif->exifIter;
exif->iptcIter++;
return (ExifItem *)item;
}
-#if EXIV2_TEST_VERSION(0,16,0)
if (exif->xmpIter != exif->xmpData().end())
{
const Exiv2::Metadatum *item = &*exif->xmpIter;
exif->xmpIter++;
return (ExifItem *)item;
}
-#endif
- return NULL;
+ return nullptr;
}
catch (Exiv2::AnyError& e) {
debug_exception(e);
- return NULL;
+ return nullptr;
}
}
exif->iptcIter++;
return (ExifItem *)item;
}
-#if EXIV2_TEST_VERSION(0,16,0)
if (exif->xmpIter != exif->xmpData().end())
{
const Exiv2::Metadatum *item = &*exif->xmpIter;
exif->xmpIter++;
return (ExifItem *)item;
}
-#endif
- return NULL;
+ return nullptr;
}
catch (Exiv2::AnyError& e) {
debug_exception(e);
- return NULL;
+ return nullptr;
}
}
char *exif_item_get_tag_name(ExifItem *item)
{
try {
- if (!item) return NULL;
- return g_strdup(((Exiv2::Metadatum *)item)->key().c_str());
+ if (!item) return nullptr;
+ return g_strdup((reinterpret_cast<Exiv2::Metadatum *>(item))->key().c_str());
}
catch (Exiv2::AnyError& e) {
debug_exception(e);
- return NULL;
+ return nullptr;
}
}
{
try {
if (!item) return 0;
- return ((Exiv2::Metadatum *)item)->tag();
+ return (reinterpret_cast<Exiv2::Metadatum *>(item))->tag();
}
catch (Exiv2::AnyError& e) {
debug_exception(e);
{
try {
if (!item) return 0;
- return ((Exiv2::Metadatum *)item)->count();
+ return (reinterpret_cast<Exiv2::Metadatum *>(item))->count();
}
catch (Exiv2::AnyError& e) {
debug_exception(e);
char *exif_item_get_data(ExifItem *item, guint *data_len)
{
try {
- if (!item) return 0;
- Exiv2::Metadatum *md = (Exiv2::Metadatum *)item;
+ if (!item) return nullptr;
+ auto md = reinterpret_cast<Exiv2::Metadatum *>(item);
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 */);
+ auto data = static_cast<char *>(g_malloc(md->size()));
+ auto res = md->copy(reinterpret_cast<Exiv2::byte *>(data), Exiv2::littleEndian /* should not matter */);
g_assert(res == md->size());
return data;
}
catch (Exiv2::AnyError& e) {
debug_exception(e);
- return NULL;
+ return nullptr;
}
}
char *exif_item_get_description(ExifItem *item)
{
try {
- if (!item) return NULL;
- return utf8_validate_or_convert(((Exiv2::Metadatum *)item)->tagLabel().c_str());
+ if (!item) return nullptr;
+ return utf8_validate_or_convert((reinterpret_cast<Exiv2::Metadatum *>(item))->tagLabel().c_str());
}
catch (std::exception& e) {
-// debug_exception(e);
- return NULL;
+ return nullptr;
}
}
{
try {
if (!item) return EXIF_FORMAT_UNKNOWN;
- guint id = ((Exiv2::Metadatum *)item)->typeId();
+ guint id = (reinterpret_cast<Exiv2::Metadatum *>(item))->typeId();
if (id >= (sizeof(format_id_trans_tbl) / sizeof(format_id_trans_tbl[0])) ) return EXIF_FORMAT_UNKNOWN;
return format_id_trans_tbl[id];
}
}
}
-const char *exif_item_get_format_name(ExifItem *item, gboolean brief)
+const char *exif_item_get_format_name(ExifItem *item, gboolean)
{
try {
- if (!item) return NULL;
- return ((Exiv2::Metadatum *)item)->typeName();
+ if (!item) return nullptr;
+ return (reinterpret_cast<Exiv2::Metadatum *>(item))->typeName();
}
catch (Exiv2::AnyError& e) {
debug_exception(e);
- return NULL;
+ return nullptr;
}
}
-gchar *exif_item_get_data_as_text(ExifItem *item)
+gchar *exif_item_get_data_as_text(ExifItem *item, ExifData *exif)
{
try {
- if (!item) return NULL;
- Exiv2::Metadatum *metadatum = (Exiv2::Metadatum *)item;
-#if EXIV2_TEST_VERSION(0,17,0)
- 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)))
- str << *iptcdatum;
-#if EXIV2_TEST_VERSION(0,16,0)
- else if ((xmpdatum = dynamic_cast<Exiv2::Xmpdatum *>(metadatum)))
- str << *xmpdatum;
-#endif
-
- return utf8_validate_or_convert(str.str().c_str());
-#endif
+ if (!item) return nullptr;
+ auto metadatum = reinterpret_cast<Exiv2::Metadatum *>(item);
+ return utf8_validate_or_convert(metadatum->print(&exif->exifData()).c_str());
}
catch (Exiv2::AnyError& e) {
- return NULL;
+ return nullptr;
}
}
gchar *exif_item_get_string(ExifItem *item, int idx)
{
try {
- if (!item) return NULL;
- Exiv2::Metadatum *em = (Exiv2::Metadatum *)item;
-#if EXIV2_TEST_VERSION(0,16,0)
+ if (!item) return nullptr;
+ auto em = reinterpret_cast<Exiv2::Metadatum *>(item);
std::string str = em->toString(idx);
-#else
- std::string str = em->toString(); // FIXME
-#endif
- if (idx == 0 && str == "") str = em->toString();
+ if (idx == 0 && str.empty()) str = em->toString();
if (str.length() > 5 && str.substr(0, 5) == "lang=")
{
std::string::size_type pos = str.find_first_of(' ');
return utf8_validate_or_convert(str.c_str());
}
catch (Exiv2::AnyError& e) {
- return NULL;
+ return nullptr;
}
}
{
try {
if (!item || exif_item_get_elements(item) == 0) return 0;
- *value = ((Exiv2::Metadatum *)item)->toLong();
+
+#if EXIV2_TEST_VERSION(0,28,0)
+ *value = ((Exiv2::Metadatum *)item)->toInt64();
+#else
+ *value = (reinterpret_cast<Exiv2::Metadatum *>(item))->toLong();
+#endif
return 1;
}
catch (Exiv2::AnyError& e) {
ExifRational *exif_item_get_rational(ExifItem *item, gint *sign, guint n)
{
try {
- if (!item) return NULL;
- if (n >= exif_item_get_elements(item)) return NULL;
- Exiv2::Rational v = ((Exiv2::Metadatum *)item)->toRational(n);
+ if (!item) return nullptr;
+ if (n >= exif_item_get_elements(item)) return nullptr;
+ Exiv2::Rational v = (reinterpret_cast<Exiv2::Metadatum *>(item))->toRational(n);
static ExifRational ret;
ret.num = v.first;
ret.den = v.second;
- if (sign) *sign = (((Exiv2::Metadatum *)item)->typeId() == Exiv2::signedRational);
+ if (sign) *sign = ((reinterpret_cast<Exiv2::Metadatum *>(item))->typeId() == Exiv2::signedRational);
return &ret;
}
catch (Exiv2::AnyError& e) {
debug_exception(e);
- return NULL;
+ return nullptr;
}
}
}
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 nullptr;
}
}
}
- return NULL;
+ return nullptr;
}
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;
+ for (const auto& k : alt_keys)
+ if (strcmp(xmp_key, k.xmp_key) == 0) return &k;
+ return nullptr;
}
static gint exif_update_metadata_simple(ExifData *exif, const gchar *key, const GList *values)
try {
Exiv2::ExifKey ekey(key);
- Exiv2::ExifData::iterator pos = exif->exifData().findKey(ekey);
+ auto pos = exif->exifData().findKey(ekey);
while (pos != exif->exifData().end())
{
exif->exifData().erase(pos);
while (work)
{
- exif->exifData()[key] = (gchar *)work->data;
+ exif->exifData()[key] = static_cast<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);
+ auto pos = exif->iptcData().findKey(ekey);
while (pos != exif->iptcData().end())
{
exif->iptcData().erase(pos);
while (work)
{
- exif->iptcData()[key] = (gchar *)work->data;
+ exif->iptcData()[key] = static_cast<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);
+ auto pos = exif->xmpData().findKey(ekey);
while (pos != exif->xmpData().end())
{
exif->xmpData().erase(pos);
while (work)
{
- exif->xmpData()[key] = (gchar *)work->data;
+ exif->xmpData()[key] = static_cast<gchar *>(work->data);
work = work->next;
}
}
-#endif
}
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 */
+ !ret /* writing to the explicitly given xmp tag failed */
)
{
/* deleted xmp metadatum can't be converted, we have to delete also the corresponding legacy tag */
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::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
+ str = item.print(metadata);
if (str.length() > 1024)
{
/* truncate very long strings, they cause problems in gui */
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;
+#if EXIV2_TEST_VERSION(0,28,0)
+ size_t i;
+#else
+ long i;
+#endif
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;
+ GList *list = nullptr;
try {
try {
Exiv2::ExifKey ekey(key);
- Exiv2::ExifData::iterator pos = exif->exifData().findKey(ekey);
+ auto 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();
+ auto pos = exif->iptcData().begin();
while (pos != exif->iptcData().end())
{
if (pos->key() == key)
- list = exif_add_value_to_glist(list, *pos, format, NULL);
+ list = exif_add_value_to_glist(list, *pos, format, nullptr);
++pos;
}
}
catch (Exiv2::AnyError& e) {
-#if EXIV2_TEST_VERSION(0,16,0)
Exiv2::XmpKey ekey(key);
- Exiv2::XmpData::iterator pos = exif->xmpData().findKey(ekey);
+ auto pos = exif->xmpData().findKey(ekey);
if (pos != exif->xmpData().end())
- list = exif_add_value_to_glist(list, *pos, format, NULL);
-#endif
+ list = exif_add_value_to_glist(list, *pos, format, nullptr);
}
}
}
GList *exif_get_metadata(ExifData *exif, const gchar *key, MetadataFormat format)
{
- GList *list = NULL;
+ GList *list = nullptr;
- if (!key) return NULL;
+ if (!key) return nullptr;
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);
+ if (key_valid) return g_list_append(nullptr, text);
}
list = exif_get_metadata_simple(exif, key, format);
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;
}
ExifItem *prof_item = exif_get_item(exif, "Exif.Image.InterColorProfile");
if (prof_item && exif_item_get_format_id(prof_item) == EXIF_FORMAT_UNDEFINED)
- ret = (guchar *)exif_item_get_data(prof_item, data_len);
+ ret = reinterpret_cast<guchar *>(exif_item_get_data(prof_item, data_len));
return ret;
}
-#if EXIV2_TEST_VERSION(0,17,90)
+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 : "");
+}
+
guchar *exif_get_preview(ExifData *exif, guint *data_len, gint requested_width, gint requested_height)
{
- if (!exif) return NULL;
+ if (!exif) return nullptr;
- if (!exif->image()) return NULL;
+ if (!exif->image()) return nullptr;
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;
+ if (!is_raw && requested_width == 0) return nullptr;
try {
if (!list.empty())
{
Exiv2::PreviewPropertiesList::iterator pos;
- Exiv2::PreviewPropertiesList::iterator last = --list.end();
+ auto last = --list.end();
if (requested_width == 0)
{
pos = list.begin();
while (pos != last)
{
- if (pos->width_ >= (uint32_t)requested_width &&
- pos->height_ >= (uint32_t)requested_height) break;
+ if (pos->width_ >= static_cast<uint32_t>(requested_width) &&
+ pos->height_ >= static_cast<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;
+ if (pos->width_ < static_cast<uint32_t>(requested_width) || pos->height_ < static_cast<uint32_t>(requested_height)) return nullptr;
}
}
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
-
-}
-#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(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::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;
+ // Let's not touch data_len until we finish copy.
+ // Just in case we run into OOM.
+ size_t img_sz = image.size();
+ auto* b = new Exiv2::byte[img_sz];
+ std::copy_n(image.pData(), img_sz, b);
+ *data_len = img_sz;
+ return b;
}
- *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;
-
+ return nullptr;
}
catch (Exiv2::AnyError& e) {
debug_exception(e);
+ return nullptr;
}
- 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(BasicIo &io) : io_(io), map_data(NULL), map_len(0), offset(0)
-{
-/*
- struct stat st;
- if (fstat(fd, &st) == -1)
- {
- throw Error(14);
- }
- map_len = st.st_size;
- map_data = (Exiv2::byte *) mmap(0, map_len, PROT_READ, MAP_PRIVATE, fd, 0);
- if (map_data == MAP_FAILED)
- {
- 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)
- TiffHeaderBase *tiffHeader = NULL;
-#else
- TiffHeade2 *tiffHeader = NULL;
-#endif
- Cr2Header *cr2Header = NULL;
-
- switch (type) {
- case Exiv2::ImageType::tiff:
- tiffHeader = new TiffHeade2();
- break;
- case Exiv2::ImageType::cr2:
- cr2Header = new Cr2Header();
- break;
-#if EXIV2_TEST_VERSION(0,16,0)
- case Exiv2::ImageType::orf:
- tiffHeader = new OrfHeader();
- break;
-#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);
- return;
-#endif
- case Exiv2::ImageType::crw:
- {
- // 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);
- if (entry) offset = entry->pData() - map_data;
- return;
- }
-
- default:
- throw Error(3, "RAW");
- }
-
- // process tiff-like formats
-
- TiffCompFactoryFct createFct = TiffCreator::create;
-
- rootDir = createFct(Tag::root, Group::none);
- if (0 == rootDir.get()) {
- throw Error(1, "No root element defined in TIFF structure");
- }
-
- if (tiffHeader)
- {
- if (!tiffHeader->read(map_data, map_len)) throw Error(3, "TIFF");
-#if EXIV2_TEST_VERSION(0,16,0)
- rootDir->setStart(map_data + tiffHeader->offset());
-#else
- 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);
-
- rootDir->accept(reader);
-
- if (tiffHeader)
- delete tiffHeader;
- if (cr2Header)
- delete cr2Header;
}
-RawFile::~RawFile(void)
+void exif_free_preview(const guchar *buf)
{
- io_.munmap();
- io_.close();
-}
-
-const Value * RawFile::find(uint16_t tag, uint16_t group)
-{
- TiffFinder finder(tag, group);
- rootDir->accept(finder);
- TiffEntryBase* te = dynamic_cast<TiffEntryBase*>(finder.result());
- if (te)
- {
- DEBUG_1("(tag: %04x %04x) ", tag, group);
- return te->pValue();
- }
- else
- return NULL;
+ delete[] static_cast<const Exiv2::byte*>(buf);
}
-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();
-
- val = find(0x111, Group::sub0_1); // dng
- if (val) return val->toLong();
-
- return 0;
-}
-
-
-#endif
-
-
-#endif
-/* HAVE_EXIV2 */
/* vim: set shiftwidth=8 softtabstop=0 cindent cinoptions={1s: */