removed warning when reusing a color index for transparency. Shouldn't

2006-02-19  Sven Neumann  <sven@gimp.org>

	* plug-ins/common/gif.c (save_image): removed warning when reusing
	a color index for transparency. Shouldn't pose a problem for any
	GIF viewer and only confuses our users.
This commit is contained in:
Sven Neumann 2006-02-19 17:17:03 +00:00 committed by Sven Neumann
parent 3ded19dea8
commit 1b01b29548
2 changed files with 13 additions and 10 deletions

View File

@ -1,3 +1,9 @@
2006-02-19 Sven Neumann <sven@gimp.org>
* plug-ins/common/gif.c (save_image): removed warning when reusing
a color index for transparency. Shouldn't pose a problem for any
GIF viewer and only confuses our users.
2006-02-19 Sven Neumann <sven@gimp.org> 2006-02-19 Sven Neumann <sven@gimp.org>
* plug-ins/common/gauss.c: added some const qualifiers and other * plug-ins/common/gauss.c: added some const qualifiers and other

View File

@ -1096,8 +1096,9 @@ save_image (const gchar *filename,
drawable->width * drawable->height); drawable->width * drawable->height);
} }
else else
transparent = -1; {
transparent = -1;
}
BitsPerPixel = colorstobpp (colors); BitsPerPixel = colorstobpp (colors);
@ -1106,23 +1107,19 @@ save_image (const gchar *filename,
/* We were able to re-use an index within the existing bitspace, /* We were able to re-use an index within the existing bitspace,
whereas the estimate in the header was pessimistic but still whereas the estimate in the header was pessimistic but still
needs to be upheld... */ needs to be upheld... */
#ifdef GIFDEBUG
static gboolean onceonly = FALSE; static gboolean onceonly = FALSE;
if (!onceonly) if (!onceonly)
{ {
#ifdef GIFDEBUG
g_warning ("Promised %d bpp, pondered writing chunk with %d bpp!", g_warning ("Promised %d bpp, pondered writing chunk with %d bpp!",
liberalBPP, BitsPerPixel); liberalBPP, BitsPerPixel);
#endif
g_message (_("Warning:\n"
"Transparent color in written file might be "
"incorrect on viewers which don't support "
"transparency."));
onceonly = TRUE; onceonly = TRUE;
} }
#endif
} }
useBPP = (BitsPerPixel > liberalBPP) ? BitsPerPixel : liberalBPP;
useBPP = (BitsPerPixel > liberalBPP) ? BitsPerPixel : liberalBPP;
if (is_gif89) if (is_gif89)
{ {
@ -1154,7 +1151,7 @@ save_image (const gchar *filename,
if (!onceonly) if (!onceonly)
{ {
g_message (_("Delay inserted to prevent evil " g_message (_("Delay inserted to prevent evil "
"CPU-sucking anim.")); "CPU-sucking animation."));
onceonly = TRUE; onceonly = TRUE;
} }
Delay89 = 1; Delay89 = 1;