Cleanup: Fix various coding style issues in src/untrusted/pthread

src/untrusted/pthread/pthread_types.h

 /*
  * Copyright (c) 2012 The Native Client Authors. All rights reserved.
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 /*
  * Native Client pthreads implementation layer
  */
 
 #ifndef NATIVE_CLIENT_SRC_UNTRUSTED_PTHREAD_NC_PTHREAD_TYPES_H_
 #define NATIVE_CLIENT_SRC_UNTRUSTED_PTHREAD_NC_PTHREAD_TYPES_H_ 1
 
 #include "native_client/src/untrusted/pthread/pthread.h"
 
 
 #ifdef __cplusplus
 extern "C" {
 #endif
 
 void nc_token_init(volatile int *token, int acquire);
-int  nc_token_acquire(volatile int *token);
+int nc_token_acquire(volatile int *token);
 void nc_token_release(volatile int *token);
 
-int  nc_pthread_condvar_ctor(pthread_cond_t * cond);
+int nc_pthread_condvar_ctor(pthread_cond_t *cond);
 
 typedef enum {
   THREAD_RUNNING,
   THREAD_TERMINATED
 } nc_thread_status_t;
 
 typedef struct entry {
   STAILQ_ENTRY(entry) entries;  /* a pointer - 4 bytes */
   int32_t is_used;              /* 4 bytes */
   int32_t size;                 /* 4 bytes */
@@ skipping 17 matching lines @@
  *       ("tcbhead_t") as long as we keep the field "dtv" untouched and
  *       use tls_base in a compatible way (which we already do).
  */
 typedef struct {
   void *tls_base;  /* tls accesses are made relative to this base */
   void *unused_dtv;  /* increase compatibility with glibc's tcbhead_t */
   int joinable;
   int join_waiting;
   nc_thread_memory_block_t *stack_node;
   nc_thread_memory_block_t *tls_node;
-  nc_thread_function  start_func;
-  void* state;
+  nc_thread_function start_func;
+  void *state;
   /*
    * irt_thread_data is used when libpthread is linked into the IRT.
    * It is used for free()ing the thread block.
    * TODO(mseaborn): This plays a similar role to tls_node; the two
    * could be unified in future.
    */
   void *irt_thread_data;
   struct __nc_basic_thread_data *basic_data;
 } nc_thread_descriptor_t;
 
 typedef struct __nc_basic_thread_data {
   void *retval;
   nc_thread_status_t status;
   pthread_cond_t join_condvar;
-  /* pointer to the tdb, will be null after the thread terminates */
+  /* Pointer to the tdb, will be null after the thread terminates. */
   nc_thread_descriptor_t *tdb;
 } nc_basic_thread_data_t;
 
 /*
  * This structure is allocated for all threads.  However, for threads
  * created with pthread_create(), the basic_data field is not used,
  * and a separate nc_basic_thread_data_t struct is allocated.  The
  * initial thread, and threads created by the IRT's thread_create()
  * interface, do use the basic_data field though.
  * TODO(mseaborn): Clean this up to be more consistent.
  */
 struct nc_combined_tdb {
   nc_thread_descriptor_t tdb;
   nc_basic_thread_data_t basic_data;
 };
 
 
 #define MEMORY_BLOCK_ALLOCATION_SIZE(real_size) \
   (sizeof(nc_thread_memory_block_t) + (real_size))
 
 #ifdef __cplusplus
 }
 #endif
 
 #endif  /* NATIVE_CLIENT_SRC_UNTRUSTED_PTHREAD_NC_PTHREAD_TYPES_H_ */