style: convert code to systemd coding style

https://systemd.io/CODING_STYLE/
Signed-off-by: George Kiagiadakis <george.kiagiadakis@collabora.com>

13 files changed, 1179 insertions, 1259 deletions

diff --git a/lib/client.c b/lib/client.c
index 735796f..4186ce4 100644
--- a/lib/client.c
+++ b/lib/client.c
@@ -13,72 +13,65 @@
 
 #define BUFFER_SIZE 1024
 
-static void
-on_lost_connection (struct icipc_sender *self,
-                               int receiver_fd,
-                               void *data)
-{
-  icipc_log_warn ("client: lost connection with server %d", receiver_fd);
+static void on_lost_connection(
+                struct icipc_sender *self,
+                int receiver_fd,
+                void *data) {
+        icipc_log_warn("client: lost connection with server %d", receiver_fd);
 }
 
 /* API */
 
-struct icipc_client *
-icipc_client_new (const char *path, bool connect)
-{
-  struct icipc_sender *base;
-  base = icipc_sender_new (path, BUFFER_SIZE, on_lost_connection, NULL, 0);
+struct icipc_client *icipc_client_new(const char *path, bool connect) {
+        struct icipc_sender *base;
+        base = icipc_sender_new(path, BUFFER_SIZE, on_lost_connection, NULL, 0);
 
-  if (connect)
-    icipc_sender_connect (base);
+        if (connect)
+                icipc_sender_connect(base);
 
-  return (struct icipc_client *)base;
+        return (struct icipc_client *)base;
 }
 
-void
-icipc_client_free (struct icipc_client *self)
-{
-  struct icipc_sender *base = icipc_client_to_sender (self);
-  icipc_sender_free (base);
+void icipc_client_free(struct icipc_client *self) {
+        struct icipc_sender *base = icipc_client_to_sender(self);
+        icipc_sender_free(base);
 }
 
-bool
-icipc_client_send_request (struct icipc_client *self,
-                           const char *name,
-                           const struct spa_pod *args,
-                           icipc_sender_reply_func_t reply,
-                           void *data)
-{
-  struct icipc_sender *base = icipc_client_to_sender (self);
+bool icipc_client_send_request(
+                struct icipc_client *self,
+                const char *name,
+                const struct spa_pod *args,
+                icipc_sender_reply_func_t reply,
+                void *data) {
+        struct icipc_sender *base = icipc_client_to_sender(self);
 
-  /* check params */
-  if (name == NULL)
-    return false;
+        /* check params */
+        if (name == NULL)
+                return false;
 
-  const size_t size = icipc_protocol_calculate_request_size (name, args);
-  uint8_t buffer[size];
-  icipc_protocol_build_request (buffer, size, name, args);
-  return icipc_sender_send (base, buffer, size, reply, data);
+        const size_t size = icipc_protocol_calculate_request_size(name, args);
+        uint8_t buffer[size];
+        icipc_protocol_build_request(buffer, size, name, args);
+        return icipc_sender_send(base, buffer, size, reply, data);
 }
 
-const struct spa_pod *
-icipc_client_send_request_finish (struct icipc_sender *self,
-                                  const uint8_t *buffer,
-                                  size_t size,
-                                  const char **error)
-{
-  /* error */
-  if (icipc_protocol_is_reply_error (buffer, size)) {
-    icipc_protocol_parse_reply_error (buffer, size, error);
-    return NULL;
-  }
+const struct spa_pod *icipc_client_send_request_finish(
+                struct icipc_sender *self,
+                const uint8_t * buffer,
+                size_t size,
+                const char **error) {
+        /* error */
+        if (icipc_protocol_is_reply_error(buffer, size)) {
+                icipc_protocol_parse_reply_error(buffer, size, error);
+                return NULL;
+        }
 
-  /* ok */
-  if (icipc_protocol_is_reply_ok (buffer, size)) {
-    const struct spa_pod *value = NULL;
-    if (icipc_protocol_parse_reply_ok (buffer, size, &value))
-      return value;
-  }
+        /* ok */
+        if (icipc_protocol_is_reply_ok(buffer, size)) {
+                const struct spa_pod *value = NULL;
+                if (icipc_protocol_parse_reply_ok(buffer, size, &value))
+                        return value;
+        }
 
-  return NULL;
+        return NULL;
 }
diff --git a/lib/client.h b/lib/client.h
index 7ac8e1a..597552c 100644
--- a/lib/client.h
+++ b/lib/client.h
@@ -25,29 +25,27 @@ extern "C" {
 struct icipc_client;
 
 ICIPC_API
-struct icipc_client *
-icipc_client_new (const char *path, bool connect);
+struct icipc_client *icipc_client_new(const char *path, bool connect);
 
 ICIPC_API
-void
-icipc_client_free (struct icipc_client *self);
+void icipc_client_free(struct icipc_client *self);
 
 ICIPC_API
-bool
-icipc_client_send_request (struct icipc_client *self,
-                           const char *name,
-                           const struct spa_pod *args,
-                           icipc_sender_reply_func_t reply,
-                           void *data);
+bool icipc_client_send_request(
+                struct icipc_client *self,
+                const char *name,
+                const struct spa_pod *args,
+                icipc_sender_reply_func_t reply,
+                void *data);
 
 /* for reply handlers only */
 
 ICIPC_API
-const struct spa_pod *
-icipc_client_send_request_finish (struct icipc_sender *self,
-                                  const uint8_t *buffer,
-                                  size_t size,
-                                  const char **error);
+const struct spa_pod *icipc_client_send_request_finish(
+                struct icipc_sender *self,
+                const uint8_t *buffer,
+                size_t size,
+                const char **error);
 
 #ifdef __cplusplus
 }
diff --git a/lib/defs.h b/lib/defs.h
index e962db7..b355fcb 100644
--- a/lib/defs.h
+++ b/lib/defs.h
@@ -10,7 +10,7 @@
 #define __ICIPC_DEFS_H__
 
 #if defined(__GNUC__)
-# define ICIPC_API_EXPORT extern __attribute__ ((visibility ("default")))
+# define ICIPC_API_EXPORT extern __attribute__((visibility("default")))
 #else
 # define ICIPC_API_EXPORT extern
 #endif
@@ -20,7 +20,7 @@
 #endif
 
 #ifndef ICIPC_PRIVATE_API
-#  define ICIPC_PRIVATE_API __attribute__ ((deprecated ("Private API")))
+#  define ICIPC_PRIVATE_API __attribute__((deprecated("Private API")))
 #endif
 
 #endif
diff --git a/lib/private.h b/lib/private.h
index a5b1b7d..fcdb448 100644
--- a/lib/private.h
+++ b/lib/private.h
@@ -25,75 +25,70 @@ extern "C" {
 /* log */
 
 #define icipc_log_info(F, ...) \
-    icipc_log(ICIPC_LOG_LEVEL_INFO, (F), ##__VA_ARGS__)
+        icipc_log(ICIPC_LOG_LEVEL_INFO, (F), ##__VA_ARGS__)
 #define icipc_log_warn(F, ...) \
-    icipc_log(ICIPC_LOG_LEVEL_WARN, (F), ##__VA_ARGS__)
+        icipc_log(ICIPC_LOG_LEVEL_WARN, (F), ##__VA_ARGS__)
 #define icipc_log_error(F, ...) \
-    icipc_log(ICIPC_LOG_LEVEL_ERROR, (F), ##__VA_ARGS__)
+        icipc_log(ICIPC_LOG_LEVEL_ERROR, (F), ##__VA_ARGS__)
 
-enum icipc_log_level {
-  ICIPC_LOG_LEVEL_NONE = 0,
-  ICIPC_LOG_LEVEL_ERROR,
-  ICIPC_LOG_LEVEL_WARN,
-  ICIPC_LOG_LEVEL_INFO,
-};
+typedef enum LogLevel {
+        ICIPC_LOG_LEVEL_NONE = 0,
+        ICIPC_LOG_LEVEL_ERROR,
+        ICIPC_LOG_LEVEL_WARN,
+        ICIPC_LOG_LEVEL_INFO,
+} LogLevel;
 
-void
-icipc_logv (enum icipc_log_level level,
-            const char *fmt,
-            va_list args) __attribute__ ((format (printf, 2, 0)));
+void icipc_logv (
+                LogLevel level,
+                const char *fmt,
+                va_list args) __attribute__((format(printf, 2, 0)));
 
-void
-icipc_log (enum icipc_log_level level,
-           const char *fmt,
-           ...) __attribute__ ((format (printf, 2, 3)));
+void icipc_log(
+                LogLevel level,
+                const char *fmt,
+                ...) __attribute__((format(printf, 2, 3)));
 
 /* socket path */
 
-int
-icipc_construct_socket_path (const char *name, char *buf, size_t buf_size);
+int icipc_construct_socket_path(const char *name, char *buf, size_t buf_size);
 
 /* socket */
 
-ssize_t
-icipc_socket_write (int fd, const uint8_t *buffer, size_t size);
+ssize_t icipc_socket_write(int fd, const uint8_t *buffer, size_t size);
 
-ssize_t
-icipc_socket_read (int fd, uint8_t **buffer, size_t *max_size);
+ssize_t icipc_socket_read(int fd, uint8_t **buffer, size_t *max_size);
 
 /* epoll thread */
 
-struct epoll_thread;
-
-typedef void (*icipc_epoll_thread_event_func_t) (struct epoll_thread *self,
-                                                 int fd,
-                                                 void *data);
-
-struct epoll_thread {
-  int socket_fd;
-  int epoll_fd;
-  int event_fd;
-  pthread_t thread;
-  icipc_epoll_thread_event_func_t socket_event_func;
-  icipc_epoll_thread_event_func_t other_event_func;
-  void *event_data;
+typedef struct EpollThread EpollThread;
+
+typedef void (*icipc_epoll_thread_event_func_t)(
+                struct EpollThread *self,
+                int fd,
+                void *data);
+
+struct EpollThread {
+        int socket_fd;
+        int epoll_fd;
+        int event_fd;
+        pthread_t thread;
+        icipc_epoll_thread_event_func_t socket_event_func;
+        icipc_epoll_thread_event_func_t other_event_func;
+        void *event_data;
 };
 
-bool
-icipc_epoll_thread_init (struct epoll_thread *self,
-                         int socket_fd,
-                         icipc_epoll_thread_event_func_t sock_func,
-                         icipc_epoll_thread_event_func_t other_func,
-                         void *data);
+bool icipc_epoll_thread_init(
+                EpollThread *self,
+                int socket_fd,
+                icipc_epoll_thread_event_func_t sock_func,
+                icipc_epoll_thread_event_func_t other_func,
+                void *data);
 
-bool
-icipc_epoll_thread_start (struct epoll_thread *self);
+bool icipc_epoll_thread_start(EpollThread *self);
 
-void
-icipc_epoll_thread_stop (struct epoll_thread *self);
+void icipc_epoll_thread_stop(EpollThread *self);
 
-void
-icipc_epoll_thread_destroy (struct epoll_thread *self);
+void icipc_epoll_thread_destroy(EpollThread *self);
 
 #ifdef __cplusplus
 }
diff --git a/lib/protocol.c b/lib/protocol.c
index 6b2bcf1..35a9485 100644
--- a/lib/protocol.c
+++ b/lib/protocol.c
@@ -15,210 +15,193 @@
 
 #define SIZE_PADDING 128
 
-enum icipc_protocol_reply_code {
-  REPLY_CODE_ERROR = 0,
-  REPLY_CODE_OK,
+enum {
+        REPLY_CODE_ERROR = 0,
+        REPLY_CODE_OK,
 };
 
-static bool
-is_reply (const uint8_t *buffer, size_t size, int code)
-{
-  const struct spa_pod *pod = (const struct spa_pod *)buffer;
-  struct spa_pod_parser p;
-  struct spa_pod_frame f;
-  int parsed_code = 0;
+static bool is_reply(const uint8_t *buffer, size_t size, int code) {
+        const struct spa_pod *pod = (const struct spa_pod *)buffer;
+        struct spa_pod_parser p;
+        struct spa_pod_frame f;
+        int parsed_code = 0;
 
-  /* check if struct */
-  if (!spa_pod_is_struct (pod))
-    return false;
+        /* check if struct */
+        if (!spa_pod_is_struct(pod))
+                return false;
 
-  /* parse */
-  spa_pod_parser_pod (&p, pod);
-  spa_pod_parser_push_struct(&p, &f);
-  spa_pod_parser_get_int (&p, &parsed_code);
+        /* parse */
+        spa_pod_parser_pod(&p, pod);
+        spa_pod_parser_push_struct(&p, &f);
+        spa_pod_parser_get_int(&p, &parsed_code);
 
-  return parsed_code == code;
+        return parsed_code == code;
 }
 
 /* API */
 
-size_t
-icipc_protocol_calculate_request_size (const char *name,
-                                       const struct spa_pod *args)
-{
-  assert (name);
-  return strlen(name) + (args ? SPA_POD_SIZE(args) : 8) + SIZE_PADDING;
+size_t icipc_protocol_calculate_request_size(
+                const char *name,
+                const struct spa_pod *args) {
+        assert (name);
+        return strlen(name) + (args ? SPA_POD_SIZE(args) : 8) + SIZE_PADDING;
 }
 
-void
-icipc_protocol_build_request (uint8_t *buffer,
-                              size_t size,
-                              const char *name,
-                              const struct spa_pod *args)
-{
-  const struct spa_pod none = SPA_POD_INIT_None();
-  struct spa_pod_builder b;
-  struct spa_pod_frame f;
-
-  memset (buffer, 0, size);
-
-  if (args == NULL)
-    args = &none;
-
-  spa_pod_builder_init (&b, buffer, size);
-  spa_pod_builder_push_struct (&b, &f);
-  spa_pod_builder_string (&b, name);
-  spa_pod_builder_primitive (&b, args);
-  spa_pod_builder_pop(&b, &f);
+void icipc_protocol_build_request(
+                uint8_t *buffer,
+                size_t size,
+                const char *name,
+                const struct spa_pod *args) {
+        const struct spa_pod none = SPA_POD_INIT_None();
+        struct spa_pod_builder b;
+        struct spa_pod_frame f;
+
+        memset(buffer, 0, size);
+
+        if (args == NULL)
+                args = &none;
+
+        spa_pod_builder_init(&b, buffer, size);
+        spa_pod_builder_push_struct(&b, &f);
+        spa_pod_builder_string(&b, name);
+        spa_pod_builder_primitive (&b, args);
+        spa_pod_builder_pop(&b, &f);
 }
 
-bool
-icipc_protocol_parse_request (const uint8_t *buffer,
-                              size_t size,
-                              const char **name,
-                              const struct spa_pod **args)
-{
-  const struct spa_pod *pod = (const struct spa_pod *)buffer;
-  struct spa_pod_parser p;
-  struct spa_pod_frame f;
-  const char *parsed_name = NULL;
-  struct spa_pod *parsed_args = NULL;
-
-  /* check if struct */
-  if (!spa_pod_is_struct (pod))
-    return false;
-
-  /* parse */
-  spa_pod_parser_pod (&p, pod);
-  spa_pod_parser_push_struct(&p, &f);
-  spa_pod_parser_get_string (&p, &parsed_name);
-  spa_pod_parser_get_pod (&p, &parsed_args);
-  spa_pod_parser_pop(&p, &f);
-
-  /* check name and args */
-  if (name == NULL || args == NULL)
-    return false;
-
-  if (name != NULL)
-    *name = parsed_name;
-  if (args != NULL)
-    *args = parsed_args;
-  return true;
+bool icipc_protocol_parse_request(
+                const uint8_t *buffer,
+                size_t size,
+                const char **name,
+                const struct spa_pod **args) {
+        const struct spa_pod *pod = (const struct spa_pod *)buffer;
+        struct spa_pod_parser p;
+        struct spa_pod_frame f;
+        const char *parsed_name = NULL;
+        struct spa_pod *parsed_args = NULL;
+
+        /* check if struct */
+        if (!spa_pod_is_struct(pod))
+                return false;
+
+        /* parse */
+        spa_pod_parser_pod(&p, pod);
+        spa_pod_parser_push_struct(&p, &f);
+        spa_pod_parser_get_string(&p, &parsed_name);
+        spa_pod_parser_get_pod(&p, &parsed_args);
+        spa_pod_parser_pop(&p, &f);
+
+        /* check name and args */
+        if (name == NULL || args == NULL)
+                return false;
+
+        if (name != NULL)
+                *name = parsed_name;
+        if (args != NULL)
+                *args = parsed_args;
+        return true;
 }
 
-size_t
-icipc_protocol_calculate_reply_ok_size (const struct spa_pod *value)
-{
-  return (value ? SPA_POD_SIZE(value) : 8) + SIZE_PADDING;
+size_t icipc_protocol_calculate_reply_ok_size(const struct spa_pod *value) {
+        return (value ? SPA_POD_SIZE(value) : 8) + SIZE_PADDING;
 }
 
-size_t
-icipc_protocol_calculate_reply_error_size (const char *msg)
-{
-  assert (msg);
-  return strlen(msg) + SIZE_PADDING;
+size_t icipc_protocol_calculate_reply_error_size(const char *msg) {
+        assert (msg);
+        return strlen(msg) + SIZE_PADDING;
 }
 
-void
-icipc_protocol_build_reply_ok (uint8_t *buffer,
-                               size_t size,
-                               const struct spa_pod *value)
-{
-  const struct spa_pod none = SPA_POD_INIT_None();
-  struct spa_pod_builder b;
-  struct spa_pod_frame f;
-
-  memset (buffer, 0, size);
-
-  if (value == NULL)
-    value = &none;
-
-  spa_pod_builder_init (&b, buffer, size);
-  spa_pod_builder_push_struct (&b, &f);
-  spa_pod_builder_int (&b, REPLY_CODE_OK);
-  spa_pod_builder_primitive (&b, value);
-  spa_pod_builder_pop(&b, &f);
+void icipc_protocol_build_reply_ok(
+                uint8_t *buffer,
+                size_t size,
+                const struct spa_pod *value) {
+        const struct spa_pod none = SPA_POD_INIT_None();
+        struct spa_pod_builder b;
+        struct spa_pod_frame f;
+
+        memset(buffer, 0, size);
+
+        if (value == NULL)
+                value = &none;
+
+        spa_pod_builder_init(&b, buffer, size);
+        spa_pod_builder_push_struct(&b, &f);
+        spa_pod_builder_int(&b, REPLY_CODE_OK);
+        spa_pod_builder_primitive(&b, value);
+        spa_pod_builder_pop(&b, &f);
 }
 
-void
-icipc_protocol_build_reply_error (uint8_t *buffer,
-                                  size_t size,
-                                  const char *msg)
-{
-  struct spa_pod_builder b;
-  struct spa_pod_frame f;
-
-  memset (buffer, 0, size);
-
-  spa_pod_builder_init (&b, buffer, size);
-  spa_pod_builder_push_struct (&b, &f);
-  spa_pod_builder_int (&b, REPLY_CODE_ERROR);
-  spa_pod_builder_string (&b, msg);
-  spa_pod_builder_pop(&b, &f);
+void icipc_protocol_build_reply_error(
+                uint8_t *buffer,
+                size_t size,
+                const char *msg) {
+        struct spa_pod_builder b;
+        struct spa_pod_frame f;
+
+        memset(buffer, 0, size);
+
+        spa_pod_builder_init(&b, buffer, size);
+        spa_pod_builder_push_struct(&b, &f);
+        spa_pod_builder_int(&b, REPLY_CODE_ERROR);
+        spa_pod_builder_string (&b, msg);
+        spa_pod_builder_pop(&b, &f);
 }
 
-bool
-icipc_protocol_is_reply_ok (const uint8_t *buffer, size_t size)
-{
-  return is_reply (buffer, size, REPLY_CODE_OK);
+bool icipc_protocol_is_reply_ok(const uint8_t *buffer, size_t size) {
+        return is_reply (buffer, size, REPLY_CODE_OK);
 }
 
-bool
-icipc_protocol_is_reply_error (const uint8_t *buffer, size_t size)
-{
-  return is_reply (buffer, size, REPLY_CODE_ERROR);
+bool icipc_protocol_is_reply_error(const uint8_t *buffer, size_t size) {
+        return is_reply (buffer, size, REPLY_CODE_ERROR);
 }
 
-bool
-icipc_protocol_parse_reply_ok (const uint8_t *buffer,
-                               size_t size,
-                               const struct spa_pod **value)
-{
-  const struct spa_pod *pod = (const struct spa_pod *)buffer;
-  struct spa_pod_parser p;
-  struct spa_pod_frame f;
-  int parsed_code = 0;
-  struct spa_pod *parsed_value = NULL;
-
-  /* check if struct */
-  if (!spa_pod_is_struct (pod))
-    return false;
-
-  /* parse */
-  spa_pod_parser_pod (&p, pod);
-  spa_pod_parser_push_struct(&p, &f);
-  spa_pod_parser_get_int (&p, &parsed_code);
-  spa_pod_parser_get_pod (&p, &parsed_value);
-  spa_pod_parser_pop (&p, &f);
-
-  if (value != NULL)
-    *value = parsed_value;
-  return true;
+bool icipc_protocol_parse_reply_ok (
+                const uint8_t *buffer,
+                size_t size,
+                const struct spa_pod **value) {
+        const struct spa_pod *pod = (const struct spa_pod *)buffer;
+        struct spa_pod_parser p;
+        struct spa_pod_frame f;
+        int parsed_code = 0;
+        struct spa_pod *parsed_value = NULL;
+
+        /* check if struct */
+        if (!spa_pod_is_struct(pod))
+                return false;
+
+        /* parse */
+        spa_pod_parser_pod(&p, pod);
+        spa_pod_parser_push_struct(&p, &f);
+        spa_pod_parser_get_int(&p, &parsed_code);
+        spa_pod_parser_get_pod(&p, &parsed_value);
+        spa_pod_parser_pop(&p, &f);
+
+        if (value != NULL)
+                *value = parsed_value;
+        return true;
 }
 
-bool
-icipc_protocol_parse_reply_error (const uint8_t *buffer,
-                                  size_t size,
-                                  const char **msg)
-{
-  const struct spa_pod *pod = (const struct spa_pod *)buffer;
-  struct spa_pod_parser p;
-  struct spa_pod_frame f;
-  int parsed_code = 0;
-  const char *parsed_msg = NULL;
-
-  /* check if struct */
-  if (!spa_pod_is_struct (pod))
-    return false;
-
-  /* parse */
-  spa_pod_parser_pod (&p, pod);
-  spa_pod_parser_push_struct(&p, &f);
-  spa_pod_parser_get_int (&p, &parsed_code);
-  spa_pod_parser_get_string (&p, &parsed_msg);
-  spa_pod_parser_pop (&p, &f);
-
-  if (msg != NULL)
-    *msg = parsed_msg;
-  return true;
+bool icipc_protocol_parse_reply_error (
+                const uint8_t *buffer,
+                size_t size,
+                const char **msg) {
+        const struct spa_pod *pod = (const struct spa_pod *)buffer;
+        struct spa_pod_parser p;
+        struct spa_pod_frame f;
+        int parsed_code = 0;
+        const char *parsed_msg = NULL;
+
+        /* check if struct */
+        if (!spa_pod_is_struct(pod))
+                return false;
+
+        /* parse */
+        spa_pod_parser_pod(&p, pod);
+        spa_pod_parser_push_struct(&p, &f);
+        spa_pod_parser_get_int(&p, &parsed_code);
+        spa_pod_parser_get_string(&p, &parsed_msg);
+        spa_pod_parser_pop(&p, &f);
+
+        if (msg != NULL)
+                *msg = parsed_msg;
+        return true;
 }
diff --git a/lib/protocol.h b/lib/protocol.h
index 730c918..34a92a0 100644
--- a/lib/protocol.h
+++ b/lib/protocol.h
@@ -20,65 +20,61 @@ extern "C" {
 /* request */
 
 ICIPC_API
-size_t
-icipc_protocol_calculate_request_size (const char *name,
-                                       const struct spa_pod *args);
+size_t icipc_protocol_calculate_request_size(
+                const char *name,
+                const struct spa_pod *args);
 
 ICIPC_API
-void
-icipc_protocol_build_request (uint8_t *buffer,
-                              size_t size,
-                              const char *name,
-                              const struct spa_pod *args);
+void icipc_protocol_build_request(
+                uint8_t *buffer,
+                size_t size,
+                const char *name,
+                const struct spa_pod *args);
 
 ICIPC_API
-bool
-icipc_protocol_parse_request (const uint8_t *buffer,
-                              size_t size,
-                              const char **name,
-                              const struct spa_pod **args);
+bool icipc_protocol_parse_request(
+                const uint8_t *buffer,
+                size_t size,
+                const char **name,
+                const struct spa_pod **args);
 
 /* reply */
 
 ICIPC_API
-size_t
-icipc_protocol_calculate_reply_ok_size (const struct spa_pod *value);
+size_t icipc_protocol_calculate_reply_ok_size(const struct spa_pod *value);
 
 ICIPC_API
-size_t
-icipc_protocol_calculate_reply_error_size (const char *msg);
+size_t icipc_protocol_calculate_reply_error_size(const char *msg);
 
 ICIPC_API
-void
-icipc_protocol_build_reply_ok (uint8_t *buffer,
-                               size_t size,
-                               const struct spa_pod *value);
+void icipc_protocol_build_reply_ok(
+                uint8_t *buffer,
+                size_t size,
+                const struct spa_pod *value);
 
 ICIPC_API
-void
-icipc_protocol_build_reply_error (uint8_t *buffer,
-                                  size_t size,
-                                  const char *msg);
+void icipc_protocol_build_reply_error(
+                uint8_t *buffer,
+                size_t size,
+                const char *msg);
 
 ICIPC_API
-bool
-icipc_protocol_is_reply_ok (const uint8_t *buffer, size_t size);
+bool icipc_protocol_is_reply_ok(const uint8_t *buffer, size_t size);
 
 ICIPC_API
-bool
-icipc_protocol_is_reply_error (const uint8_t *buffer, size_t size);
+bool icipc_protocol_is_reply_error(const uint8_t *buffer, size_t size);
 
 ICIPC_API
-bool
-icipc_protocol_parse_reply_ok (const uint8_t *buffer,
-                               size_t size,
-                               const struct spa_pod **value);
+bool icipc_protocol_parse_reply_ok(
+                const uint8_t *buffer,
+                size_t size,
+                const struct spa_pod **value);
 
 ICIPC_API
-bool
-icipc_protocol_parse_reply_error (const uint8_t *buffer,
-                                  size_t size,
-                                  const char **msg);
+bool icipc_protocol_parse_reply_error(
+                const uint8_t *buffer,
+                size_t size,
+                const char **msg);
 
 #ifdef __cplusplus
 }
diff --git a/lib/receiver.c b/lib/receiver.c
index ba7edda..9c21063 100644
--- a/lib/receiver.c
+++ b/lib/receiver.c
@@ -1,4 +1,4 @@
-/* PipeWire AGL Cluster IPC
+/* PipeWire AGL Cluster IPC
  *
  * Copyright © 2021 Collabora Ltd.
  *    @author Julian Bouzas <julian.bouzas@collabora.com>
@@ -15,196 +15,188 @@
 #include <string.h>
 #include <errno.h>
 #include <assert.h>
-
 #include "private.h"
 #include "receiver.h"
-
 #include "icipc.h"
 
 #define MAX_SENDERS 128
 
 struct icipc_receiver {
-  struct sockaddr_un addr;
-  int socket_fd;
+        struct sockaddr_un addr;
+        int socket_fd;
 
-  uint8_t *buffer_read;
-  size_t buffer_size;
+        uint8_t *buffer_read;
+        size_t buffer_size;
 
-  struct epoll_thread epoll_thread;
-  bool thread_running;
+        EpollThread epoll_thread;
+        bool thread_running;
 
-  const struct icipc_receiver_events *events;
-  void *events_data;
+        const struct icipc_receiver_events *events;
+        void *events_data;
 
-  /* for subclasses */
-  void *user_data;
+        /* for subclasses */
+        void *user_data;
 };
 
-static bool
-reply_message (struct icipc_receiver *self,
-               int sender_fd,
-               uint8_t *buffer,
-               size_t size)
-{
-  return self->events && self->events->handle_message ?
-    self->events->handle_message (self, sender_fd, buffer, size, self->events_data) :
-    icipc_socket_write (sender_fd, buffer, size) == (ssize_t)size;
+static bool reply_message(
+                struct icipc_receiver *self,
+                int sender_fd,
+                uint8_t * buffer,
+                size_t size) {
+        return self->events && self->events->handle_message ?
+            self->events->handle_message(self, sender_fd, buffer, size,
+                                         self->events_data) :
+            icipc_socket_write(sender_fd, buffer, size) == (ssize_t) size;
 }
 
-static void
-socket_event_received (struct epoll_thread *t, int fd, void *data)
-{
-  /* sender wants to connect, accept connection */
-  struct icipc_receiver *self = data;
-  socklen_t addr_size = sizeof(self->addr);
-  int sender_fd = accept4 (fd, (struct sockaddr*)&self->addr, &addr_size,
-      SOCK_CLOEXEC | SOCK_NONBLOCK);
-  struct epoll_event event;
-  event.events = EPOLLIN;
-  event.data.fd = sender_fd;
-  epoll_ctl (t->epoll_fd, EPOLL_CTL_ADD, sender_fd, &event);
-  if (self->events && self->events->sender_state)
-    self->events->sender_state (self, sender_fd,
-        ICIPC_RECEIVER_SENDER_STATE_CONNECTED, self->events_data);
+static void socket_event_received(EpollThread *t, int fd, void *data) {
+        /* sender wants to connect, accept connection */
+        struct icipc_receiver *self = data;
+        socklen_t addr_size = sizeof(self->addr);
+        int sender_fd = accept4(fd, (struct sockaddr *)&self->addr, &addr_size,
+                                SOCK_CLOEXEC | SOCK_NONBLOCK);
+        struct epoll_event event;
+        event.events = EPOLLIN;
+        event.data.fd = sender_fd;
+        epoll_ctl(t->epoll_fd, EPOLL_CTL_ADD, sender_fd, &event);
+        if (self->events && self->events->sender_state)
+                self->events->sender_state(self, sender_fd,
+                                           ICIPC_RECEIVER_SENDER_STATE_CONNECTED,
+                                           self->events_data);
 }
 
-static void
-other_event_received (struct epoll_thread *t, int fd, void *data)
-{
-  struct icipc_receiver *self = data;
-
-  /* sender sends a message, read it and reply */
-  ssize_t size = icipc_socket_read (fd, &self->buffer_read, &self->buffer_size);
-  if (size <= 0) {
-    if (size < 0)
-      icipc_log_error ("receiver: could not read message: %s", strerror(errno));
-    /* client disconnected */
-    epoll_ctl (t->epoll_fd, EPOLL_CTL_DEL, fd, NULL);
-    close (fd);
-    if (self->events && self->events->sender_state)
-      self->events->sender_state (self, fd,
-          ICIPC_RECEIVER_SENDER_STATE_DISCONNECTED, self->events_data);
-    return;
-  }
-
-  /* reply */
-  if (!reply_message (self, fd, self->buffer_read, size))
-    icipc_log_error ("receiver: could not reply message: %s", strerror(errno));
-
-  return;
+static void other_event_received(EpollThread *t, int fd, void *data) {
+        struct icipc_receiver *self = data;
+
+        /* sender sends a message, read it and reply */
+        ssize_t size =
+            icipc_socket_read(fd, &self->buffer_read, &self->buffer_size);
+        if (size <= 0) {
+                if (size < 0)
+                        icipc_log_error("receiver: could not read message: %s",
+                                        strerror(errno));
+                /* client disconnected */
+                epoll_ctl(t->epoll_fd, EPOLL_CTL_DEL, fd, NULL);
+                close(fd);
+                if (self->events && self->events->sender_state)
+                        self->events->sender_state(self, fd,
+                                                   ICIPC_RECEIVER_SENDER_STATE_DISCONNECTED,
+                                                   self->events_data);
+                return;
+        }
+
+        /* reply */
+        if (!reply_message(self, fd, self->buffer_read, size))
+                icipc_log_error("receiver: could not reply message: %s",
+                                strerror(errno));
+
+        return;
 }
 
 /* API */
 
-struct icipc_receiver *
-icipc_receiver_new (const char *path,
-                    size_t buffer_size,
-                    const struct icipc_receiver_events *events,
-                    void *events_data,
-                    size_t user_size)
-{
-  struct icipc_receiver *self;
-  int res;
-
-  /* check params */
-  if (path == NULL || buffer_size == 0)
-    return NULL;
-
-  self = calloc (1, sizeof (struct icipc_receiver) + user_size);
-  if (self == NULL)
-    return NULL;
-
-  self->socket_fd = -1;
-
-  /* set address */
-  self->addr.sun_family = AF_LOCAL;
-  res = icipc_construct_socket_path (path, self->addr.sun_path, sizeof(self->addr.sun_path));
-  if (res < 0)
-    goto error;
-
-  unlink (self->addr.sun_path);
-
-  /* create socket */
-  self->socket_fd =
-      socket(PF_LOCAL, SOCK_STREAM | SOCK_CLOEXEC | SOCK_NONBLOCK, 0);
-  if (self->socket_fd < 0)
-    goto error;
-
-  /* bind socket */
-  if (bind (self->socket_fd, (struct sockaddr *)&self->addr,
-      sizeof(self->addr)) != 0)
-    goto error;
-
-  /* listen socket */
-  if (listen (self->socket_fd, MAX_SENDERS) != 0)
-    goto error;
-
-  /* alloc buffer read */
-  self->buffer_size = buffer_size;
-  self->buffer_read = calloc (buffer_size, sizeof (uint8_t));
-  if (self->buffer_read == NULL)
-    goto error;
-
-  /* init epoll thread */
-  if (!icipc_epoll_thread_init (&self->epoll_thread, self->socket_fd,
-      socket_event_received, other_event_received, self))
-    goto error;
-
-  self->events = events;
-  self->events_data = events_data;
-  if (user_size > 0)
-    self->user_data = (void *)((uint8_t *)self + sizeof (struct icipc_receiver));
-
-  return self;
-
-error:
-  if (self->buffer_read)
-    free (self->buffer_read);
-  if (self->socket_fd != -1)
-    close (self->socket_fd);
-  free (self);
-  return NULL;
+struct icipc_receiver *icipc_receiver_new(
+                const char *path,
+                size_t buffer_size,
+                const struct icipc_receiver_events *events,
+                void *events_data,
+                size_t user_size) {
+        struct icipc_receiver *self;
+        int res;
+
+        /* check params */
+        if (path == NULL || buffer_size == 0)
+                return NULL;
+
+        self = calloc(1, sizeof(struct icipc_receiver) + user_size);
+        if (self == NULL)
+                return NULL;
+
+        self->socket_fd = -1;
+
+        /* set address */
+        self->addr.sun_family = AF_LOCAL;
+        res =
+            icipc_construct_socket_path(path, self->addr.sun_path,
+                                        sizeof(self->addr.sun_path));
+        if (res < 0)
+                goto error;
+
+        unlink(self->addr.sun_path);
+
+        /* create socket */
+        self->socket_fd =
+            socket(PF_LOCAL, SOCK_STREAM | SOCK_CLOEXEC | SOCK_NONBLOCK, 0);
+        if (self->socket_fd < 0)
+                goto error;
+
+        /* bind socket */
+        if (bind(self->socket_fd, (struct sockaddr *)&self->addr,
+                 sizeof(self->addr)) != 0)
+                goto error;
+
+        /* listen socket */
+        if (listen(self->socket_fd, MAX_SENDERS) != 0)
+                goto error;
+
+        /* alloc buffer read */
+        self->buffer_size = buffer_size;
+        self->buffer_read = calloc(buffer_size, sizeof(uint8_t));
+        if (self->buffer_read == NULL)
+                goto error;
+
+        /* init epoll thread */
+        if (!icipc_epoll_thread_init(&self->epoll_thread, self->socket_fd,
+                                     socket_event_received,
+                                     other_event_received, self))
+                goto error;
+
+        self->events = events;
+        self->events_data = events_data;
+        if (user_size > 0)
+                self->user_data =
+                    (void *)((uint8_t *) self + sizeof(struct icipc_receiver));
+
+        return self;
+
+ error:
+        if (self->buffer_read)
+                free(self->buffer_read);
+        if (self->socket_fd != -1)
+                close(self->socket_fd);
+        free(self);
+        return NULL;
 }
 
-void
-icipc_receiver_free (struct icipc_receiver *self)
-{
-  icipc_receiver_stop (self);
+void icipc_receiver_free(struct icipc_receiver *self) {
+        icipc_receiver_stop(self);
 
-  icipc_epoll_thread_destroy (&self->epoll_thread);
-  free (self->buffer_read);
-  close (self->socket_fd);
-  unlink (self->addr.sun_path);
-  free (self);
+        icipc_epoll_thread_destroy(&self->epoll_thread);
+        free(self->buffer_read);
+        close(self->socket_fd);
+        unlink(self->addr.sun_path);
+        free(self);
 }
 
-bool
-icipc_receiver_start (struct icipc_receiver *self)
-{
-  if (icipc_receiver_is_running (self))
-    return true;
+bool icipc_receiver_start(struct icipc_receiver *self) {
+        if (icipc_receiver_is_running(self))
+                return true;
 
-  self->thread_running = icipc_epoll_thread_start (&self->epoll_thread);
-  return self->thread_running;
+        self->thread_running = icipc_epoll_thread_start(&self->epoll_thread);
+        return self->thread_running;
 }
 
-void
-icipc_receiver_stop (struct icipc_receiver *self)
-{
-  if (icipc_receiver_is_running (self)) {
-    icipc_epoll_thread_stop (&self->epoll_thread);
-    self->thread_running = false;
-  }
+void icipc_receiver_stop(struct icipc_receiver *self) {
+        if (icipc_receiver_is_running(self)) {
+                icipc_epoll_thread_stop(&self->epoll_thread);
+                self->thread_running = false;
+        }
 }
 
-bool
-icipc_receiver_is_running (struct icipc_receiver *self)
-{
-  return self->thread_running;
+bool icipc_receiver_is_running(struct icipc_receiver *self) {
+        return self->thread_running;
 }
 
-void *
-icipc_receiver_get_user_data (struct icipc_receiver *self)
-{
-  return self->user_data;
+void *icipc_receiver_get_user_data(struct icipc_receiver *self) {
+        return self->user_data;
 }
diff --git a/lib/receiver.h b/lib/receiver.h
index cf13e1b..b14da35 100644
--- a/lib/receiver.h
+++ b/lib/receiver.h
@@ -22,54 +22,51 @@ extern "C" {
 struct icipc_receiver;
 
 enum icipc_receiver_sender_state {
-  ICIPC_RECEIVER_SENDER_STATE_CONNECTED = 0,
-  ICIPC_RECEIVER_SENDER_STATE_DISCONNECTED
+        ICIPC_RECEIVER_SENDER_STATE_CONNECTED = 0,
+        ICIPC_RECEIVER_SENDER_STATE_DISCONNECTED
 };
 
 struct icipc_receiver_events {
-  /* emitted when a sender state changes */
-  void (*sender_state) (struct icipc_receiver *self,
+        /* emitted when a sender state changes */
+        void (*sender_state)(
+                        struct icipc_receiver *self,
                         int sender_fd,
                         enum icipc_receiver_sender_state state,
                         void *data);
 
-  /* emitted when message is received and needs to be handled */
-  bool (*handle_message) (struct icipc_receiver *self,
-                          int sender_fd,
-                          const uint8_t *buffer,
-                          size_t size,
-                          void *data);
+        /* emitted when message is received and needs to be handled */
+        bool (*handle_message)(
+                        struct icipc_receiver *self,
+                        int sender_fd,
+                        const uint8_t *buffer,
+                        size_t size,
+                        void *data);
 };
 
 ICIPC_API
-struct icipc_receiver *
-icipc_receiver_new (const char *path,
-                    size_t buffer_size,
-                    const struct icipc_receiver_events *events,
-                    void *events_data,
-                    size_t user_size);
+struct icipc_receiver *icipc_receiver_new(
+                const char *path,
+                size_t buffer_size,
+                const struct icipc_receiver_events *events,
+                void *events_data,
+                size_t user_size);
 
 ICIPC_API
-void
-icipc_receiver_free (struct icipc_receiver *self);
+void icipc_receiver_free(struct icipc_receiver *self);
 
 ICIPC_API
-bool
-icipc_receiver_start (struct icipc_receiver *self);
+bool icipc_receiver_start(struct icipc_receiver *self);
 
 ICIPC_API
-void
-icipc_receiver_stop (struct icipc_receiver *self);
+void icipc_receiver_stop(struct icipc_receiver *self);
 
 ICIPC_API
-bool
-icipc_receiver_is_running (struct icipc_receiver *self);
+bool icipc_receiver_is_running(struct icipc_receiver *self);
 
 /* for subclasses only */
 
 ICIPC_API
-void *
-icipc_receiver_get_user_data (struct icipc_receiver *self);
+void *icipc_receiver_get_user_data(struct icipc_receiver *self);
 
 #ifdef __cplusplus
 }
diff --git a/lib/sender.c b/lib/sender.c
index e795bc4..fa2882e 100644
--- a/lib/sender.c
+++ b/lib/sender.c
@@ -21,256 +21,247 @@
 
 #define MAX_ASYNC_TASKS 128
 
-struct icipc_sender_task {
-  icipc_sender_reply_func_t func;
-  void *data;
-};
+typedef struct SenderTask {
+        icipc_sender_reply_func_t func;
+        void *data;
+} SenderTask;
 
 struct icipc_sender {
-  struct sockaddr_un addr;
-  int socket_fd;
+        struct sockaddr_un addr;
+        int socket_fd;
 
-  uint8_t *buffer_read;
-  size_t buffer_size;
+        uint8_t *buffer_read;
+        size_t buffer_size;
 
-  struct epoll_thread epoll_thread;
-  bool is_connected;
+        EpollThread epoll_thread;
+        bool is_connected;
 
-  icipc_sender_lost_conn_func_t lost_func;
-  void *lost_data;
-  bool lost_connection;
+        icipc_sender_lost_conn_func_t lost_func;
+        void *lost_data;
+        bool lost_connection;
 
-  struct icipc_sender_task async_tasks[MAX_ASYNC_TASKS];
+        SenderTask async_tasks[MAX_ASYNC_TASKS];
 
-  /* for subclasses */
-  void *user_data;
+        /* for subclasses */
+        void *user_data;
 };
 
-static int
-push_sync_task (struct icipc_sender *self,
+static int push_sync_task(
+                struct icipc_sender *self,
                 icipc_sender_reply_func_t func,
-                void *data)
-{
-  size_t i;
-  for (i = MAX_ASYNC_TASKS; i > 1; i--) {
-    struct icipc_sender_task *curr = self->async_tasks + i - 1;
-    struct icipc_sender_task *next = self->async_tasks + i - 2;
-    if (next->func != NULL && curr->func == NULL) {
-      curr->func = func;
-      curr->data = data;
-      return i - 1;
-    } else if (i - 2 == 0 && next->func == NULL) {
-      /* empty queue */
-      next->func = func;
-      next->data = data;
-      return 0;
-    }
-  }
-  return -1;
+                void *data) {
+        size_t i;
+        for (i = MAX_ASYNC_TASKS; i > 1; i--) {
+                SenderTask *curr = self->async_tasks + i - 1;
+                SenderTask *next = self->async_tasks + i - 2;
+                if (next->func != NULL && curr->func == NULL) {
+                        curr->func = func;
+                        curr->data = data;
+                        return i - 1;
+                } else if (i - 2 == 0 && next->func == NULL) {
+                        /* empty queue */
+                        next->func = func;
+                        next->data = data;
+                        return 0;
+                }
+        }
+        return -1;
 }
 
-static void
-pop_sync_task (struct icipc_sender *self,
-               bool trigger,
-               bool all,
-               const uint8_t *buffer,
-               size_t size)
-{
-  size_t i;
-  for (i = 0; i < MAX_ASYNC_TASKS; i++) {
-    struct icipc_sender_task *task = self->async_tasks + i;
-    if (task->func != NULL) {
-      if (trigger)
-        task->func (self, buffer, size, task->data);
-      task->func = NULL;
-      if (!all)
-        return;
-    }
-  }
+static void pop_sync_task(
+                struct icipc_sender *self,
+                bool trigger,
+                bool all,
+                const uint8_t * buffer,
+                size_t size) {
+        size_t i;
+        for (i = 0; i < MAX_ASYNC_TASKS; i++) {
+                SenderTask *task = self->async_tasks + i;
+                if (task->func != NULL) {
+                        if (trigger)
+                                task->func(self, buffer, size, task->data);
+                        task->func = NULL;
+                        if (!all)
+                                return;
+                }
+        }
 }
 
-static void
-socket_event_received (struct epoll_thread *t, int fd, void *data)
-{
-  struct icipc_sender *self = data;
-
-  /* receiver sends a reply, read it trigger corresponding task */
-  ssize_t size = icipc_socket_read (fd, &self->buffer_read, &self->buffer_size);
-  if (size <= 0) {
-    if (size < 0)
-      icipc_log_error ("sender: could not read reply: %s", strerror(errno));
-    /* receiver disconnected */
-    epoll_ctl (t->epoll_fd, EPOLL_CTL_DEL, fd, NULL);
-    shutdown(self->socket_fd, SHUT_RDWR);
-    self->is_connected = false;
-    self->lost_connection = true;
-    if (self->lost_func)
-      self->lost_func (self, fd, self->lost_data);
-    /* clear queue */
-    pop_sync_task (self, true, true, NULL, 0);
-    return;
-  }
-
-  /* trigger async task */
-  pop_sync_task (self, true, false, self->buffer_read, size);
-  return;
+static void socket_event_received(EpollThread *t, int fd, void *data) {
+        struct icipc_sender *self = data;
+
+        /* receiver sends a reply, read it trigger corresponding task */
+        ssize_t size =
+            icipc_socket_read(fd, &self->buffer_read, &self->buffer_size);
+        if (size <= 0) {
+                if (size < 0)
+                        icipc_log_error("sender: could not read reply: %s",
+                                        strerror(errno));
+                /* receiver disconnected */
+                epoll_ctl(t->epoll_fd, EPOLL_CTL_DEL, fd, NULL);
+                shutdown(self->socket_fd, SHUT_RDWR);
+                self->is_connected = false;
+                self->lost_connection = true;
+                if (self->lost_func)
+                        self->lost_func(self, fd, self->lost_data);
+                /* clear queue */
+                pop_sync_task(self, true, true, NULL, 0);
+                return;
+        }
+
+        /* trigger async task */
+        pop_sync_task(self, true, false, self->buffer_read, size);
+        return;
 }
 
 /* API */
 
-struct icipc_sender *
-icipc_sender_new (const char *path,
-                  size_t buffer_size,
-                  icipc_sender_lost_conn_func_t lost_func,
-                  void *lost_data,
-                  size_t user_size)
-{
-  struct icipc_sender *self;
-  int res;
-
-  if (path == NULL)
-    return NULL;
-
-  self = calloc (1, sizeof (struct icipc_sender) + user_size);
-  if (self == NULL)
-    return NULL;
-
-  self->socket_fd = -1;
-
-  /* set address */
-  self->addr.sun_family = AF_LOCAL;
-  res = icipc_construct_socket_path (path, self->addr.sun_path, sizeof(self->addr.sun_path));
-  if (res < 0)
-    goto error;
-
-  /* create socket */
-  self->socket_fd =
-      socket(PF_LOCAL, SOCK_STREAM | SOCK_CLOEXEC| SOCK_NONBLOCK, 0);
-  if (self->socket_fd < 0)
-    goto error;
-
-  /* alloc buffer read */
-  self->buffer_size = buffer_size;
-  self->buffer_read = calloc (buffer_size, sizeof (uint8_t));
-  if (self->buffer_read == NULL)
-    goto error;
-
-  /* init epoll thread */
-  if (!icipc_epoll_thread_init (&self->epoll_thread, self->socket_fd,
-      socket_event_received, NULL, self))
-    goto error;
-
-  self->lost_func = lost_func;
-  self->lost_data = lost_data;
-  self->lost_connection = false;
-  if (user_size > 0)
-    self->user_data = (void *)((uint8_t *)self + sizeof (struct icipc_sender));
-
-  return self;
-
-error:
-  if (self->buffer_read)
-    free (self->buffer_read);
-  if (self->socket_fd != -1)
-    close (self->socket_fd);
-  free (self);
-  return NULL;
+struct icipc_sender *icipc_sender_new(
+                const char *path,
+                size_t buffer_size,
+                icipc_sender_lost_conn_func_t lost_func,
+                void *lost_data,
+                size_t user_size) {
+        struct icipc_sender *self;
+        int res;
+
+        if (path == NULL)
+                return NULL;
+
+        self = calloc(1, sizeof(struct icipc_sender) + user_size);
+        if (self == NULL)
+                return NULL;
+
+        self->socket_fd = -1;
+
+        /* set address */
+        self->addr.sun_family = AF_LOCAL;
+        res =
+            icipc_construct_socket_path(path, self->addr.sun_path,
+                                        sizeof(self->addr.sun_path));
+        if (res < 0)
+                goto error;
+
+        /* create socket */
+        self->socket_fd =
+            socket(PF_LOCAL, SOCK_STREAM | SOCK_CLOEXEC | SOCK_NONBLOCK, 0);
+        if (self->socket_fd < 0)
+                goto error;
+
+        /* alloc buffer read */
+        self->buffer_size = buffer_size;
+        self->buffer_read = calloc(buffer_size, sizeof(uint8_t));
+        if (self->buffer_read == NULL)
+                goto error;
+
+        /* init epoll thread */
+        if (!icipc_epoll_thread_init(&self->epoll_thread, self->socket_fd,
+                                     socket_event_received, NULL, self))
+                goto error;
+
+        self->lost_func = lost_func;
+        self->lost_data = lost_data;
+        self->lost_connection = false;
+        if (user_size > 0)
+                self->user_data =
+                    (void *)((uint8_t *) self + sizeof(struct icipc_sender));
+
+        return self;
+
+ error:
+        if (self->buffer_read)
+                free(self->buffer_read);
+        if (self->socket_fd != -1)
+                close(self->socket_fd);
+        free(self);
+        return NULL;
 }
 
-void
-icipc_sender_free (struct icipc_sender *self)
-{
-  icipc_sender_disconnect (self);
+void icipc_sender_free(struct icipc_sender *self) {
+        icipc_sender_disconnect(self);
 
-  icipc_epoll_thread_destroy (&self->epoll_thread);
-  free (self->buffer_read);
-  close (self->socket_fd);
-  free (self);
+        icipc_epoll_thread_destroy(&self->epoll_thread);
+        free(self->buffer_read);
+        close(self->socket_fd);
+        free(self);
 }
 
-bool
-icipc_sender_connect (struct icipc_sender *self)
-{
-  if (icipc_sender_is_connected (self))
-    return true;
-
-  /* if connection was lost, re-init epoll thread with new socket */
-  if (self->lost_connection) {
-    icipc_epoll_thread_stop (&self->epoll_thread);
-    icipc_epoll_thread_destroy (&self->epoll_thread);
-    self->socket_fd =
-        socket(PF_LOCAL, SOCK_STREAM | SOCK_CLOEXEC| SOCK_NONBLOCK, 0);
-    if (self->socket_fd < 0)
-      return false;
-    if (!icipc_epoll_thread_init (&self->epoll_thread, self->socket_fd,
-        socket_event_received, NULL, self)) {
-      close (self->socket_fd);
-      return false;
-    }
-    self->lost_connection = false;
-  }
-
-  /* connect */
-  if (connect(self->socket_fd, (struct sockaddr *)&self->addr,
-        sizeof(self->addr)) == 0 &&
-      icipc_epoll_thread_start (&self->epoll_thread)) {
-    self->is_connected = true;
-    return true;
-  }
-
-  return false;
+bool icipc_sender_connect(struct icipc_sender *self) {
+        if (icipc_sender_is_connected(self))
+                return true;
+
+        /* if connection was lost, re-init epoll thread with new socket */
+        if (self->lost_connection) {
+                icipc_epoll_thread_stop(&self->epoll_thread);
+                icipc_epoll_thread_destroy(&self->epoll_thread);
+                self->socket_fd =
+                    socket(PF_LOCAL, SOCK_STREAM | SOCK_CLOEXEC | SOCK_NONBLOCK,
+                           0);
+                if (self->socket_fd < 0)
+                        return false;
+                if (!icipc_epoll_thread_init
+                    (&self->epoll_thread, self->socket_fd,
+                     socket_event_received, NULL, self)) {
+                        close(self->socket_fd);
+                        return false;
+                }
+                self->lost_connection = false;
+        }
+
+        /* connect */
+        if (connect(self->socket_fd, (struct sockaddr *)&self->addr,
+                    sizeof(self->addr)) == 0 &&
+            icipc_epoll_thread_start(&self->epoll_thread)) {
+                self->is_connected = true;
+                return true;
+        }
+
+        return false;
 }
 
-void
-icipc_sender_disconnect (struct icipc_sender *self)
-{
-  if (icipc_sender_is_connected (self)) {
-    icipc_epoll_thread_stop (&self->epoll_thread);
-    shutdown(self->socket_fd, SHUT_RDWR);
-    self->is_connected = false;
-  }
+void icipc_sender_disconnect(struct icipc_sender *self) {
+        if (icipc_sender_is_connected(self)) {
+                icipc_epoll_thread_stop(&self->epoll_thread);
+                shutdown(self->socket_fd, SHUT_RDWR);
+                self->is_connected = false;
+        }
 }
 
-bool
-icipc_sender_is_connected (struct icipc_sender *self)
-{
-  return self->is_connected;
+bool icipc_sender_is_connected(struct icipc_sender *self) {
+        return self->is_connected;
 }
 
-bool
-icipc_sender_send (struct icipc_sender *self,
-                   const uint8_t *buffer,
-                   size_t size,
-                   icipc_sender_reply_func_t func,
-                   void *data)
-{
-  int id = -1;
-
-  if (buffer == NULL || size == 0)
-    return false;
-
-  if (!icipc_sender_is_connected (self))
-    return false;
-
-  /* add the task in the queue */
-  if (func) {
-    id = push_sync_task (self, func, data);
-    if (id == -1)
-      return false;
-  }
-
-  /* write buffer and remove task if it fails */
-  if (icipc_socket_write (self->socket_fd, buffer, size) <= 0) {
-    if (id != -1)
-      self->async_tasks[id].func = NULL;
-    return false;
-  }
-
-  return true;
+bool icipc_sender_send(
+                struct icipc_sender *self,
+                const uint8_t * buffer,
+                size_t size,
+                icipc_sender_reply_func_t func,
+                void *data) {
+        int id = -1;
+
+        if (buffer == NULL || size == 0)
+                return false;
+
+        if (!icipc_sender_is_connected(self))
+                return false;
+
+        /* add the task in the queue */
+        if (func) {
+                id = push_sync_task(self, func, data);
+                if (id == -1)
+                        return false;
+        }
+
+        /* write buffer and remove task if it fails */
+        if (icipc_socket_write(self->socket_fd, buffer, size) <= 0) {
+                if (id != -1)
+                        self->async_tasks[id].func = NULL;
+                return false;
+        }
+
+        return true;
 }
 
-void *
-icipc_sender_get_user_data (struct icipc_sender *self)
-{
-  return self->user_data;
+void *icipc_sender_get_user_data(struct icipc_sender *self) {
+        return self->user_data;
 }
diff --git a/lib/sender.h b/lib/sender.h
index 88aaf3a..d1b4083 100644
--- a/lib/sender.h
+++ b/lib/sender.h
@@ -21,52 +21,49 @@ extern "C" {
 
 struct icipc_sender;
 
-typedef void (*icipc_sender_lost_conn_func_t) (struct icipc_sender *self,
-                                               int receiver_fd,
-                                               void *data);
+typedef void (*icipc_sender_lost_conn_func_t)(
+                struct icipc_sender *self,
+                int receiver_fd,
+                void *data);
 
-typedef void (*icipc_sender_reply_func_t) (struct icipc_sender *self,
-                                           const uint8_t *buffer,
-                                           size_t size,
-                                           void *data);
+typedef void (*icipc_sender_reply_func_t)(
+                struct icipc_sender *self,
+                const uint8_t *buffer,
+                size_t size,
+                void *data);
 
 ICIPC_API
-struct icipc_sender *
-icipc_sender_new (const char *path,
-                  size_t buffer_size,
-                  icipc_sender_lost_conn_func_t lost_func,
-                  void *lost_data,
-                  size_t user_size);
+struct icipc_sender *icipc_sender_new(
+                const char *path,
+                size_t buffer_size,
+                icipc_sender_lost_conn_func_t lost_func,
+                void *lost_data,
+                size_t user_size);
 
 ICIPC_API
-void
-icipc_sender_free (struct icipc_sender *self);
+void icipc_sender_free(struct icipc_sender *self);
 
 ICIPC_API
-bool
-icipc_sender_connect (struct icipc_sender *self);
+bool icipc_sender_connect(struct icipc_sender *self);
 
 ICIPC_API
-void
-icipc_sender_disconnect (struct icipc_sender *self);
+void icipc_sender_disconnect(struct icipc_sender *self);
 
 ICIPC_API
-bool
-icipc_sender_is_connected (struct icipc_sender *self);
+bool icipc_sender_is_connected(struct icipc_sender *self);
 
 ICIPC_API
-bool
-icipc_sender_send (struct icipc_sender *self,
-                   const uint8_t *buffer,
-                   size_t size,
-                   icipc_sender_reply_func_t reply,
-                   void *data);
+bool icipc_sender_send(
+                struct icipc_sender *self,
+                const uint8_t *buffer,
+                size_t size,
+                icipc_sender_reply_func_t reply,
+                void *data);
 
 /* for subclasses only */
 
 ICIPC_API
-void *
-icipc_sender_get_user_data (struct icipc_sender *self);
+void *icipc_sender_get_user_data(struct icipc_sender *self);
 
 #ifdef __cplusplus
 }
diff --git a/lib/server.c b/lib/server.c
index 86f8322..747b829 100644
--- a/lib/server.c
+++ b/lib/server.c
@@ -16,246 +16,237 @@
 #define BUFFER_SIZE 1024
 #define MAX_REQUEST_HANDLERS 128
 
-struct icipc_server_client_handler
-{
-  icipc_server_client_handler_func_t handler;
-  void *data;
-};
-
-struct icipc_server_request_handler
-{
-  const char *name;
-  icipc_server_request_handler_func_t handler;
-  void *data;
-};
-
-struct icipc_server_priv {
-  pthread_mutex_t mutex;
-  struct icipc_server_client_handler client_handler;
-  size_t n_request_handlers;
-  struct icipc_server_request_handler request_handlers[MAX_REQUEST_HANDLERS];
-};
-
-static void
-sender_state (struct icipc_receiver *base,
-              int sender_fd,
-              enum icipc_receiver_sender_state sender_state,
-              void *data)
-{
-  struct icipc_server_priv *priv = icipc_receiver_get_user_data (base);
-
-  icipc_log_info ("server: new state %d on client %d", sender_state, sender_fd);
-
-  pthread_mutex_lock (&priv->mutex);
-  if (priv->client_handler.handler)
-    priv->client_handler.handler ((struct icipc_server *)base, sender_fd,
-        sender_state, priv->client_handler.data);
-  pthread_mutex_unlock (&priv->mutex);
+typedef struct ClientHandler {
+        icipc_server_client_handler_func_t handler;
+        void *data;
+} ClientHandler;
+
+typedef struct RequestHandler {
+        const char *name;
+        icipc_server_request_handler_func_t handler;
+        void *data;
+} RequestHandler;
+
+typedef struct ServerPriv {
+        pthread_mutex_t mutex;
+        ClientHandler client_handler;
+        size_t n_request_handlers;
+        RequestHandler request_handlers[MAX_REQUEST_HANDLERS];
+} ServerPriv;
+
+static void sender_state(
+                struct icipc_receiver *base,
+                int sender_fd,
+                enum icipc_receiver_sender_state sender_state,
+                void *data) {
+        ServerPriv *priv = icipc_receiver_get_user_data(base);
+
+        icipc_log_info("server: new state %d on client %d", sender_state,
+                       sender_fd);
+
+        pthread_mutex_lock(&priv->mutex);
+        if (priv->client_handler.handler)
+                priv->client_handler.handler((struct icipc_server *)base,
+                                             sender_fd, sender_state,
+                                             priv->client_handler.data);
+        pthread_mutex_unlock(&priv->mutex);
 }
 
-static bool
-handle_message (struct icipc_receiver *base,
+static bool handle_message(
+                struct icipc_receiver *base,
                 int sender_fd,
-                const uint8_t *buffer,
+                const uint8_t * buffer,
                 size_t size,
-                void *data)
-{
-  struct icipc_server_priv *priv = icipc_receiver_get_user_data (base);
-  const char *name = NULL;
-  const struct spa_pod *args = NULL;
-
-  icipc_log_info ("server: message from client %d received", sender_fd);
-
-  /* parse */
-  if (!icipc_protocol_parse_request (buffer, size, &name, &args)) {
-    const char *msg = "could not parse request";
-    const size_t s = icipc_protocol_calculate_reply_error_size (msg);
-    uint8_t b[s];
-    icipc_protocol_build_reply_error (b, s, msg);
-    return icipc_socket_write (sender_fd, b, s) == (ssize_t)s;
-  }
-
-  /* handle */
-  size_t i;
-  bool res = false;
-  pthread_mutex_lock (&priv->mutex);
-
-  for (i = 0; i < MAX_REQUEST_HANDLERS; i++) {
-    struct icipc_server_request_handler *rh = priv->request_handlers + i;
-    if (rh->name != NULL && strcmp (rh->name, name) == 0 &&
-        rh->handler != NULL) {
-      res = rh->handler ((struct icipc_server *)base, sender_fd, name, args,
-          rh->data);
-      pthread_mutex_unlock (&priv->mutex);
-      return res;
-    }
-  }
-
-  /* handler was not found, reply with error */
-  res = icipc_server_reply_error ((struct icipc_server *)base, sender_fd,
-      "request handler not found");
-
-  pthread_mutex_unlock (&priv->mutex);
-  return res;
+                void *data) {
+        ServerPriv *priv = icipc_receiver_get_user_data(base);
+        const char *name = NULL;
+        const struct spa_pod *args = NULL;
+
+        icipc_log_info("server: message from client %d received", sender_fd);
+
+        /* parse */
+        if (!icipc_protocol_parse_request(buffer, size, &name, &args)) {
+                const char *msg = "could not parse request";
+                const size_t s = icipc_protocol_calculate_reply_error_size(msg);
+                uint8_t b[s];
+                icipc_protocol_build_reply_error(b, s, msg);
+                return icipc_socket_write(sender_fd, b, s) == (ssize_t) s;
+        }
+
+        /* handle */
+        size_t i;
+        bool res = false;
+        pthread_mutex_lock(&priv->mutex);
+
+        for (i = 0; i < MAX_REQUEST_HANDLERS; i++) {
+                RequestHandler *rh = priv->request_handlers + i;
+                if (rh->name != NULL && strcmp(rh->name, name) == 0
+                    && rh->handler != NULL) {
+                        res =
+                            rh->handler((struct icipc_server *)base, sender_fd,
+                                        name, args, rh->data);
+                        pthread_mutex_unlock(&priv->mutex);
+                        return res;
+                }
+        }
+
+        /* handler was not found, reply with error */
+        res = icipc_server_reply_error((struct icipc_server *)base, sender_fd,
+                                       "request handler not found");
+
+        pthread_mutex_unlock(&priv->mutex);
+        return res;
 }
 
 static struct icipc_receiver_events events = {
-  .sender_state = sender_state,
-  .handle_message = handle_message,
+        .sender_state = sender_state,
+        .handle_message = handle_message,
 };
 
 /* API */
 
-struct icipc_server *
-icipc_server_new (const char *path, bool start)
-{
-  struct icipc_server_priv * priv = NULL;
-  struct icipc_receiver *base = NULL;
+struct icipc_server *icipc_server_new(const char *path, bool start) {
+        ServerPriv *priv = NULL;
+        struct icipc_receiver *base = NULL;
 
-  base = icipc_receiver_new (path, BUFFER_SIZE, &events, NULL,
-      sizeof (struct icipc_server_priv));
-  if (base == NULL)
-    return NULL;
+        base = icipc_receiver_new(path, BUFFER_SIZE, &events, NULL,
+                                  sizeof(ServerPriv));
+        if (base == NULL)
+                return NULL;
 
-  priv = icipc_receiver_get_user_data (base);
-  pthread_mutex_init (&priv->mutex, NULL);
-  priv->n_request_handlers = 0;
+        priv = icipc_receiver_get_user_data(base);
+        pthread_mutex_init(&priv->mutex, NULL);
+        priv->n_request_handlers = 0;
 
-  if (start && !icipc_receiver_start (base)) {
-    icipc_log_error ("failed to start receiver");
-    icipc_server_free ((struct icipc_server *)base);
-    return NULL;
-  }
+        if (start && !icipc_receiver_start(base)) {
+                icipc_log_error("failed to start receiver");
+                icipc_server_free((struct icipc_server *)base);
+                return NULL;
+        }
 
-  return (struct icipc_server *)base;
+        return (struct icipc_server *)base;
 }
 
-void
-icipc_server_free (struct icipc_server *self)
-{
-  struct icipc_receiver *base = icipc_server_to_receiver (self);
-  struct icipc_server_priv *priv = icipc_receiver_get_user_data (base);
+void icipc_server_free(struct icipc_server *self) {
+        struct icipc_receiver *base = icipc_server_to_receiver(self);
+        ServerPriv *priv = icipc_receiver_get_user_data(base);
 
-  pthread_mutex_destroy (&priv->mutex);
+        pthread_mutex_destroy(&priv->mutex);
 
-  icipc_receiver_free (base);
+        icipc_receiver_free(base);
 }
 
-void
-icipc_server_set_client_handler (struct icipc_server *self,
-                                 icipc_server_client_handler_func_t handler,
-                                 void *data)
-{
-  struct icipc_receiver *base = icipc_server_to_receiver (self);
-  struct icipc_server_priv *priv = icipc_receiver_get_user_data (base);
-
-  pthread_mutex_lock (&priv->mutex);
-  priv->client_handler.handler = handler;
-  priv->client_handler.data = data;
-  pthread_mutex_unlock (&priv->mutex);
+void icipc_server_set_client_handler(
+                struct icipc_server *self,
+                icipc_server_client_handler_func_t handler,
+                void *data) {
+        struct icipc_receiver *base = icipc_server_to_receiver(self);
+        ServerPriv *priv = icipc_receiver_get_user_data(base);
+
+        pthread_mutex_lock(&priv->mutex);
+        priv->client_handler.handler = handler;
+        priv->client_handler.data = data;
+        pthread_mutex_unlock(&priv->mutex);
 }
 
-void
-icipc_server_clear_client_handler (struct icipc_server *self)
-{
-  struct icipc_receiver *base = icipc_server_to_receiver (self);
-  struct icipc_server_priv *priv = icipc_receiver_get_user_data (base);
+void icipc_server_clear_client_handler(struct icipc_server *self) {
+        struct icipc_receiver *base = icipc_server_to_receiver(self);
+        ServerPriv *priv = icipc_receiver_get_user_data(base);
 
-  pthread_mutex_lock (&priv->mutex);
-  priv->client_handler.handler = NULL;
-  priv->client_handler.data = NULL;
-  pthread_mutex_unlock (&priv->mutex);
+        pthread_mutex_lock(&priv->mutex);
+        priv->client_handler.handler = NULL;
+        priv->client_handler.data = NULL;
+        pthread_mutex_unlock(&priv->mutex);
 }
 
-bool
-icipc_server_set_request_handler (struct icipc_server *self,
-                                  const char *name,
-                                  icipc_server_request_handler_func_t handler,
-                                  void *data)
-{
-  struct icipc_receiver *base = icipc_server_to_receiver (self);
-  struct icipc_server_priv *priv = icipc_receiver_get_user_data (base);
-  size_t i;
-
-  /* check params */
-  if (name == NULL)
-    return false;
-
-  pthread_mutex_lock (&priv->mutex);
-
-  /* make sure handler does not exist */
-  for (i = 0; i < MAX_REQUEST_HANDLERS; i++) {
-    struct icipc_server_request_handler *rh = priv->request_handlers + i;
-    if (rh->name != NULL && strcmp (rh->name, name) == 0) {
-      pthread_mutex_unlock (&priv->mutex);
-      return false;
-    }
-  }
-
-  /* set handler */
-  for (i = 0; i < MAX_REQUEST_HANDLERS; i++) {
-    struct icipc_server_request_handler *rh = priv->request_handlers + i;
-    if (rh->name == NULL) {
-      rh->name = name;
-      rh->handler = handler;
-      rh->data = data;
-      pthread_mutex_unlock (&priv->mutex);
-      return true;
-    }
-  }
-
-  pthread_mutex_unlock (&priv->mutex);
-
-  return false;
+bool icipc_server_set_request_handler(
+                struct icipc_server *self,
+                const char *name,
+                icipc_server_request_handler_func_t handler,
+                void *data) {
+        struct icipc_receiver *base = icipc_server_to_receiver(self);
+        ServerPriv *priv = icipc_receiver_get_user_data(base);
+        size_t i;
+
+        /* check params */
+        if (name == NULL)
+                return false;
+
+        pthread_mutex_lock(&priv->mutex);
+
+        /* make sure handler does not exist */
+        for (i = 0; i < MAX_REQUEST_HANDLERS; i++) {
+                RequestHandler *rh =
+                    priv->request_handlers + i;
+                if (rh->name != NULL && strcmp(rh->name, name) == 0) {
+                        pthread_mutex_unlock(&priv->mutex);
+                        return false;
+                }
+        }
+
+        /* set handler */
+        for (i = 0; i < MAX_REQUEST_HANDLERS; i++) {
+                RequestHandler *rh =
+                    priv->request_handlers + i;
+                if (rh->name == NULL) {
+                        rh->name = name;
+                        rh->handler = handler;
+                        rh->data = data;
+                        pthread_mutex_unlock(&priv->mutex);
+                        return true;
+                }
+        }
+
+        pthread_mutex_unlock(&priv->mutex);
+
+        return false;
 }
 
-void
-icipc_server_clear_request_handler (struct icipc_server *self,
-                                    const char *name)
-{
-  struct icipc_receiver *base = icipc_server_to_receiver (self);
-  struct icipc_server_priv *priv = icipc_receiver_get_user_data (base);
-  size_t i;
-
-  /* check params */
-  if (name == NULL)
-    return;
-
-  pthread_mutex_lock (&priv->mutex);
-
-  /* clear handler */
-  for (i = 0; i < MAX_REQUEST_HANDLERS; i++) {
-    struct icipc_server_request_handler *rh = priv->request_handlers + i;
-    if (rh->name != NULL && strcmp (rh->name, name) == 0) {
-      rh->name = NULL;
-      break;
-    }
-  }
-
-  pthread_mutex_unlock (&priv->mutex);
+void icipc_server_clear_request_handler(
+                struct icipc_server *self,
+                const char *name) {
+        struct icipc_receiver *base = icipc_server_to_receiver(self);
+        ServerPriv *priv = icipc_receiver_get_user_data(base);
+        size_t i;
+
+        /* check params */
+        if (name == NULL)
+                return;
+
+        pthread_mutex_lock(&priv->mutex);
+
+        /* clear handler */
+        for (i = 0; i < MAX_REQUEST_HANDLERS; i++) {
+                RequestHandler *rh =
+                    priv->request_handlers + i;
+                if (rh->name != NULL && strcmp(rh->name, name) == 0) {
+                        rh->name = NULL;
+                        break;
+                }
+        }
+
+        pthread_mutex_unlock(&priv->mutex);
 }
 
-bool
-icipc_server_reply_ok (struct icipc_server *self,
-                       int client_fd,
-                       const struct spa_pod *value)
-{
-  const size_t s = icipc_protocol_calculate_reply_ok_size (value);
-  uint8_t b[s];
-  icipc_protocol_build_reply_ok (b, s, value);
-  return icipc_socket_write (client_fd, b, s) == (ssize_t)s;
+bool icipc_server_reply_ok(
+                struct icipc_server *self,
+                int client_fd,
+                const struct spa_pod *value) {
+        const size_t s = icipc_protocol_calculate_reply_ok_size(value);
+        uint8_t b[s];
+        icipc_protocol_build_reply_ok(b, s, value);
+        return icipc_socket_write(client_fd, b, s) == (ssize_t) s;
 }
 
-bool
-icipc_server_reply_error (struct icipc_server *self,
-                          int client_fd,
-                          const char *msg)
-{
-  if (msg == NULL)
-    return false;
-
-  const size_t s = icipc_protocol_calculate_reply_error_size (msg);
-  uint8_t b[s];
-  icipc_protocol_build_reply_error (b, s, msg);
-  return icipc_socket_write (client_fd, b, s) == (ssize_t)s;
+bool icipc_server_reply_error(
+                struct icipc_server *self,
+                int client_fd,
+                const char *msg) {
+        if (msg == NULL)
+                return false;
+
+        const size_t s = icipc_protocol_calculate_reply_error_size(msg);
+        uint8_t b[s];
+        icipc_protocol_build_reply_error(b, s, msg);
+        return icipc_socket_write(client_fd, b, s) == (ssize_t) s;
 }
diff --git a/lib/server.h b/lib/server.h
index 3cc0ed9..6345035 100644
--- a/lib/server.h
+++ b/lib/server.h
@@ -12,7 +12,6 @@
 #include <spa/pod/pod.h>
 
 #include "defs.h"
-
 #include "receiver.h"
 
 #ifdef __cplusplus
@@ -23,60 +22,59 @@ extern "C" {
 
 struct icipc_server;
 
-typedef void (*icipc_server_client_handler_func_t) (struct icipc_server *self,
-                                                    int client_fd,
-                                                    enum icipc_receiver_sender_state client_state,
-                                                    void *data);
+typedef void (*icipc_server_client_handler_func_t)(
+                struct icipc_server *self,
+                int client_fd,
+                enum icipc_receiver_sender_state client_state,
+                void *data);
 
-typedef bool (*icipc_server_request_handler_func_t) (struct icipc_server *self,
-                                                     int client_fd,
-                                                     const char *name,
-                                                     const struct spa_pod *args,
-                                                     void *data);
+typedef bool (*icipc_server_request_handler_func_t)(
+                struct icipc_server *self,
+                int client_fd,
+                const char *name,
+                const struct spa_pod *args,
+                void *data);
 
 ICIPC_API
-struct icipc_server *
-icipc_server_new (const char *path, bool start);
+struct icipc_server *icipc_server_new(const char *path, bool start);
 
 ICIPC_API
-void
-icipc_server_free (struct icipc_server *self);
+void icipc_server_free(struct icipc_server *self);
 
 ICIPC_API
-void
-icipc_server_set_client_handler (struct icipc_server *self,
-                                 icipc_server_client_handler_func_t handler,
-                                 void *data);
+void icipc_server_set_client_handler(
+                struct icipc_server *self,
+                icipc_server_client_handler_func_t handler,
+                void *data);
 
 ICIPC_API
-void
-icipc_server_clear_client_handler (struct icipc_server *self);
+void icipc_server_clear_client_handler(struct icipc_server *self);
 
 ICIPC_API
-bool
-icipc_server_set_request_handler (struct icipc_server *self,
-                                  const char *name,
-                                  icipc_server_request_handler_func_t handler,
-                                  void *data);
+bool icipc_server_set_request_handler(
+                struct icipc_server *self,
+                const char *name,
+                icipc_server_request_handler_func_t handler,
+                void *data);
 
 ICIPC_API
-void
-icipc_server_clear_request_handler (struct icipc_server *self,
-                                    const char *name);
+void icipc_server_clear_request_handler(
+                struct icipc_server *self,
+                const char *name);
 
 /* for request handlers only */
 
 ICIPC_API
-bool
-icipc_server_reply_ok (struct icipc_server *self,
-                       int client_fd,
-                       const struct spa_pod *value);
+bool icipc_server_reply_ok(
+                struct icipc_server *self,
+                int client_fd,
+                const struct spa_pod *value);
 
 ICIPC_API
-bool
-icipc_server_reply_error (struct icipc_server *self,
-                          int client_fd,
-                          const char *msg);
+bool icipc_server_reply_error(
+                struct icipc_server *self,
+                int client_fd,
+                const char *msg);
 
 #ifdef __cplusplus
 }
diff --git a/lib/utils.c b/lib/utils.c
index fd9cdff..4d91241 100644
--- a/lib/utils.c
+++ b/lib/utils.c
@@ -28,284 +28,273 @@
 /* log */
 
 const char *icipc_logger_level_text[] = {
-  [ICIPC_LOG_LEVEL_ERROR] = "E",
-  [ICIPC_LOG_LEVEL_WARN] = "W",
-  [ICIPC_LOG_LEVEL_INFO] = "I",
+        [ICIPC_LOG_LEVEL_ERROR] = "E",
+        [ICIPC_LOG_LEVEL_WARN] = "W",
+        [ICIPC_LOG_LEVEL_INFO] = "I",
 };
 
-struct icipc_logger {
-  enum icipc_log_level level;
-};
+typedef struct Logger {
+        LogLevel level;
+} Logger;
 
-static const struct icipc_logger *
-icipc_log_get_instance (void)
-{
-  static struct icipc_logger logger_ = { 0, };
-  static struct icipc_logger* instance_ = NULL;
+static const Logger *icipc_log_get_instance(void) {
+        static Logger logger_ = { 0, };
+        static Logger *instance_ = NULL;
 
-  if (instance_ == NULL) {
-    char * val_str = NULL;
-    enum icipc_log_level val = 0;
+        if (instance_ == NULL) {
+                char *val_str = NULL;
+                LogLevel val = 0;
 
-    /* default to error */
-    logger_.level = ICIPC_LOG_LEVEL_WARN;
+                /* default to error */
+                logger_.level = ICIPC_LOG_LEVEL_WARN;
 
-    /* get level from env */
-    val_str = getenv ("ICIPC_DEBUG");
-    if (val_str && sscanf (val_str, "%u", &val) == 1 &&
-        val >= ICIPC_LOG_LEVEL_NONE)
-      logger_.level = val;
+                /* get level from env */
+                val_str = getenv("ICIPC_DEBUG");
+                if (val_str && sscanf(val_str, "%u", &val) == 1 &&
+                    val >= ICIPC_LOG_LEVEL_NONE)
+                        logger_.level = val;
 
-    instance_ = &logger_;
-  }
+                instance_ = &logger_;
+        }
 
-  return instance_;
+        return instance_;
 }
 
-void
-icipc_logv (enum icipc_log_level level, const char *fmt, va_list args)
-{
-  const struct icipc_logger *logger = NULL;
-
-  logger = icipc_log_get_instance ();
-  assert (logger);
-
-  if (logger->level >= level) {
-    assert (level > 0);
-    char msg[MAX_LOG_MESSAGE];
-    struct timespec time;
-    clock_gettime (CLOCK_REALTIME, &time);
-    vsnprintf (msg, MAX_LOG_MESSAGE, fmt, args);
-    fprintf (stderr, "[%s][%lu.%lu] %s\n", icipc_logger_level_text[level],
-        time.tv_sec, time.tv_sec, msg);
-  }
+void icipc_logv(LogLevel level, const char *fmt, va_list args) {
+        const Logger *logger = NULL;
+
+        logger = icipc_log_get_instance();
+        assert(logger);
+
+        if (logger->level >= level) {
+                assert(level > 0);
+                char msg[MAX_LOG_MESSAGE];
+                struct timespec time;
+                clock_gettime(CLOCK_REALTIME, &time);
+                vsnprintf(msg, MAX_LOG_MESSAGE, fmt, args);
+                fprintf(stderr, "[%s][%lu.%lu] %s\n",
+                        icipc_logger_level_text[level], time.tv_sec,
+                        time.tv_sec, msg);
+        }
 }
 
-void
-icipc_log (enum icipc_log_level level, const char *fmt, ...)
-{
-  va_list args;
-  va_start (args, fmt);
-  icipc_logv (level, fmt, args);
-  va_end (args);
+void icipc_log(LogLevel level, const char *fmt, ...) {
+        va_list args;
+        va_start(args, fmt);
+        icipc_logv(level, fmt, args);
+        va_end(args);
 }
 
 /* socket path */
 
-int
-icipc_construct_socket_path (const char *name, char *buf, size_t buf_size)
-{
-  bool path_is_absolute;
-  const char *runtime_dir = NULL;
-  struct passwd pwd, *result = NULL;
-  char buffer[4096];
-  int name_size;
-
-  path_is_absolute = name[0] == '/';
-
-  if (!path_is_absolute) {
-    runtime_dir = getenv("PIPEWIRE_RUNTIME_DIR");
-    if (runtime_dir == NULL)
-      runtime_dir = getenv("XDG_RUNTIME_DIR");
-    if (runtime_dir == NULL)
-      runtime_dir = getenv("HOME");
-    if (runtime_dir == NULL)
-      runtime_dir = getenv("USERPROFILE");
-    if (runtime_dir == NULL) {
-      if (getpwuid_r(getuid(), &pwd, buffer, sizeof(buffer), &result) == 0)
-        runtime_dir = result ? result->pw_dir : NULL;
-    }
-  }
-
-  if (runtime_dir == NULL && !path_is_absolute)
-    return -ENOENT;
-
-  if (path_is_absolute)
-    name_size = snprintf (buf, buf_size, "%s", name) + 1;
-  else
-    name_size = snprintf (buf, buf_size, "%s/%s", runtime_dir, name) + 1;
-
-  if (name_size > (int) buf_size)
-    return -ENAMETOOLONG;
-
-  return 0;
+int icipc_construct_socket_path(const char *name, char *buf, size_t buf_size) {
+        bool path_is_absolute;
+        const char *runtime_dir = NULL;
+        struct passwd pwd, *result = NULL;
+        char buffer[4096];
+        int name_size;
+
+        path_is_absolute = name[0] == '/';
+
+        if (!path_is_absolute) {
+                runtime_dir = getenv("PIPEWIRE_RUNTIME_DIR");
+                if (runtime_dir == NULL)
+                        runtime_dir = getenv("XDG_RUNTIME_DIR");
+                if (runtime_dir == NULL)
+                        runtime_dir = getenv("HOME");
+                if (runtime_dir == NULL)
+                        runtime_dir = getenv("USERPROFILE");
+                if (runtime_dir == NULL) {
+                        if (getpwuid_r
+                            (getuid(), &pwd, buffer, sizeof(buffer),
+                             &result) == 0)
+                                runtime_dir = result ? result->pw_dir : NULL;
+                }
+        }
+
+        if (runtime_dir == NULL && !path_is_absolute)
+                return -ENOENT;
+
+        if (path_is_absolute)
+                name_size = snprintf(buf, buf_size, "%s", name) + 1;
+        else
+                name_size =
+                    snprintf(buf, buf_size, "%s/%s", runtime_dir, name) + 1;
+
+        if (name_size > (int)buf_size)
+                return -ENAMETOOLONG;
+
+        return 0;
 }
 
 /* socket */
 
-ssize_t
-icipc_socket_write (int fd, const uint8_t *buffer, size_t size)
-{
-  size_t total_written = 0;
-  size_t n;
-
-  assert (fd >= 0);
-  assert (buffer != NULL);
-  assert (size > 0);
-
-  do {
-    n = write(fd, buffer, size);
-    if (n < size) {
-      if (errno == EINTR)
-        continue;
-      if (errno == EAGAIN || errno == EWOULDBLOCK)
-        return total_written;
-      return -1;
-    }
-    total_written += n;
-  } while (total_written < size);
+ssize_t icipc_socket_write(int fd, const uint8_t * buffer, size_t size) {
+        size_t total_written = 0;
+        size_t n;
+
+        assert(fd >= 0);
+        assert(buffer != NULL);
+        assert(size > 0);
+
+        do {
+                n = write(fd, buffer, size);
+                if (n < size) {
+                        if (errno == EINTR)
+                                continue;
+                        if (errno == EAGAIN || errno == EWOULDBLOCK)
+                                return total_written;
+                        return -1;
+                }
+                total_written += n;
+        } while (total_written < size);
 
-  return total_written;
+        return total_written;
 }
 
-ssize_t
-icipc_socket_read (int fd, uint8_t **buffer, size_t *max_size)
-{
-  ssize_t n;
-  ssize_t size;
-  size_t offset = 0;
-
-  assert (buffer);
-  assert (*buffer);
-  assert (max_size);
-  assert (*max_size > 0);
-
-again:
-  size = *max_size - offset;
-  n = read (fd, *buffer + offset, size);
-  if (n == 0)
-    return 0;
-
-  /* check for errors */
-  if (n < 0) {
-    if (errno == EINTR)
-      goto again;
-    if (errno == EAGAIN || errno == EWOULDBLOCK)
-      return offset;
-    return -1;
-  }
-
-  /* realloc if we need more space, and read again */
-  if (n >= size) {
-    *max_size += *max_size;
-    *buffer = reallocarray (*buffer, *max_size, sizeof (uint8_t));
-    offset += n;
-    goto again;
-  }
-
-  return offset + n;
+ssize_t icipc_socket_read(int fd, uint8_t ** buffer, size_t *max_size) {
+        ssize_t n;
+        ssize_t size;
+        size_t offset = 0;
+
+        assert(buffer);
+        assert(*buffer);
+        assert(max_size);
+        assert(*max_size > 0);
+
+ again:
+        size = *max_size - offset;
+        n = read(fd, *buffer + offset, size);
+        if (n == 0)
+                return 0;
+
+        /* check for errors */
+        if (n < 0) {
+                if (errno == EINTR)
+                        goto again;
+                if (errno == EAGAIN || errno == EWOULDBLOCK)
+                        return offset;
+                return -1;
+        }
+
+        /* realloc if we need more space, and read again */
+        if (n >= size) {
+                *max_size += *max_size;
+                *buffer = reallocarray(*buffer, *max_size, sizeof(uint8_t));
+                offset += n;
+                goto again;
+        }
+
+        return offset + n;
 }
 
 /* epoll thread */
 
-bool
-icipc_epoll_thread_init (struct epoll_thread *self,
-                         int socket_fd,
-                         icipc_epoll_thread_event_func_t sock_func,
-                         icipc_epoll_thread_event_func_t other_func,
-                         void *data)
-{
-  struct epoll_event event;
-
-  self->socket_fd = socket_fd;
-  self->event_fd = -1;
-  self->epoll_fd = -1;
-
-  /* create event fd */
-  self->event_fd = eventfd (0, EFD_CLOEXEC | EFD_NONBLOCK);
-  if (self->event_fd == -1)
-    goto error;
-
-  /* create epoll fd */
-  self->epoll_fd = epoll_create1 (EPOLL_CLOEXEC);
-  if (self->epoll_fd == -1)
-    goto error;
-
-  /* poll socket fd */
-  event.events = EPOLLIN;
-  event.data.fd = self->socket_fd;
-  if (epoll_ctl (self->epoll_fd, EPOLL_CTL_ADD, self->socket_fd, &event) != 0)
-    goto error;
-
-  /* poll event fd */
-  event.events = EPOLLIN;
-  event.data.fd = self->event_fd;
-  if (epoll_ctl (self->epoll_fd, EPOLL_CTL_ADD, self->event_fd, &event) != 0)
-    goto error;
-
-  self->socket_event_func = sock_func;
-  self->other_event_func = other_func;
-  self->event_data = data;
-  return true;
-
-error:
-  if (self->epoll_fd != -1)
-    close (self->epoll_fd);
-  if (self->event_fd != -1)
-    close (self->event_fd);
-  return false;
+bool icipc_epoll_thread_init(
+                EpollThread *self,
+                int socket_fd,
+                icipc_epoll_thread_event_func_t sock_func,
+                icipc_epoll_thread_event_func_t other_func,
+                void *data) {
+        struct epoll_event event;
+
+        self->socket_fd = socket_fd;
+        self->event_fd = -1;
+        self->epoll_fd = -1;
+
+        /* create event fd */
+        self->event_fd = eventfd(0, EFD_CLOEXEC | EFD_NONBLOCK);
+        if (self->event_fd == -1)
+                goto error;
+
+        /* create epoll fd */
+        self->epoll_fd = epoll_create1(EPOLL_CLOEXEC);
+        if (self->epoll_fd == -1)
+                goto error;
+
+        /* poll socket fd */
+        event.events = EPOLLIN;
+        event.data.fd = self->socket_fd;
+        if (epoll_ctl(self->epoll_fd, EPOLL_CTL_ADD, self->socket_fd, &event) != 0)
+                goto error;
+
+        /* poll event fd */
+        event.events = EPOLLIN;
+        event.data.fd = self->event_fd;
+        if (epoll_ctl(self->epoll_fd, EPOLL_CTL_ADD, self->event_fd, &event) != 0)
+                goto error;
+
+        self->socket_event_func = sock_func;
+        self->other_event_func = other_func;
+        self->event_data = data;
+        return true;
+
+ error:
+        if (self->epoll_fd != -1)
+                close(self->epoll_fd);
+        if (self->event_fd != -1)
+                close(self->event_fd);
+        return false;
 }
 
-static void *
-epoll_thread_run (void *data)
-{
-  struct epoll_thread *self = data;
-  bool exit = false;
-
-  while (!exit) {
-    /* wait for events */
-    struct epoll_event ep[MAX_POLL_EVENTS];
-    int n = epoll_wait (self->epoll_fd, ep, MAX_POLL_EVENTS, -1);
-    if (n < 0) {
-      icipc_log_error ("epoll_thread: failed to wait for event: %s",
-          strerror(errno));
-      continue;
-    }
-
-    for (int i = 0; i < n; i++) {
-      /* socket fd */
-      if (ep[i].data.fd == self->socket_fd) {
-        if (self->socket_event_func)
-          self->socket_event_func (self, ep[i].data.fd, self->event_data);
-      }
-
-      /* event fd */
-      else if (ep[i].data.fd == self->event_fd) {
-        uint64_t stop = 0;
-        ssize_t res = read (ep[i].data.fd, &stop, sizeof(uint64_t));
-        if (res == sizeof(uint64_t) && stop == 1)
-          exit = true;
-      }
-
-      /* other */
-      else {
-        if (self->other_event_func)
-          self->other_event_func (self, ep[i].data.fd, self->event_data);
-      }
-    }
-  }
-
-  return NULL;
+static void *epoll_thread_run(void *data) {
+        EpollThread *self = data;
+        bool exit = false;
+
+        while (!exit) {
+                /* wait for events */
+                struct epoll_event ep[MAX_POLL_EVENTS];
+                int n = epoll_wait(self->epoll_fd, ep, MAX_POLL_EVENTS, -1);
+                if (n < 0) {
+                        icipc_log_error
+                            ("epoll_thread: failed to wait for event: %s",
+                             strerror(errno));
+                        continue;
+                }
+
+                for (int i = 0; i < n; i++) {
+                        /* socket fd */
+                        if (ep[i].data.fd == self->socket_fd) {
+                                if (self->socket_event_func)
+                                        self->socket_event_func(self,
+                                                                ep[i].data.fd,
+                                                                self->event_data);
+                        }
+
+                        /* event fd */
+                        else if (ep[i].data.fd == self->event_fd) {
+                                uint64_t stop = 0;
+                                ssize_t res = read(ep[i].data.fd, &stop,
+                                                   sizeof(uint64_t));
+                                if (res == sizeof(uint64_t) && stop == 1)
+                                        exit = true;
+                        }
+
+                        /* other */
+                        else {
+                                if (self->other_event_func)
+                                        self->other_event_func(self,
+                                                               ep[i].data.fd,
+                                                               self->event_data);
+                        }
+                }
+        }
+
+        return NULL;
 }
 
-bool
-icipc_epoll_thread_start (struct epoll_thread *self)
-{
-  return pthread_create (&self->thread, NULL, epoll_thread_run, self) == 0;
+bool icipc_epoll_thread_start(EpollThread *self) {
+        return pthread_create(&self->thread, NULL, epoll_thread_run, self) == 0;
 }
 
-void
-icipc_epoll_thread_stop (struct epoll_thread *self)
-{
-  uint64_t value = 1;
-  ssize_t res = write (self->event_fd, &value, sizeof(uint64_t));
-  if (res == sizeof(uint64_t))
-    pthread_join (self->thread, NULL);
+void icipc_epoll_thread_stop(EpollThread *self) {
+        uint64_t value = 1;
+        ssize_t res = write(self->event_fd, &value, sizeof(uint64_t));
+        if (res == sizeof(uint64_t))
+                pthread_join(self->thread, NULL);
 }
 
-void
-icipc_epoll_thread_destroy (struct epoll_thread *self)
-{
-  close (self->epoll_fd);
-  close (self->event_fd);
+void icipc_epoll_thread_destroy(EpollThread *self) {
+        close(self->epoll_fd);
+        close(self->event_fd);
 }