diff --git a/ChangeLog b/ChangeLog index 38916702..6ee840d8 100644 --- a/ChangeLog +++ b/ChangeLog @@ -6,7 +6,7 @@ - disable webp alpha output if all frames fill the canvas and are solid - add "compression" option to heifsave [lovell] - support webp and zstd compression in tiff -- use "minimise" to close input early +- loaders use "minimise" to close input files earlier - integrate support for oss-fuzz [omira-sch] 9/7/19 started 8.8.2 diff --git a/libvips/foreign/foreign.c b/libvips/foreign/foreign.c index 6dda604d..a48591ad 100644 --- a/libvips/foreign/foreign.c +++ b/libvips/foreign/foreign.c @@ -838,7 +838,7 @@ vips_foreign_load_start( VipsImage *out, void *a, void *b ) /* Load the image and check the result. * - * ->header() read the header into @out, load has read the + * ->header() read the header into @out, load will read the * image into @real. They must match exactly in size, bands, * format and coding for the copy to work. * diff --git a/libvips/foreign/jpeg2vips.c b/libvips/foreign/jpeg2vips.c index e60dc31e..aacb6918 100644 --- a/libvips/foreign/jpeg2vips.c +++ b/libvips/foreign/jpeg2vips.c @@ -724,7 +724,7 @@ read_jpeg_generate( VipsRegion *or, jpeg->y_pos += 1; } - /* Shut down the input file. + /* Shut down the input early if we can. */ if( jpeg->y_pos >= or->im->Ysize ) readjpeg_close_input( jpeg ); diff --git a/libvips/iofuncs/image.c b/libvips/iofuncs/image.c index 15274c68..211c1727 100644 --- a/libvips/iofuncs/image.c +++ b/libvips/iofuncs/image.c @@ -451,11 +451,9 @@ vips_image_finalize( GObject *gobject ) VIPS_FREE( image->time ); } - /* Any image data? + /* Free attached memory. */ if( image->data ) { - /* Buffer image. Only free stuff we know we allocated. - */ if( image->dtype == VIPS_IMAGE_SETBUF ) { VIPS_DEBUG_MSG( "vips_image_finalize: " "freeing buffer\n" ); @@ -466,7 +464,7 @@ vips_image_finalize( GObject *gobject ) image->data = NULL; } - /* If this is a temp, delete it. + /* Delete associated files. */ vips_image_delete( image ); @@ -2082,9 +2080,8 @@ vips_image_new_from_memory_copy_cb( VipsImage *image, void *data_copy ) * @format: image format * * Like vips_image_new_from_memory(), but VIPS will make a copy of the memory - * area. This - * means more memory use and an extra copy operation, but is much simpler and - * safer. + * area. This means more memory use and an extra copy operation, but is much + * simpler and safer. * * See also: vips_image_new_from_memory(). *