[hash] Return lookup entry.

Use the return value to return the result from _cairo_hash_table_lookup()
(as opposed to filling an output parameter on the stack) as this (a)
results in cleaner code (no strict-alias breaking pointer casts), (b)
produces a smaller binary and (c) is measurably faster.

1 files changed, 3 insertions, 4 deletions

diff --git a/src/cairo-hash-private.h b/src/cairo-hash-private.h
index 9101f2ed7..a0be097a8 100644
--- a/src/cairo-hash-private.h
+++ b/src/cairo-hash-private.h
@@ -63,10 +63,9 @@ _cairo_hash_table_create (cairo_hash_keys_equal_func_t keys_equal);
 cairo_private void
 _cairo_hash_table_destroy (cairo_hash_table_t *hash_table);
 
-cairo_private cairo_bool_t
+cairo_private void *
 _cairo_hash_table_lookup (cairo_hash_table_t  *hash_table,
-			  cairo_hash_entry_t  *key,
-			  cairo_hash_entry_t **entry_return);
+			  cairo_hash_entry_t  *key);
 
 cairo_private void *
 _cairo_hash_table_random_entry (cairo_hash_table_t	   *hash_table,
@@ -81,7 +80,7 @@ _cairo_hash_table_remove (cairo_hash_table_t *hash_table,
 			  cairo_hash_entry_t *key);
 
 cairo_private void
-_cairo_hash_table_foreach (cairo_hash_table_t 	      *hash_table,
+_cairo_hash_table_foreach (cairo_hash_table_t	      *hash_table,
 			   cairo_hash_callback_func_t  hash_callback,
 			   void			      *closure);