indent pass to get logical source lines on one physical line

1 files changed, 316 insertions, 242 deletions

diff --git a/src/core/iomgr/fd_posix.c b/src/core/iomgr/fd_posix.c
index 5e455f32d7..279ed28001 100644
--- a/src/core/iomgr/fd_posix.c
+++ b/src/core/iomgr/fd_posix.c
@@ -45,10 +45,11 @@
 #include <grpc/support/log.h>
 #include <grpc/support/useful.h>
 
-enum descriptor_state {
+enum descriptor_state
+{
   NOT_READY = 0,
   READY = 1
-}; /* or a pointer to a closure to call */
+};				/* or a pointer to a closure to call */
 
 /* We need to keep a freelist not because of any concerns of malloc performance
  * but instead so that implementations with multiple threads in (for example)
@@ -70,35 +71,40 @@ enum descriptor_state {
 static grpc_fd *fd_freelist = NULL;
 static gpr_mu fd_freelist_mu;
 
-static void freelist_fd(grpc_fd *fd) {
-  gpr_mu_lock(&fd_freelist_mu);
+static void
+freelist_fd (grpc_fd * fd)
+{
+  gpr_mu_lock (&fd_freelist_mu);
   fd->freelist_next = fd_freelist;
   fd_freelist = fd;
-  grpc_iomgr_unregister_object(&fd->iomgr_object);
-  gpr_mu_unlock(&fd_freelist_mu);
+  grpc_iomgr_unregister_object (&fd->iomgr_object);
+  gpr_mu_unlock (&fd_freelist_mu);
 }
 
-static grpc_fd *alloc_fd(int fd) {
+static grpc_fd *
+alloc_fd (int fd)
+{
   grpc_fd *r = NULL;
-  gpr_mu_lock(&fd_freelist_mu);
-  if (fd_freelist != NULL) {
-    r = fd_freelist;
-    fd_freelist = fd_freelist->freelist_next;
-  }
-  gpr_mu_unlock(&fd_freelist_mu);
-  if (r == NULL) {
-    r = gpr_malloc(sizeof(grpc_fd));
-    gpr_mu_init(&r->set_state_mu);
-    gpr_mu_init(&r->watcher_mu);
-  }
-
-  gpr_atm_rel_store(&r->refst, 1);
-  gpr_atm_rel_store(&r->readst, NOT_READY);
-  gpr_atm_rel_store(&r->writest, NOT_READY);
-  gpr_atm_rel_store(&r->shutdown, 0);
+  gpr_mu_lock (&fd_freelist_mu);
+  if (fd_freelist != NULL)
+    {
+      r = fd_freelist;
+      fd_freelist = fd_freelist->freelist_next;
+    }
+  gpr_mu_unlock (&fd_freelist_mu);
+  if (r == NULL)
+    {
+      r = gpr_malloc (sizeof (grpc_fd));
+      gpr_mu_init (&r->set_state_mu);
+      gpr_mu_init (&r->watcher_mu);
+    }
+
+  gpr_atm_rel_store (&r->refst, 1);
+  gpr_atm_rel_store (&r->readst, NOT_READY);
+  gpr_atm_rel_store (&r->writest, NOT_READY);
+  gpr_atm_rel_store (&r->shutdown, 0);
   r->fd = fd;
-  r->inactive_watcher_root.next = r->inactive_watcher_root.prev =
-      &r->inactive_watcher_root;
+  r->inactive_watcher_root.next = r->inactive_watcher_root.prev = &r->inactive_watcher_root;
   r->freelist_next = NULL;
   r->read_watcher = r->write_watcher = NULL;
   r->on_done_closure = NULL;
@@ -106,147 +112,198 @@ static grpc_fd *alloc_fd(int fd) {
   return r;
 }
 
-static void destroy(grpc_fd *fd) {
-  gpr_mu_destroy(&fd->set_state_mu);
-  gpr_mu_destroy(&fd->watcher_mu);
-  gpr_free(fd);
+static void
+destroy (grpc_fd * fd)
+{
+  gpr_mu_destroy (&fd->set_state_mu);
+  gpr_mu_destroy (&fd->watcher_mu);
+  gpr_free (fd);
 }
 
 #ifdef GRPC_FD_REF_COUNT_DEBUG
 #define REF_BY(fd, n, reason) ref_by(fd, n, reason, __FILE__, __LINE__)
 #define UNREF_BY(fd, n, reason) unref_by(fd, n, reason, __FILE__, __LINE__)
-static void ref_by(grpc_fd *fd, int n, const char *reason, const char *file,
-                   int line) {
-  gpr_log(GPR_DEBUG, "FD %d %p   ref %d %d -> %d [%s; %s:%d]", fd->fd, fd, n,
-          gpr_atm_no_barrier_load(&fd->refst),
-          gpr_atm_no_barrier_load(&fd->refst) + n, reason, file, line);
+static void
+ref_by (grpc_fd * fd, int n, const char *reason, const char *file, int line)
+{
+  gpr_log (GPR_DEBUG, "FD %d %p   ref %d %d -> %d [%s; %s:%d]", fd->fd, fd, n, gpr_atm_no_barrier_load (&fd->refst), gpr_atm_no_barrier_load (&fd->refst) + n, reason, file, line);
 #else
 #define REF_BY(fd, n, reason) ref_by(fd, n)
 #define UNREF_BY(fd, n, reason) unref_by(fd, n)
-static void ref_by(grpc_fd *fd, int n) {
+static void
+ref_by (grpc_fd * fd, int n)
+{
 #endif
-  GPR_ASSERT(gpr_atm_no_barrier_fetch_add(&fd->refst, n) > 0);
+  GPR_ASSERT (gpr_atm_no_barrier_fetch_add (&fd->refst, n) > 0);
 }
 
 #ifdef GRPC_FD_REF_COUNT_DEBUG
-static void unref_by(grpc_fd *fd, int n, const char *reason, const char *file,
-                     int line) {
+static void
+unref_by (grpc_fd * fd, int n, const char *reason, const char *file, int line)
+{
   gpr_atm old;
-  gpr_log(GPR_DEBUG, "FD %d %p unref %d %d -> %d [%s; %s:%d]", fd->fd, fd, n,
-          gpr_atm_no_barrier_load(&fd->refst),
-          gpr_atm_no_barrier_load(&fd->refst) - n, reason, file, line);
+  gpr_log (GPR_DEBUG, "FD %d %p unref %d %d -> %d [%s; %s:%d]", fd->fd, fd, n, gpr_atm_no_barrier_load (&fd->refst), gpr_atm_no_barrier_load (&fd->refst) - n, reason, file, line);
 #else
-static void unref_by(grpc_fd *fd, int n) {
+static void
+unref_by (grpc_fd * fd, int n)
+{
   gpr_atm old;
 #endif
-  old = gpr_atm_full_fetch_add(&fd->refst, -n);
-  if (old == n) {
-    freelist_fd(fd);
-  } else {
-    GPR_ASSERT(old > n);
-  }
+  old = gpr_atm_full_fetch_add (&fd->refst, -n);
+  if (old == n)
+    {
+      freelist_fd (fd);
+    }
+  else
+    {
+      GPR_ASSERT (old > n);
+    }
 }
 
-void grpc_fd_global_init(void) { gpr_mu_init(&fd_freelist_mu); }
-
-void grpc_fd_global_shutdown(void) {
-  gpr_mu_lock(&fd_freelist_mu);
-  gpr_mu_unlock(&fd_freelist_mu);
-  while (fd_freelist != NULL) {
-    grpc_fd *fd = fd_freelist;
-    fd_freelist = fd_freelist->freelist_next;
-    destroy(fd);
-  }
-  gpr_mu_destroy(&fd_freelist_mu);
+void
+grpc_fd_global_init (void)
+{
+  gpr_mu_init (&fd_freelist_mu);
 }
 
-grpc_fd *grpc_fd_create(int fd, const char *name) {
-  grpc_fd *r = alloc_fd(fd);
-  grpc_iomgr_register_object(&r->iomgr_object, name);
+void
+grpc_fd_global_shutdown (void)
+{
+  gpr_mu_lock (&fd_freelist_mu);
+  gpr_mu_unlock (&fd_freelist_mu);
+  while (fd_freelist != NULL)
+    {
+      grpc_fd *fd = fd_freelist;
+      fd_freelist = fd_freelist->freelist_next;
+      destroy (fd);
+    }
+  gpr_mu_destroy (&fd_freelist_mu);
+}
+
+grpc_fd *
+grpc_fd_create (int fd, const char *name)
+{
+  grpc_fd *r = alloc_fd (fd);
+  grpc_iomgr_register_object (&r->iomgr_object, name);
   return r;
 }
 
-int grpc_fd_is_orphaned(grpc_fd *fd) {
-  return (gpr_atm_acq_load(&fd->refst) & 1) == 0;
+int
+grpc_fd_is_orphaned (grpc_fd * fd)
+{
+  return (gpr_atm_acq_load (&fd->refst) & 1) == 0;
 }
 
-static void pollset_kick_locked(grpc_pollset *pollset) {
-  gpr_mu_lock(GRPC_POLLSET_MU(pollset));
-  grpc_pollset_kick(pollset, NULL);
-  gpr_mu_unlock(GRPC_POLLSET_MU(pollset));
+static void
+pollset_kick_locked (grpc_pollset * pollset)
+{
+  gpr_mu_lock (GRPC_POLLSET_MU (pollset));
+  grpc_pollset_kick (pollset, NULL);
+  gpr_mu_unlock (GRPC_POLLSET_MU (pollset));
 }
 
-static void maybe_wake_one_watcher_locked(grpc_fd *fd) {
-  if (fd->inactive_watcher_root.next != &fd->inactive_watcher_root) {
-    pollset_kick_locked(fd->inactive_watcher_root.next->pollset);
-  } else if (fd->read_watcher) {
-    pollset_kick_locked(fd->read_watcher->pollset);
-  } else if (fd->write_watcher) {
-    pollset_kick_locked(fd->write_watcher->pollset);
-  }
+static void
+maybe_wake_one_watcher_locked (grpc_fd * fd)
+{
+  if (fd->inactive_watcher_root.next != &fd->inactive_watcher_root)
+    {
+      pollset_kick_locked (fd->inactive_watcher_root.next->pollset);
+    }
+  else if (fd->read_watcher)
+    {
+      pollset_kick_locked (fd->read_watcher->pollset);
+    }
+  else if (fd->write_watcher)
+    {
+      pollset_kick_locked (fd->write_watcher->pollset);
+    }
 }
 
-static void maybe_wake_one_watcher(grpc_fd *fd) {
-  gpr_mu_lock(&fd->watcher_mu);
-  maybe_wake_one_watcher_locked(fd);
-  gpr_mu_unlock(&fd->watcher_mu);
+static void
+maybe_wake_one_watcher (grpc_fd * fd)
+{
+  gpr_mu_lock (&fd->watcher_mu);
+  maybe_wake_one_watcher_locked (fd);
+  gpr_mu_unlock (&fd->watcher_mu);
 }
 
-static void wake_all_watchers_locked(grpc_fd *fd) {
+static void
+wake_all_watchers_locked (grpc_fd * fd)
+{
   grpc_fd_watcher *watcher;
-  for (watcher = fd->inactive_watcher_root.next;
-       watcher != &fd->inactive_watcher_root; watcher = watcher->next) {
-    pollset_kick_locked(watcher->pollset);
-  }
-  if (fd->read_watcher) {
-    pollset_kick_locked(fd->read_watcher->pollset);
-  }
-  if (fd->write_watcher && fd->write_watcher != fd->read_watcher) {
-    pollset_kick_locked(fd->write_watcher->pollset);
-  }
+  for (watcher = fd->inactive_watcher_root.next; watcher != &fd->inactive_watcher_root; watcher = watcher->next)
+    {
+      pollset_kick_locked (watcher->pollset);
+    }
+  if (fd->read_watcher)
+    {
+      pollset_kick_locked (fd->read_watcher->pollset);
+    }
+  if (fd->write_watcher && fd->write_watcher != fd->read_watcher)
+    {
+      pollset_kick_locked (fd->write_watcher->pollset);
+    }
 }
 
-static int has_watchers(grpc_fd *fd) {
-  return fd->read_watcher != NULL || fd->write_watcher != NULL ||
-         fd->inactive_watcher_root.next != &fd->inactive_watcher_root;
+static int
+has_watchers (grpc_fd * fd)
+{
+  return fd->read_watcher != NULL || fd->write_watcher != NULL || fd->inactive_watcher_root.next != &fd->inactive_watcher_root;
 }
 
-void grpc_fd_orphan(grpc_fd *fd, grpc_closure *on_done, const char *reason,
-                    grpc_closure_list *closure_list) {
+void
+grpc_fd_orphan (grpc_fd * fd, grpc_closure * on_done, const char *reason, grpc_closure_list * closure_list)
+{
   fd->on_done_closure = on_done;
-  shutdown(fd->fd, SHUT_RDWR);
-  gpr_mu_lock(&fd->watcher_mu);
-  REF_BY(fd, 1, reason); /* remove active status, but keep referenced */
-  if (!has_watchers(fd)) {
-    fd->closed = 1;
-    close(fd->fd);
-    grpc_closure_list_add(closure_list, fd->on_done_closure, 1);
-  } else {
-    wake_all_watchers_locked(fd);
-  }
-  gpr_mu_unlock(&fd->watcher_mu);
-  UNREF_BY(fd, 2, reason); /* drop the reference */
+  shutdown (fd->fd, SHUT_RDWR);
+  gpr_mu_lock (&fd->watcher_mu);
+  REF_BY (fd, 1, reason);	/* remove active status, but keep referenced */
+  if (!has_watchers (fd))
+    {
+      fd->closed = 1;
+      close (fd->fd);
+      grpc_closure_list_add (closure_list, fd->on_done_closure, 1);
+    }
+  else
+    {
+      wake_all_watchers_locked (fd);
+    }
+  gpr_mu_unlock (&fd->watcher_mu);
+  UNREF_BY (fd, 2, reason);	/* drop the reference */
 }
 
 /* increment refcount by two to avoid changing the orphan bit */
 #ifdef GRPC_FD_REF_COUNT_DEBUG
