cache: avoid an unnecessary cache entry lookup

libvips/iofuncs/cache.c

@@ -616,10 +616,8 @@ vips_object_ref_arg(VipsObject *object,
 }
 
 static void
-vips_operation_touch(VipsOperation *operation)
+vips_entry_touch(VipsOperationCacheEntry *entry)
 {
-	VipsOperationCacheEntry *entry = vips_cache_operation_get(operation);
-
 	vips_cache_time += 1;
 
 	/* Don't up the time for invalid items -- we want them to fall out of
@@ -629,21 +627,21 @@ vips_operation_touch(VipsOperation *operation)
 		entry->time = vips_cache_time;
 }
 
-/* Ref an operation for the cache. The operation itself, plus all the output
+/* Ref a cache entry. The operation itself, plus all the output
  * objects it makes.
  */
 static void
-vips_cache_ref(VipsOperation *operation)
+vips_entry_ref(VipsOperationCacheEntry *entry)
 {
 #ifdef DEBUG
 	printf("vips_cache_ref: ");
-	vips_object_print_summary(VIPS_OBJECT(operation));
+	vips_object_print_summary(VIPS_OBJECT(entry->operation));
 #endif /*DEBUG*/
 
-	g_object_ref(operation);
-	(void) vips_argument_map(VIPS_OBJECT(operation),
+	g_object_ref(entry->operation);
+	(void) vips_argument_map(VIPS_OBJECT(entry->operation),
 		vips_object_ref_arg, NULL, NULL);
-	vips_operation_touch(operation);
+	vips_entry_touch(entry);
 }
 
 static void
@@ -674,7 +672,7 @@ vips_cache_insert(VipsOperation *operation)
 	entry->invalid = FALSE;
 
 	g_hash_table_insert(vips_cache_table, operation, entry);
-	vips_cache_ref(operation);
+	vips_entry_ref(entry);
 
 	/* If the operation signals "invalidate", we must tag this cache entry
 	 * for removal.
@@ -822,7 +820,7 @@ vips_cache_operation_buildp(VipsOperation **operation)
 	 * passed.
 	 */
 	if (hit) {
-		vips_cache_ref(hit->operation);
+		vips_entry_ref(hit);
 		g_object_unref(*operation);
 		*operation = hit->operation;