clang-format all core files

1 files changed, 280 insertions, 347 deletions

diff --git a/src/core/client_config/resolvers/zookeeper_resolver.c b/src/core/client_config/resolvers/zookeeper_resolver.c
index 3e2040c412..213d5a172f 100644
--- a/src/core/client_config/resolvers/zookeeper_resolver.c
+++ b/src/core/client_config/resolvers/zookeeper_resolver.c
@@ -50,8 +50,7 @@
 /** Zookeeper session expiration time in milliseconds */
 #define GRPC_ZOOKEEPER_SESSION_TIMEOUT 15000
 
-typedef struct
-{
+typedef struct {
   /** base class: must be first */
   grpc_resolver base;
   /** refcount */
@@ -88,119 +87,101 @@ typedef struct
   int resolved_num;
 } zookeeper_resolver;
 
-static void zookeeper_destroy (grpc_resolver * r);
+static void zookeeper_destroy(grpc_resolver *r);
 
-static void zookeeper_start_resolving_locked (zookeeper_resolver * r);
-static grpc_closure *
-zookeeper_maybe_finish_next_locked (zookeeper_resolver * r)
-  GRPC_MUST_USE_RESULT;
+static void zookeeper_start_resolving_locked(zookeeper_resolver *r);
+static grpc_closure *zookeeper_maybe_finish_next_locked(zookeeper_resolver *r)
+    GRPC_MUST_USE_RESULT;
 
-     static void zookeeper_shutdown (grpc_resolver * r);
-     static void zookeeper_channel_saw_error (grpc_resolver * r, struct sockaddr *failing_address, int failing_address_len);
-     static void zookeeper_next (grpc_resolver * r, grpc_client_config ** target_config, grpc_closure * on_complete);
+static void zookeeper_shutdown(grpc_resolver *r);
+static void zookeeper_channel_saw_error(grpc_resolver *r,
+                                        struct sockaddr *failing_address,
+                                        int failing_address_len);
+static void zookeeper_next(grpc_resolver *r, grpc_client_config **target_config,
+                           grpc_closure *on_complete);
 
-     static const grpc_resolver_vtable zookeeper_resolver_vtable = {
-       zookeeper_destroy, zookeeper_shutdown, zookeeper_channel_saw_error,
-       zookeeper_next
-     };
+static const grpc_resolver_vtable zookeeper_resolver_vtable = {
+    zookeeper_destroy, zookeeper_shutdown, zookeeper_channel_saw_error,
+    zookeeper_next};
 
-static void
-zookeeper_shutdown (grpc_resolver * resolver)
-{
-  zookeeper_resolver *r = (zookeeper_resolver *) resolver;
+static void zookeeper_shutdown(grpc_resolver *resolver) {
+  zookeeper_resolver *r = (zookeeper_resolver *)resolver;
   grpc_closure *call = NULL;
-  gpr_mu_lock (&r->mu);
-  if (r->next_completion != NULL)
-    {
-      *r->target_config = NULL;
-      call = r->next_completion;
-      r->next_completion = NULL;
-    }
-  zookeeper_close (r->zookeeper_handle);
-  gpr_mu_unlock (&r->mu);
-  if (call != NULL)
-    {
-      call->cb (call->cb_arg, 1);
-    }
+  gpr_mu_lock(&r->mu);
+  if (r->next_completion != NULL) {
+    *r->target_config = NULL;
+    call = r->next_completion;
+    r->next_completion = NULL;
+  }
+  zookeeper_close(r->zookeeper_handle);
+  gpr_mu_unlock(&r->mu);
+  if (call != NULL) {
+    call->cb(call->cb_arg, 1);
+  }
 }
 
-static void
-zookeeper_channel_saw_error (grpc_resolver * resolver, struct sockaddr *sa, int len)
-{
-  zookeeper_resolver *r = (zookeeper_resolver *) resolver;
-  gpr_mu_lock (&r->mu);
-  if (r->resolving == 0)
-    {
-      zookeeper_start_resolving_locked (r);
-    }
-  gpr_mu_unlock (&r->mu);
+static void zookeeper_channel_saw_error(grpc_resolver *resolver,
+                                        struct sockaddr *sa, int len) {
+  zookeeper_resolver *r = (zookeeper_resolver *)resolver;
+  gpr_mu_lock(&r->mu);
+  if (r->resolving == 0) {
+    zookeeper_start_resolving_locked(r);
+  }
+  gpr_mu_unlock(&r->mu);
 }
 
-static void
-zookeeper_next (grpc_resolver * resolver, grpc_client_config ** target_config, grpc_closure * on_complete)
-{
-  zookeeper_resolver *r = (zookeeper_resolver *) resolver;
+static void zookeeper_next(grpc_resolver *resolver,
+                           grpc_client_config **target_config,
+                           grpc_closure *on_complete) {
+  zookeeper_resolver *r = (zookeeper_resolver *)resolver;
   grpc_closure *call;
-  gpr_mu_lock (&r->mu);
-  GPR_ASSERT (r->next_completion == NULL);
+  gpr_mu_lock(&r->mu);
+  GPR_ASSERT(r->next_completion == NULL);
   r->next_completion = on_complete;
   r->target_config = target_config;
-  if (r->resolved_version == 0 && r->resolving == 0)
-    {
-      zookeeper_start_resolving_locked (r);
-    }
-  else
-    {
-      call = zookeeper_maybe_finish_next_locked (r);
-    }
-  gpr_mu_unlock (&r->mu);
-  if (call)
-    call->cb (call->cb_arg, 1);
+  if (r->resolved_version == 0 && r->resolving == 0) {
+    zookeeper_start_resolving_locked(r);
+  } else {
+    call = zookeeper_maybe_finish_next_locked(r);
+  }
+  gpr_mu_unlock(&r->mu);
+  if (call) call->cb(call->cb_arg, 1);
 }
 
 /** Zookeeper global watcher for connection management
     TODO: better connection management besides logs */
-static void
-zookeeper_global_watcher (zhandle_t * zookeeper_handle, int type, int state, const char *path, void *watcher_ctx)
-{
-  if (type == ZOO_SESSION_EVENT)
-    {
-      if (state == ZOO_EXPIRED_SESSION_STATE)
-	{
-	  gpr_log (GPR_ERROR, "Zookeeper session expired");
-	}
-      else if (state == ZOO_AUTH_FAILED_STATE)
-	{
-	  gpr_log (GPR_ERROR, "Zookeeper authentication failed");
-	}
+static void zookeeper_global_watcher(zhandle_t *zookeeper_handle, int type,
+                                     int state, const char *path,
+                                     void *watcher_ctx) {
+  if (type == ZOO_SESSION_EVENT) {
+    if (state == ZOO_EXPIRED_SESSION_STATE) {
+      gpr_log(GPR_ERROR, "Zookeeper session expired");
+    } else if (state == ZOO_AUTH_FAILED_STATE) {
+      gpr_log(GPR_ERROR, "Zookeeper authentication failed");
     }
+  }
 }
 
 /** Zookeeper watcher triggered by changes to watched nodes
     Once triggered, it tries to resolve again to get updated addresses */
-static void
-zookeeper_watcher (zhandle_t * zookeeper_handle, int type, int state, const char *path, void *watcher_ctx)
-{
-  if (watcher_ctx != NULL)
-    {
-      zookeeper_resolver *r = (zookeeper_resolver *) watcher_ctx;
-      if (state == ZOO_CONNECTED_STATE)
-	{
-	  gpr_mu_lock (&r->mu);
-	  if (r->resolving == 0)
-	    {
-	      zookeeper_start_resolving_locked (r);
-	    }
-	  gpr_mu_unlock (&r->mu);
-	}
+static void zookeeper_watcher(zhandle_t *zookeeper_handle, int type, int state,
+                              const char *path, void *watcher_ctx) {
+  if (watcher_ctx != NULL) {
+    zookeeper_resolver *r = (zookeeper_resolver *)watcher_ctx;
+    if (state == ZOO_CONNECTED_STATE) {
+      gpr_mu_lock(&r->mu);
+      if (r->resolving == 0) {
+        zookeeper_start_resolving_locked(r);
+      }
+      gpr_mu_unlock(&r->mu);
     }
+  }
 }
 
 /** Callback function after getting all resolved addresses
     Creates a subchannel for each address */
-static void
-zookeeper_on_resolved (void *arg, grpc_resolved_addresses * addresses)
-{
+static void zookeeper_on_resolved(void *arg,
+                                  grpc_resolved_addresses *addresses) {
   zookeeper_resolver *r = arg;
   grpc_client_config *config = NULL;
   grpc_subchannel **subchannels;
@@ -208,77 +189,74 @@ zookeeper_on_resolved (void *arg, grpc_resolved_addresses * addresses)
   grpc_lb_policy *lb_policy;
   grpc_closure *call;
   size_t i;
-  if (addresses != NULL)
-    {
-      grpc_lb_policy_args lb_policy_args;
-      config = grpc_client_config_create ();
-      subchannels = gpr_malloc (sizeof (grpc_subchannel *) * addresses->naddrs);
-      for (i = 0; i < addresses->naddrs; i++)
-	{
-	  memset (&args, 0, sizeof (args));
-	  args.addr = (struct sockaddr *) (addresses->addrs[i].addr);
-	  args.addr_len = addresses->addrs[i].len;
-	  subchannels[i] = grpc_subchannel_factory_create_subchannel (r->subchannel_factory, &args);
-	}
-      lb_policy_args.subchannels = subchannels;
-      lb_policy_args.num_subchannels = addresses->naddrs;
-      lb_policy = grpc_lb_policy_create (r->lb_policy_name, &lb_policy_args);
-      grpc_client_config_set_lb_policy (config, lb_policy);
-      GRPC_LB_POLICY_UNREF (lb_policy, "construction");
-      grpc_resolved_addresses_destroy (addresses);
-      gpr_free (subchannels);
+  if (addresses != NULL) {
+    grpc_lb_policy_args lb_policy_args;
+    config = grpc_client_config_create();
+    subchannels = gpr_malloc(sizeof(grpc_subchannel *) * addresses->naddrs);
+    for (i = 0; i < addresses->naddrs; i++) {
+      memset(&args, 0, sizeof(args));
+      args.addr = (struct sockaddr *)(addresses->addrs[i].addr);
+      args.addr_len = addresses->addrs[i].len;
+      subchannels[i] = grpc_subchannel_factory_create_subchannel(
+          r->subchannel_factory, &args);
     }
-  gpr_mu_lock (&r->mu);
-  GPR_ASSERT (r->resolving == 1);
+    lb_policy_args.subchannels = subchannels;
+    lb_policy_args.num_subchannels = addresses->naddrs;
+    lb_policy = grpc_lb_policy_create(r->lb_policy_name, &lb_policy_args);
+    grpc_client_config_set_lb_policy(config, lb_policy);
+    GRPC_LB_POLICY_UNREF(lb_policy, "construction");
+    grpc_resolved_addresses_destroy(addresses);
+    gpr_free(subchannels);
+  }
+  gpr_mu_lock(&r->mu);
+  GPR_ASSERT(r->resolving == 1);
   r->resolving = 0;
-  if (r->resolved_config != NULL)
-    {
-      grpc_client_config_unref (r->resolved_config);
-    }
+  if (r->resolved_config != NULL) {
+    grpc_client_config_unref(r->resolved_config);
+  }
   r->resolved_config = config;
   r->resolved_version++;
-  call = zookeeper_maybe_finish_next_locked (r);
-  gpr_mu_unlock (&r->mu);
+  call = zookeeper_maybe_finish_next_locked(r);
+  gpr_mu_unlock(&r->mu);
 
-  if (call)
-    call->cb (call->cb_arg, 1);
+  if (call) call->cb(call->cb_arg, 1);
 
-  GRPC_RESOLVER_UNREF (&r->base, "zookeeper-resolving");
+  GRPC_RESOLVER_UNREF(&r->base, "zookeeper-resolving");
 }
 
 /** Callback function for each DNS resolved address */
-static void
-zookeeper_dns_resolved (void *arg, grpc_resolved_addresses * addresses)
-{
+static void zookeeper_dns_resolved(void *arg,
+                                   grpc_resolved_addresses *addresses) {
   size_t i;
   zookeeper_resolver *r = arg;
   int resolve_done = 0;
 
-  gpr_mu_lock (&r->mu);
+  gpr_mu_lock(&r->mu);
   r->resolved_num++;
-  r->resolved_addrs->addrs = gpr_realloc (r->resolved_addrs->addrs, sizeof (grpc_resolved_address) * (r->resolved_addrs->naddrs + addresses->naddrs));
-  for (i = 0; i < addresses->naddrs; i++)
-    {
-      memcpy (r->resolved_addrs->addrs[i + r->resolved_addrs->naddrs].addr, addresses->addrs[i].addr, addresses->addrs[i].len);
-      r->resolved_addrs->addrs[i + r->resolved_addrs->naddrs].len = addresses->addrs[i].len;
-    }
+  r->resolved_addrs->addrs =
+      gpr_realloc(r->resolved_addrs->addrs,
+                  sizeof(grpc_resolved_address) *
+                      (r->resolved_addrs->naddrs + addresses->naddrs));
+  for (i = 0; i < addresses->naddrs; i++) {
+    memcpy(r->resolved_addrs->addrs[i + r->resolved_addrs->naddrs].addr,
+           addresses->addrs[i].addr, addresses->addrs[i].len);
+    r->resolved_addrs->addrs[i + r->resolved_addrs->naddrs].len =
+        addresses->addrs[i].len;
+  }
 
   r->resolved_addrs->naddrs += addresses->naddrs;
-  grpc_resolved_addresses_destroy (addresses);
+  grpc_resolved_addresses_destroy(addresses);
 
   /** Wait for all addresses to be resolved */
   resolve_done = (r->resolved_num == r->resolved_total);
-  gpr_mu_unlock (&r->mu);
-  if (resolve_done)
-    {
-      zookeeper_on_resolved (r, r->resolved_addrs);
-    }
+  gpr_mu_unlock(&r->mu);
+  if (resolve_done) {
+    zookeeper_on_resolved(r, r->resolved_addrs);
+  }
 }
 
 /** Parses JSON format address of a zookeeper node */
-static char *
-zookeeper_parse_address (const char *value, size_t value_len)
-{
+static char *zookeeper_parse_address(const char *value, size_t value_len) {
   grpc_json *json;
   grpc_json *cur;
   const char *host;
@@ -286,301 +264,256 @@ zookeeper_parse_address (const char *value, size_t value_len)
   char *buffer;
   char *address = NULL;
 
-  buffer = gpr_malloc (value_len);
-  memcpy (buffer, value, value_len);
-  json = grpc_json_parse_string_with_len (buffer, value_len);
-  if (json != NULL)
-    {
-      host = NULL;
-      port = NULL;
-      for (cur = json->child; cur != NULL; cur = cur->next)
-	{
-	  if (!strcmp (cur->key, "host"))
-	    {
-	      host = cur->value;
-	      if (port != NULL)
-		{
-		  break;
-		}
-	    }
-	  else if (!strcmp (cur->key, "port"))
-	    {
-	      port = cur->value;
-	      if (host != NULL)
-		{
-		  break;
-		}
-	    }
-	}
-      if (host != NULL && port != NULL)
-	{
-	  gpr_asprintf (&address, "%s:%s", host, port);
-	}
-      grpc_json_destroy (json);
+  buffer = gpr_malloc(value_len);
+  memcpy(buffer, value, value_len);
+  json = grpc_json_parse_string_with_len(buffer, value_len);
+  if (json != NULL) {
+    host = NULL;
+    port = NULL;
+    for (cur = json->child; cur != NULL; cur = cur->next) {
+      if (!strcmp(cur->key, "host")) {
+        host = cur->value;
+        if (port != NULL) {
+          break;
+        }
+      } else if (!strcmp(cur->key, "port")) {
+        port = cur->value;
+        if (host != NULL) {
+          break;
+        }
+      }
+    }
+    if (host != NULL && port != NULL) {
+      gpr_asprintf(&address, "%s:%s", host, port);
     }
-  gpr_free (buffer);
+    grpc_json_destroy(json);
+  }
+  gpr_free(buffer);
 
   return address;
 }
 
-static void
-zookeeper_get_children_node_completion (int rc, const char *value, int value_len, const struct Stat *stat, const void *arg)
-{
+static void zookeeper_get_children_node_completion(int rc, const char *value,
+                                                   int value_len,
+                                                   const struct Stat *stat,
+                                                   const void *arg) {
   char *address = NULL;
-  zookeeper_resolver *r = (zookeeper_resolver *) arg;
+  zookeeper_resolver *r = (zookeeper_resolver *)arg;
   int resolve_done = 0;
 
-  if (rc != 0)
-    {
-      gpr_log (GPR_ERROR, "Error in getting a child node of %s", r->name);
-      return;
-    }
+  if (rc != 0) {
+    gpr_log(GPR_ERROR, "Error in getting a child node of %s", r->name);
+    return;
+  }
 
-  address = zookeeper_parse_address (value, (size_t) value_len);
-  if (address != NULL)
-    {
+  address = zookeeper_parse_address(value, (size_t)value_len);
+  if (address != NULL) {
     /** Further resolves address by DNS */
-      grpc_resolve_address (address, NULL, zookeeper_dns_resolved, r);
-      gpr_free (address);
-    }
-  else
-    {
-      gpr_log (GPR_ERROR, "Error in resolving a child node of %s", r->name);
-      gpr_mu_lock (&r->mu);
-      r->resolved_total--;
-      resolve_done = (r->resolved_num == r->resolved_total);
-      gpr_mu_unlock (&r->mu);
-      if (resolve_done)
-	{
-	  zookeeper_on_resolved (r, r->resolved_addrs);
-	}
+    grpc_resolve_address(address, NULL, zookeeper_dns_resolved, r);
+    gpr_free(address);
+  } else {
+    gpr_log(GPR_ERROR, "Error in resolving a child node of %s", r->name);
+    gpr_mu_lock(&r->mu);
+    r->resolved_total--;
+    resolve_done = (r->resolved_num == r->resolved_total);
+    gpr_mu_unlock(&r->mu);
+    if (resolve_done) {
+      zookeeper_on_resolved(r, r->resolved_addrs);
     }
+  }
 }
 
-static void
-zookeeper_get_children_completion (int rc, const struct String_vector *children, const void *arg)
-{
+static void zookeeper_get_children_completion(
+    int rc, const struct String_vector *children, const void *arg) {
   char *path;
   int status;
   int i;
-  zookeeper_resolver *r = (zookeeper_resolver *) arg;
+  zookeeper_resolver *r = (zookeeper_resolver *)arg;
 
-  if (rc != 0)
-    {
-      gpr_log (GPR_ERROR, "Error in getting zookeeper children of %s", r->name);
-      return;
-    }
+  if (rc != 0) {
+    gpr_log(GPR_ERROR, "Error in getting zookeeper children of %s", r->name);
+    return;
+  }
 
-  if (children->count == 0)
-    {
-      gpr_log (GPR_ERROR, "Error in resolving zookeeper address %s", r->name);
-      return;
-    }
+  if (children->count == 0) {
+    gpr_log(GPR_ERROR, "Error in resolving zookeeper address %s", r->name);
+    return;
+  }
 
-  r->resolved_addrs = gpr_malloc (sizeof (grpc_resolved_addresses));
+  r->resolved_addrs = gpr_malloc(sizeof(grpc_resolved_addresses));
   r->resolved_addrs->addrs = NULL;
   r->resolved_addrs->naddrs = 0;
   r->resolved_total = children->count;
 
   /** TODO: Replace expensive heap allocation with stack
       if we can get maximum length of zookeeper path */
-  for (i = 0; i < children->count; i++)
-    {
-      gpr_asprintf (&path, "%s/%s", r->name, children->data[i]);
-      status = zoo_awget (r->zookeeper_handle, path, zookeeper_watcher, r, zookeeper_get_children_node_completion, r);
-      gpr_free (path);
-      if (status != 0)
-	{
-	  gpr_log (GPR_ERROR, "Error in getting zookeeper node %s", path);
-	}
+  for (i = 0; i < children->count; i++) {
+    gpr_asprintf(&path, "%s/%s", r->name, children->data[i]);
+    status = zoo_awget(r->zookeeper_handle, path, zookeeper_watcher, r,
+                       zookeeper_get_children_node_completion, r);
+    gpr_free(path);
+    if (status != 0) {
+      gpr_log(GPR_ERROR, "Error in getting zookeeper node %s", path);
     }
+  }
 }
 
-static void
-zookeeper_get_node_completion (int rc, const char *value, int value_len, const struct Stat *stat, const void *arg)
-{
+static void zookeeper_get_node_completion(int rc, const char *value,
+                                          int value_len,
+                                          const struct Stat *stat,
+                                          const void *arg) {
   int status;
   char *address = NULL;
-  zookeeper_resolver *r = (zookeeper_resolver *) arg;
+  zookeeper_resolver *r = (zookeeper_resolver *)arg;
   r->resolved_addrs = NULL;
   r->resolved_total = 0;
   r->resolved_num = 0;
 
-  if (rc != 0)
-    {
-      gpr_log (GPR_ERROR, "Error in getting zookeeper node %s", r->name);
-      return;
-    }
+  if (rc != 0) {
+    gpr_log(GPR_ERROR, "Error in getting zookeeper node %s", r->name);
+    return;
+  }
 
   /** If zookeeper node of path r->name does not have address
       (i.e. service node), get its children */
-  address = zookeeper_parse_address (value, (size_t) value_len);
-  if (address != NULL)
-    {
-      r->resolved_addrs = gpr_malloc (sizeof (grpc_resolved_addresses));
-      r->resolved_addrs->addrs = NULL;
-      r->resolved_addrs->naddrs = 0;
-      r->resolved_total = 1;
+  address = zookeeper_parse_address(value, (size_t)value_len);
+  if (address != NULL) {
+    r->resolved_addrs = gpr_malloc(sizeof(grpc_resolved_addresses));
+    r->resolved_addrs->addrs = NULL;
+    r->resolved_addrs->naddrs = 0;
+    r->resolved_total = 1;
     /** Further resolves address by DNS */
-      grpc_resolve_address (address, NULL, zookeeper_dns_resolved, r);
-      gpr_free (address);
-      return;
-    }
-
-  status = zoo_awget_children (r->zookeeper_handle, r->name, zookeeper_watcher, r, zookeeper_get_children_completion, r);
-  if (status != 0)
-    {
-      gpr_log (GPR_ERROR, "Error in getting zookeeper children of %s", r->name);
-    }
+    grpc_resolve_address(address, NULL, zookeeper_dns_resolved, r);
+    gpr_free(address);
+    return;
+  }
+
+  status = zoo_awget_children(r->zookeeper_handle, r->name, zookeeper_watcher,
+                              r, zookeeper_get_children_completion, r);
+  if (status != 0) {
+    gpr_log(GPR_ERROR, "Error in getting zookeeper children of %s", r->name);
+  }
 }
 
-static void
-zookeeper_resolve_address (zookeeper_resolver * r)
-{
+static void zookeeper_resolve_address(zookeeper_resolver *r) {
   int status;
-  status = zoo_awget (r->zookeeper_handle, r->name, zookeeper_watcher, r, zookeeper_get_node_completion, r);
-  if (status != 0)
-    {
-      gpr_log (GPR_ERROR, "Error in getting zookeeper node %s", r->name);
-    }
+  status = zoo_awget(r->zookeeper_handle, r->name, zookeeper_watcher, r,
+                     zookeeper_get_node_completion, r);
+  if (status != 0) {
+    gpr_log(GPR_ERROR, "Error in getting zookeeper node %s", r->name);
+  }
 }
 
-static void
-zookeeper_start_resolving_locked (zookeeper_resolver * r)
-{
-  GRPC_RESOLVER_REF (&r->base, "zookeeper-resolving");
-  GPR_ASSERT (r->resolving == 0);
+static void zookeeper_start_resolving_locked(zookeeper_resolver *r) {
+  GRPC_RESOLVER_REF(&r->base, "zookeeper-resolving");
+  GPR_ASSERT(r->resolving == 0);
   r->resolving = 1;
-  zookeeper_resolve_address (r);
+  zookeeper_resolve_address(r);
 }
 
-static grpc_closure *
-zookeeper_maybe_finish_next_locked (zookeeper_resolver * r)
-{
+static grpc_closure *zookeeper_maybe_finish_next_locked(zookeeper_resolver *r) {
   grpc_closure *call = NULL;
-  if (r->next_completion != NULL && r->resolved_version != r->published_version)
-    {
-      *r->target_config = r->resolved_config;
-      if (r->resolved_config != NULL)
-	{
-	  grpc_client_config_ref (r->resolved_config);
-	}
-      call = r->next_completion;
-      r->next_completion = NULL;
-      r->published_version = r->resolved_version;
+  if (r->next_completion != NULL &&
+      r->resolved_version != r->published_version) {
+    *r->target_config = r->resolved_config;
+    if (r->resolved_config != NULL) {
+      grpc_client_config_ref(r->resolved_config);
     }
+    call = r->next_completion;
+    r->next_completion = NULL;
+    r->published_version = r->resolved_version;
+  }
   return call;
 }
 
-static void
-zookeeper_destroy (grpc_resolver * gr)
-{
-  zookeeper_resolver *r = (zookeeper_resolver *) gr;
-  gpr_mu_destroy (&r->mu);
-  if (r->resolved_config != NULL)
-    {
-      grpc_client_config_unref (r->resolved_config);
-    }
-  grpc_subchannel_factory_unref (r->subchannel_factory);
-  gpr_free (r->name);
-  gpr_free (r->lb_policy_name);
-  gpr_free (r);
+static void zookeeper_destroy(grpc_resolver *gr) {
+  zookeeper_resolver *r = (zookeeper_resolver *)gr;
+  gpr_mu_destroy(&r->mu);
+  if (r->resolved_config != NULL) {
+    grpc_client_config_unref(r->resolved_config);
+  }
+  grpc_subchannel_factory_unref(r->subchannel_factory);
+  gpr_free(r->name);
+  gpr_free(r->lb_policy_name);
+  gpr_free(r);
 }
 
-static grpc_resolver *
-zookeeper_create (grpc_resolver_args * args, const char *lb_policy_name)
-{
+static grpc_resolver *zookeeper_create(grpc_resolver_args *args,
+                                       const char *lb_policy_name) {
   zookeeper_resolver *r;
   size_t length;
   char *path = args->uri->path;
 
-  if (0 == strcmp (args->uri->authority, ""))
-    {
-      gpr_log (GPR_ERROR, "No authority specified in zookeeper uri");
-      return NULL;
-    }
+  if (0 == strcmp(args->uri->authority, "")) {
+    gpr_log(GPR_ERROR, "No authority specified in zookeeper uri");
+    return NULL;
+  }
 
   /** Removes the trailing slash if exists */
-  length = strlen (path);
-  if (length > 1 && path[length - 1] == '/')
-    {
-      path[length - 1] = 0;
-    }
-
-  r = gpr_malloc (sizeof (zookeeper_resolver));
-  memset (r, 0, sizeof (*r));
-  gpr_ref_init (&r->refs, 1);
-  gpr_mu_init (&r->mu);
-  grpc_resolver_init (&r->base, &zookeeper_resolver_vtable);
-  r->name = gpr_strdup (path);
+  length = strlen(path);
+  if (length > 1 && path[length - 1] == '/') {
+    path[length - 1] = 0;
+  }
+
+  r = gpr_malloc(sizeof(zookeeper_resolver));
+  memset(r, 0, sizeof(*r));
+  gpr_ref_init(&r->refs, 1);
+  gpr_mu_init(&r->mu);
+  grpc_resolver_init(&r->base, &zookeeper_resolver_vtable);
+  r->name = gpr_strdup(path);
 
   r->subchannel_factory = args->subchannel_factory;
-  grpc_subchannel_factory_ref (r->subchannel_factory);
+  grpc_subchannel_factory_ref(r->subchannel_factory);
 
-  r->lb_policy_name = gpr_strdup (lb_policy_name);
+  r->lb_policy_name = gpr_strdup(lb_policy_name);
 
   /** Initializes zookeeper client */
-  zoo_set_debug_level (ZOO_LOG_LEVEL_WARN);
-  r->zookeeper_handle = zookeeper_init (args->uri->authority, zookeeper_global_watcher, GRPC_ZOOKEEPER_SESSION_TIMEOUT, 0, 0, 0);
-  if (r->zookeeper_handle == NULL)
-    {
-      gpr_log (GPR_ERROR, "Unable to connect to zookeeper server");
-      return NULL;
-    }
+  zoo_set_debug_level(ZOO_LOG_LEVEL_WARN);
+  r->zookeeper_handle =
+      zookeeper_init(args->uri->authority, zookeeper_global_watcher,
+                     GRPC_ZOOKEEPER_SESSION_TIMEOUT, 0, 0, 0);
+  if (r->zookeeper_handle == NULL) {
+    gpr_log(GPR_ERROR, "Unable to connect to zookeeper server");
+    return NULL;
+  }
 
   return &r->base;
 }
 
-static void
-zookeeper_plugin_init ()
-{
-  grpc_register_resolver_type (grpc_zookeeper_resolver_factory_create ());
+static void zookeeper_plugin_init() {
+  grpc_register_resolver_type(grpc_zookeeper_resolver_factory_create());
 }
 
-void
-grpc_zookeeper_register ()
-{
-  grpc_register_plugin (zookeeper_plugin_init, NULL);
+void grpc_zookeeper_register() {
+  grpc_register_plugin(zookeeper_plugin_init, NULL);
 }
 
 /*
  * FACTORY
  */
 
-static void
-zookeeper_factory_ref (grpc_resolver_factory * factory)
-{
-}
+static void zookeeper_factory_ref(grpc_resolver_factory *factory) {}
 
-static void
-zookeeper_factory_unref (grpc_resolver_factory * factory)
-{
-}
+static void zookeeper_factory_unref(grpc_resolver_factory *factory) {}
 
-static char *
-zookeeper_factory_get_default_hostname (grpc_resolver_factory * factory, grpc_uri * uri)
-{
+static char *zookeeper_factory_get_default_hostname(
+    grpc_resolver_factory *factory, grpc_uri *uri) {
   return NULL;
 }
 
-static grpc_resolver *
-zookeeper_factory_create_resolver (grpc_resolver_factory * factory, grpc_resolver_args * args)
-{
-  return zookeeper_create (args, "pick_first");
+static grpc_resolver *zookeeper_factory_create_resolver(
+    grpc_resolver_factory *factory, grpc_resolver_args *args) {
+  return zookeeper_create(args, "pick_first");
 }
 
 static const grpc_resolver_factory_vtable zookeeper_factory_vtable = {
-  zookeeper_factory_ref, zookeeper_factory_unref,
-  zookeeper_factory_create_resolver, zookeeper_factory_get_default_hostname,
-  "zookeeper"
-};
+    zookeeper_factory_ref, zookeeper_factory_unref,
+    zookeeper_factory_create_resolver, zookeeper_factory_get_default_hostname,
+    "zookeeper"};
 
 static grpc_resolver_factory zookeeper_resolver_factory = {
-  &zookeeper_factory_vtable
-};
+    &zookeeper_factory_vtable};
 
-grpc_resolver_factory *
-grpc_zookeeper_resolver_factory_create ()
-{
+grpc_resolver_factory *grpc_zookeeper_resolver_factory_create() {
   return &zookeeper_resolver_factory;
 }