Bug 790552 - do not save metadata by default.

This is a privacy concern. Whereas importing metadata is usually a good
idea, exporting it should be a conscious action. A lot of private data
can be leaked through metadata and many people don't realize it (which
also usually means they don't need it). On the other hand, the people
who realize it are the ones who would explicitly edit the metadata and
check what they want to be exported or not.

This is only a first step. Some people may want to always export the
metadata and for these people, there should be abilities to change the
default.
This commit is contained in:
Jehan
2018-01-11 00:39:22 +01:00
parent b118e8242b
commit 096debb0fd
4 changed files with 12 additions and 17 deletions

View File

@ -212,9 +212,9 @@ static const PngSaveVals defaults =
TRUE, TRUE,
TRUE, TRUE,
9, 9,
TRUE, /* save exif */ FALSE, /* save exif */
TRUE, /* save xmp */ FALSE, /* save xmp */
TRUE, /* save iptc */ FALSE, /* save iptc */
TRUE, /* save thumbnail */ TRUE, /* save thumbnail */
PNG_FORMAT_AUTO PNG_FORMAT_AUTO
}; };

View File

@ -64,10 +64,10 @@ static gboolean arithc_supported = FALSE;
#define DEFAULT_RESTART_MCU_ROWS 16 #define DEFAULT_RESTART_MCU_ROWS 16
#define DEFAULT_DCT 0 #define DEFAULT_DCT 0
#define DEFAULT_PREVIEW FALSE #define DEFAULT_PREVIEW FALSE
#define DEFAULT_EXIF TRUE #define DEFAULT_EXIF FALSE
#define DEFAULT_THUMBNAIL FALSE #define DEFAULT_THUMBNAIL FALSE
#define DEFAULT_XMP TRUE #define DEFAULT_XMP FALSE
#define DEFAULT_IPTC TRUE #define DEFAULT_IPTC FALSE
#define DEFAULT_USE_ORIG_QUALITY FALSE #define DEFAULT_USE_ORIG_QUALITY FALSE
#define JPEG_DEFAULTS_PARASITE "jpeg-save-defaults" #define JPEG_DEFAULTS_PARASITE "jpeg-save-defaults"

View File

@ -86,9 +86,9 @@ static TiffSaveVals tsvals =
COMPRESSION_NONE, /* compression */ COMPRESSION_NONE, /* compression */
TRUE, /* alpha handling */ TRUE, /* alpha handling */
TRUE, /* save transp. pixels */ TRUE, /* save transp. pixels */
TRUE, /* save exif */ FALSE, /* save exif */
TRUE, /* save xmp */ FALSE, /* save xmp */
TRUE, /* save iptc */ FALSE, /* save iptc */
TRUE /* save thumbnail */ TRUE /* save thumbnail */
}; };
@ -362,11 +362,6 @@ run (const gchar *name,
"image/tiff", "image/tiff",
&metadata_flags); &metadata_flags);
tsvals.save_exif = (metadata_flags & GIMP_METADATA_SAVE_EXIF) != 0;
tsvals.save_xmp = (metadata_flags & GIMP_METADATA_SAVE_XMP) != 0;
tsvals.save_iptc = (metadata_flags & GIMP_METADATA_SAVE_IPTC) != 0;
tsvals.save_thumbnail = (metadata_flags & GIMP_METADATA_SAVE_THUMBNAIL) != 0;
parasite = gimp_image_get_parasite (orig_image, "gimp-comment"); parasite = gimp_image_get_parasite (orig_image, "gimp-comment");
if (parasite) if (parasite)
{ {

View File

@ -199,9 +199,9 @@ run (const gchar *name,
params.kf_distance = 50; params.kf_distance = 50;
params.quality = 90.0f; params.quality = 90.0f;
params.alpha_quality = 100.0f; params.alpha_quality = 100.0f;
params.exif = TRUE; params.exif = FALSE;
params.iptc = TRUE; params.iptc = FALSE;
params.xmp = TRUE; params.xmp = FALSE;
params.delay = 200; params.delay = 200;
params.force_delay = FALSE; params.force_delay = FALSE;