Only call vips_tile_move() when reusing a tile

kleisauke · kleisauke · commit abbe955f00ed · 2023-07-27T14:57:40.000+02:00
This is not necessary for creating a new tile, since the x/y
position does not change.
diff --git a/libvips/conversion/tilecache.c b/libvips/conversion/tilecache.c
@@ -220,7 +220,7 @@ vips_tile_new(VipsBlockCache *cache, int x, int y)
 
 	vips__region_no_ownership(tile->region);
 
-	if (vips_tile_move(tile, x, y)) {
+	if (vips_region_buffer(tile->region, &tile->pos)) {
 		g_hash_table_remove(cache->tiles, &tile->pos);
 		return NULL;
 	}

Original file line number	Diff line number	Diff line change
`@@ -220,7 +220,7 @@ vips_tile_new(VipsBlockCache *cache, int x, int y)`
`220`	`220`
`221`	`221`	`vips__region_no_ownership(tile->region);`
`222`	`222`
`223`		`- if (vips_tile_move(tile, x, y)) {`
	`223`	`+ if (vips_region_buffer(tile->region, &tile->pos)) {`
`224`	`224`	`g_hash_table_remove(cache->tiles, &tile->pos);`
`225`	`225`	`return NULL;`
`226`	`226`	`}`