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

try disabling operation cache for magickload

Browse Source 
magickload can eat huge amounts of memory and disc, fon't keep it in the
operation cache

see https://github.com/jcupitt/libvips/issues/731
This commit is contained in:
John Cupitt 2017-09-08 04:55:55 +01:00
parent 739389d18d
commit 8970df2e1f
1 changed files with 7 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
libvips/foreign/magickload.c
View File

@ -10,6 +10,8 @@
* - add @page option, 0 by default * - add @page option, 0 by default
* 25/11/16 * 25/11/16
* - add @n, deprecate @all_frames (just sets n = -1) * - add @n, deprecate @all_frames (just sets n = -1)
* 8/9/17
* - don't cache magickload
*/ */
/* /*
@ -95,6 +97,7 @@ vips_foreign_load_magick_class_init( VipsForeignLoadMagickClass *class )
{ {
GObjectClass *gobject_class = G_OBJECT_CLASS( class ); GObjectClass *gobject_class = G_OBJECT_CLASS( class );
VipsObjectClass *object_class = (VipsObjectClass *) class; VipsObjectClass *object_class = (VipsObjectClass *) class;
VipsOperationClass *operation_class = VIPS_OPERATION_CLASS( class );
VipsForeignClass *foreign_class = (VipsForeignClass *) class; VipsForeignClass *foreign_class = (VipsForeignClass *) class;
VipsForeignLoadClass *load_class = (VipsForeignLoadClass *) class; VipsForeignLoadClass *load_class = (VipsForeignLoadClass *) class;
@ -104,6 +107,10 @@ vips_foreign_load_magick_class_init( VipsForeignLoadMagickClass *class )
object_class->nickname = "magickload_base"; object_class->nickname = "magickload_base";
object_class->description = _( "load with ImageMagick" ); object_class->description = _( "load with ImageMagick" );
/* Don't cache magickload: it can gobble up memory and disc.
*/
operation_class->flags = VIPS_OPERATION_NOCACHE;
/* We need to be well to the back of the queue since vips's /* We need to be well to the back of the queue since vips's
* dedicated loaders are usually preferable. * dedicated loaders are usually preferable.
*/ */