sergiotarxz/libvips
sergiotarxz/libvips
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

set webp loop even if we strip

Browse Source 
Make sure that webp loop_count is set, even if strip is enabled.

See https://github.com/libvips/libvips/issues/1339
This commit is contained in:
John Cupitt 2019-07-08 08:48:06 +01:00
parent bf9a2adf58
commit 063234fde1
1 changed files with 16 additions and 15 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

31
libvips/foreign/vips2webp.c
View File

@ -14,6 +14,8 @@
* - use libwebpmux instead of our own thing, phew * - use libwebpmux instead of our own thing, phew
* 6/7/19 [deftomat] * 6/7/19 [deftomat]
* - support array of delays * - support array of delays
* 8/7/19
* - set loop even if we strip
*/ */
/* /*
@ -437,15 +439,6 @@ vips_webp_add_chunks( VipsWebPWrite *write, VipsImage *image )
{ {
int i; int i;
if( vips_image_get_typeof( image, "gif-loop" ) ) {
int gif_loop;
if( vips_image_get_int( image, "gif-loop", &gif_loop ) )
return( -1 );
vips_webp_set_count( write, gif_loop );
}
for( i = 0; i < vips__n_webp_names; i++ ) { for( i = 0; i < vips__n_webp_names; i++ ) {
const char *vips_name = vips__webp_names[i].vips; const char *vips_name = vips__webp_names[i].vips;
const char *webp_name = vips__webp_names[i].webp; const char *webp_name = vips__webp_names[i].webp;
@ -466,7 +459,7 @@ vips_webp_add_chunks( VipsWebPWrite *write, VipsImage *image )
} }
static int static int
vips_webp_add_metadata( VipsWebPWrite *write, VipsImage *image ) vips_webp_add_metadata( VipsWebPWrite *write, VipsImage *image, gboolean strip )
{ {
WebPData data; WebPData data;
@ -485,9 +478,19 @@ vips_webp_add_metadata( VipsWebPWrite *write, VipsImage *image )
return( -1 ); return( -1 );
} }
if( vips_image_get_typeof( image, "gif-loop" ) ) {
int gif_loop;
if( vips_image_get_int( image, "gif-loop", &gif_loop ) )
return( -1 );
vips_webp_set_count( write, gif_loop );
}
/* Add extra metadata. /* Add extra metadata.
*/ */
if( vips_webp_add_chunks( write, image ) ) if( !strip &&
vips_webp_add_chunks( write, image ) )
return( -1 ); return( -1 );
if( WebPMuxAssemble( write->mux, &data ) != WEBP_MUX_OK ) { if( WebPMuxAssemble( write->mux, &data ) != WEBP_MUX_OK ) {
@ -525,8 +528,7 @@ vips__webp_write_file( VipsImage *image, const char *filename,
return( -1 ); return( -1 );
} }
if( !strip && if( vips_webp_add_metadata( &write, image, strip ) ) {
vips_webp_add_metadata( &write, image ) ) {
vips_webp_write_unset( &write ); vips_webp_write_unset( &write );
return( -1 ); return( -1 );
} }
@ -570,8 +572,7 @@ vips__webp_write_buffer( VipsImage *image, void **obuf, size_t *olen,
return( -1 ); return( -1 );
} }
if( !strip && if( vips_webp_add_metadata( &write, image, strip ) ) {
vips_webp_add_metadata( &write, image ) ) {
vips_webp_write_unset( &write ); vips_webp_write_unset( &write );
return( -1 ); return( -1 );
} }