connection: Don't declare a local variable just to takes its size
We can sizeof the struct type instead of declaring a pointer and
taking the size of what it points to.
Signed-off-by: Derek Foreman <derekf@osg.samsung.com>
Reviewed-by: Daniel Stone <daniels@collabora.com>
diff --git a/src/connection.c b/src/connection.c
index e60ad75..d029914 100644
--- a/src/connection.c
+++ b/src/connection.c
@@ -632,7 +632,7 @@
const char *signature;
struct argument_details arg;
struct wl_closure *closure;
- struct wl_array *array, *array_extra;
+ struct wl_array *array_extra;
count = arg_count_for_signature(message->signature);
if (count > WL_CLOSURE_MAX_ARGS) {
@@ -643,7 +643,8 @@
}
num_arrays = wl_message_count_arrays(message);
- closure = malloc(sizeof *closure + size + num_arrays * sizeof *array);
+ closure = malloc(sizeof *closure + size +
+ num_arrays * sizeof(struct wl_array));
if (closure == NULL) {
errno = ENOMEM;
wl_connection_consume(connection, size);