-void grpc_fd_ref(grpc_fd *fd, const char *reason, const char *file, int line) {
-  ref_by(fd, 2, reason, file, line);
+void
+grpc_fd_ref (grpc_fd * fd, const char *reason, const char *file, int line)
+{
+  ref_by (fd, 2, reason, file, line);
 }
 
-void grpc_fd_unref(grpc_fd *fd, const char *reason, const char *file,
-                   int line) {
-  unref_by(fd, 2, reason, file, line);
+void
+grpc_fd_unref (grpc_fd * fd, const char *reason, const char *file, int line)
+{
+  unref_by (fd, 2, reason, file, line);
 }
 #else
-void grpc_fd_ref(grpc_fd *fd) { ref_by(fd, 2); }
+void
+grpc_fd_ref (grpc_fd * fd)
+{
+  ref_by (fd, 2);
+}
 
-void grpc_fd_unref(grpc_fd *fd) { unref_by(fd, 2); }
+void
+grpc_fd_unref (grpc_fd * fd)
+{
+  unref_by (fd, 2);
+}
 #endif
 
-static void notify_on(grpc_fd *fd, gpr_atm *st, grpc_closure *closure,
-                      grpc_closure_list *closure_list) {
-  switch (gpr_atm_acq_load(st)) {
+static void
+notify_on (grpc_fd * fd, gpr_atm * st, grpc_closure * closure, grpc_closure_list * closure_list)
+{
+  switch (gpr_atm_acq_load (st))
+    {
     case NOT_READY:
       /* There is no race if the descriptor is already ready, so we skip
          the interlocked op in that case.  As long as the app doesn't
@@ -254,178 +311,195 @@ static void notify_on(grpc_fd *fd, gpr_atm *st, grpc_closure *closure,
          oldval should never be anything other than READY or NOT_READY.  We
          don't
          check for user error on the fast path. */
-      if (gpr_atm_rel_cas(st, NOT_READY, (gpr_intptr)closure)) {
-        /* swap was successful -- the closure will run after the next
-           set_ready call.  NOTE: we don't have an ABA problem here,
-           since we should never have concurrent calls to the same
-           notify_on function. */
-        maybe_wake_one_watcher(fd);
-        return;
-      }
-    /* swap was unsuccessful due to an intervening set_ready call.
-       Fall through to the READY code below */
+      if (gpr_atm_rel_cas (st, NOT_READY, (gpr_intptr) closure))
+	{
+	  /* swap was successful -- the closure will run after the next
+	     set_ready call.  NOTE: we don't have an ABA problem here,
+	     since we should never have concurrent calls to the same
+	     notify_on function. */
+	  maybe_wake_one_watcher (fd);
+	  return;
+	}
+      /* swap was unsuccessful due to an intervening set_ready call.
+         Fall through to the READY code below */
     case READY:
-      GPR_ASSERT(gpr_atm_no_barrier_load(st) == READY);
-      gpr_atm_rel_store(st, NOT_READY);
-      grpc_closure_list_add(closure_list, closure,
-                            !gpr_atm_acq_load(&fd->shutdown));
+      GPR_ASSERT (gpr_atm_no_barrier_load (st) == READY);
+      gpr_atm_rel_store (st, NOT_READY);
+      grpc_closure_list_add (closure_list, closure, !gpr_atm_acq_load (&fd->shutdown));
       return;
-    default: /* WAITING */
+    default:			/* WAITING */
       /* upcallptr was set to a different closure.  This is an error! */
-      gpr_log(GPR_ERROR,
-              "User called a notify_on function with a previous callback still "
-              "pending");
-      abort();
-  }
-  gpr_log(GPR_ERROR, "Corrupt memory in &st->state");
-  abort();
+      gpr_log (GPR_ERROR, "User called a notify_on function with a previous callback still " "pending");
+      abort ();
+    }
+  gpr_log (GPR_ERROR, "Corrupt memory in &st->state");
+  abort ();
 }
 
-static void set_ready_locked(grpc_fd *fd, gpr_atm *st,
-                             grpc_closure_list *closure_list) {
-  gpr_intptr state = gpr_atm_acq_load(st);
+static void
+set_ready_locked (grpc_fd * fd, gpr_atm * st, grpc_closure_list * closure_list)
+{
+  gpr_intptr state = gpr_atm_acq_load (st);
 
-  switch (state) {
+  switch (state)
+    {
     case READY:
       /* duplicate ready, ignore */
       return;
     case NOT_READY:
-      if (gpr_atm_rel_cas(st, NOT_READY, READY)) {
-        /* swap was successful -- the closure will run after the next
-           notify_on call. */
-        return;
-      }
+      if (gpr_atm_rel_cas (st, NOT_READY, READY))
+	{
+	  /* swap was successful -- the closure will run after the next
+	     notify_on call. */
+	  return;
+	}
       /* swap was unsuccessful due to an intervening set_ready call.
          Fall through to the WAITING code below */
-      state = gpr_atm_acq_load(st);
-    default: /* waiting */
-      GPR_ASSERT(gpr_atm_no_barrier_load(st) != READY &&
-                 gpr_atm_no_barrier_load(st) != NOT_READY);
-      grpc_closure_list_add(closure_list, (grpc_closure *)state,
-                            !gpr_atm_acq_load(&fd->shutdown));
-      gpr_atm_rel_store(st, NOT_READY);
+      state = gpr_atm_acq_load (st);
+    default:			/* waiting */
+      GPR_ASSERT (gpr_atm_no_barrier_load (st) != READY && gpr_atm_no_barrier_load (st) != NOT_READY);
+      grpc_closure_list_add (closure_list, (grpc_closure *) state, !gpr_atm_acq_load (&fd->shutdown));
+      gpr_atm_rel_store (st, NOT_READY);
       return;
-  }
+    }
 }
 
-static void set_ready(grpc_fd *fd, gpr_atm *st,
-                      grpc_closure_list *closure_list) {
+static void
+set_ready (grpc_fd * fd, gpr_atm * st, grpc_closure_list * closure_list)
+{
   /* only one set_ready can be active at once (but there may be a racing
      notify_on) */
-  gpr_mu_lock(&fd->set_state_mu);
-  set_ready_locked(fd, st, closure_list);
-  gpr_mu_unlock(&fd->set_state_mu);
+  gpr_mu_lock (&fd->set_state_mu);
+  set_ready_locked (fd, st, closure_list);
+  gpr_mu_unlock (&fd->set_state_mu);
 }
 
-void grpc_fd_shutdown(grpc_fd *fd, grpc_closure_list *closure_list) {
-  gpr_mu_lock(&fd->set_state_mu);
-  GPR_ASSERT(!gpr_atm_no_barrier_load(&fd->shutdown));
-  gpr_atm_rel_store(&fd->shutdown, 1);
-  set_ready_locked(fd, &fd->readst, closure_list);
-  set_ready_locked(fd, &fd->writest, closure_list);
-  gpr_mu_unlock(&fd->set_state_mu);
+void
+grpc_fd_shutdown (grpc_fd * fd, grpc_closure_list * closure_list)
+{
+  gpr_mu_lock (&fd->set_state_mu);
+  GPR_ASSERT (!gpr_atm_no_barrier_load (&fd->shutdown));
+  gpr_atm_rel_store (&fd->shutdown, 1);
+  set_ready_locked (fd, &fd->readst, closure_list);
+  set_ready_locked (fd, &fd->writest, closure_list);
+  gpr_mu_unlock (&fd->set_state_mu);
 }
 
-void grpc_fd_notify_on_read(grpc_fd *fd, grpc_closure *closure,
-                            grpc_closure_list *closure_list) {
-  notify_on(fd, &fd->readst, closure, closure_list);
+void
+grpc_fd_notify_on_read (grpc_fd * fd, grpc_closure * closure, grpc_closure_list * closure_list)
+{
+  notify_on (fd, &fd->readst, closure, closure_list);
 }
 
-void grpc_fd_notify_on_write(grpc_fd *fd, grpc_closure *closure,
-                             grpc_closure_list *closure_list) {
-  notify_on(fd, &fd->writest, closure, closure_list);
+void
+grpc_fd_notify_on_write (grpc_fd * fd, grpc_closure * closure, grpc_closure_list * closure_list)
+{
+  notify_on (fd, &fd->writest, closure, closure_list);
 }
 
-gpr_uint32 grpc_fd_begin_poll(grpc_fd *fd, grpc_pollset *pollset,
-                              gpr_uint32 read_mask, gpr_uint32 write_mask,
-                              grpc_fd_watcher *watcher) {
+gpr_uint32
+grpc_fd_begin_poll (grpc_fd * fd, grpc_pollset * pollset, gpr_uint32 read_mask, gpr_uint32 write_mask, grpc_fd_watcher * watcher)
+{
   gpr_uint32 mask = 0;
   /* keep track of pollers that have requested our events, in case they change
    */
-  GRPC_FD_REF(fd, "poll");
+  GRPC_FD_REF (fd, "poll");
 
-  gpr_mu_lock(&fd->watcher_mu);
+  gpr_mu_lock (&fd->watcher_mu);
   /* if we are shutdown, then don't add to the watcher set */
-  if (gpr_atm_no_barrier_load(&fd->shutdown)) {
-    watcher->fd = NULL;
-    watcher->pollset = NULL;
-    gpr_mu_unlock(&fd->watcher_mu);
-    GRPC_FD_UNREF(fd, "poll");
-    return 0;
-  }
+  if (gpr_atm_no_barrier_load (&fd->shutdown))
+    {
+      watcher->fd = NULL;
+      watcher->pollset = NULL;
+      gpr_mu_unlock (&fd->watcher_mu);
+      GRPC_FD_UNREF (fd, "poll");
+      return 0;
+    }
   /* if there is nobody polling for read, but we need to, then start doing so */
-  if (read_mask && !fd->read_watcher &&
-      (gpr_uintptr)gpr_atm_acq_load(&fd->readst) > READY) {
-    fd->read_watcher = watcher;
-    mask |= read_mask;
-  }
+  if (read_mask && !fd->read_watcher && (gpr_uintptr) gpr_atm_acq_load (&fd->readst) > READY)
+    {
+      fd->read_watcher = watcher;
+      mask |= read_mask;
+    }
   /* if there is nobody polling for write, but we need to, then start doing so
    */
-  if (write_mask && !fd->write_watcher &&
-      (gpr_uintptr)gpr_atm_acq_load(&fd->writest) > READY) {
-    fd->write_watcher = watcher;
-    mask |= write_mask;
-  }
+  if (write_mask && !fd->write_watcher && (gpr_uintptr) gpr_atm_acq_load (&fd->writest) > READY)
+    {
+      fd->write_watcher = watcher;
+      mask |= write_mask;
+    }
   /* if not polling, remember this watcher in case we need someone to later */
-  if (mask == 0) {
-    watcher->next = &fd->inactive_watcher_root;
-    watcher->prev = watcher->next->prev;
-    watcher->next->prev = watcher->prev->next = watcher;
-  }
+  if (mask == 0)
+    {
+      watcher->next = &fd->inactive_watcher_root;
+      watcher->prev = watcher->next->prev;
+      watcher->next->prev = watcher->prev->next = watcher;
+    }
   watcher->pollset = pollset;
   watcher->fd = fd;
-  gpr_mu_unlock(&fd->watcher_mu);
+  gpr_mu_unlock (&fd->watcher_mu);
 
   return mask;
 }
 
-void grpc_fd_end_poll(grpc_fd_watcher *watcher, int got_read, int got_write,
-                      grpc_closure_list *closure_list) {
+void
+grpc_fd_end_poll (grpc_fd_watcher * watcher, int got_read, int got_write, grpc_closure_list * closure_list)
+{
   int was_polling = 0;
   int kick = 0;
   grpc_fd *fd = watcher->fd;
 
-  if (fd == NULL) {
-    return;
-  }
-
-  gpr_mu_lock(&fd->watcher_mu);
-  if (watcher == fd->read_watcher) {
-    /* remove read watcher, kick if we still need a read */
-    was_polling = 1;
-    kick = kick || !got_read;
-    fd->read_watcher = NULL;
-  }
-  if (watcher == fd->write_watcher) {
-    /* remove write watcher, kick if we still need a write */
-    was_polling = 1;
-    kick = kick || !got_write;
-    fd->write_watcher = NULL;
-  }
-  if (!was_polling) {
-    /* remove from inactive list */
-    watcher->next->prev = watcher->prev;
-    watcher->prev->next = watcher->next;
-  }
-  if (kick) {
-    maybe_wake_one_watcher_locked(fd);
-  }
-  if (grpc_fd_is_orphaned(fd) && !has_watchers(fd) && !fd->closed) {
-    fd->closed = 1;
-    close(fd->fd);
-    grpc_closure_list_add(closure_list, fd->on_done_closure, 1);
-  }
-  gpr_mu_unlock(&fd->watcher_mu);
-
-  GRPC_FD_UNREF(fd, "poll");
+  if (fd == NULL)
+    {
+      return;
+    }
+
+  gpr_mu_lock (&fd->watcher_mu);
+  if (watcher == fd->read_watcher)
+    {
+      /* remove read watcher, kick if we still need a read */
+      was_polling = 1;
+      kick = kick || !got_read;
+      fd->read_watcher = NULL;
+    }
+  if (watcher == fd->write_watcher)
+    {
+      /* remove write watcher, kick if we still need a write */
+      was_polling = 1;
+      kick = kick || !got_write;
+      fd->write_watcher = NULL;
+    }
+  if (!was_polling)
+    {
+      /* remove from inactive list */
+      watcher->next->prev = watcher->prev;
+      watcher->prev->next = watcher->next;
+    }
+  if (kick)
+    {
+      maybe_wake_one_watcher_locked (fd);
+    }
+  if (grpc_fd_is_orphaned (fd) && !has_watchers (fd) && !fd->closed)
+    {
+      fd->closed = 1;
+      close (fd->fd);
+      grpc_closure_list_add (closure_list, fd->on_done_closure, 1);
+    }
+  gpr_mu_unlock (&fd->watcher_mu);
+
+  GRPC_FD_UNREF (fd, "poll");
 }
 
-void grpc_fd_become_readable(grpc_fd *fd, grpc_closure_list *closure_list) {
-  set_ready(fd, &fd->readst, closure_list);
+void
+grpc_fd_become_readable (grpc_fd * fd, grpc_closure_list * closure_list)
+{
+  set_ready (fd, &fd->readst, closure_list);
 }
 
-void grpc_fd_become_writable(grpc_fd *fd, grpc_closure_list *closure_list) {
-  set_ready(fd, &fd->writest, closure_list);
+void
+grpc_fd_become_writable (grpc_fd * fd, grpc_closure_list * closure_list)
+{
+  set_ready (fd, &fd->writest, closure_list);
 }
 
 #endif