Run clang-format and check in Travis CI (#14)

* Run clang-format and add pre-commit hook for it.

* Modify .travis.yml to check

* Try to fix problems with .travis.yml

* Try to fix .travis.yml yet again

* Update .clang-format to Philipp's preferences

* Don't allow lint to fail in Travis

* Remove git-hooks directory

* Improve clang-format failure output

* Fix clang-format error

* Report which commit clang-format is comparing against, and add whitespace error

* Handle non-PR Travis in clang-format, and add another error

* Check $TRAVIS_PULL_REQUEST correctly and add another error

* Fix syntax error in check-git-clang-format-output.sh

* Add whitespace error

* Remove extra whitespace, add clang-format to README

.clang-format

@@ -0,0 +1,5 @@
+BasedOnStyle: Chromium
+DerivePointerAlignment: true
+IndentCaseLabels: false
+PointerAlignment: Right
+SpaceAfterCStyleCast: true

.travis.yml

@@ -16,6 +16,19 @@ matrix:
     - os: osx
       osx_image: xcode7
       python: "3.5"
+    - os: linux
+      dist: trusty
+      python: "2.7"
+      env: LINT=1
+      before_install:
+        # In case we ever want to use a different version of clang-format:
+        #- wget -O - http://apt.llvm.org/llvm-snapshot.gpg.key | sudo apt-key add -
+        #- echo "deb http://apt.llvm.org/trusty/ llvm-toolchain-trusty main" | sudo tee -a /etc/apt/sources.list > /dev/null
+        - sudo apt-get update -qq
+        - sudo apt-get install -qq clang-format-3.8
+      install: []
+      script:
+        - .travis/check-git-clang-format-output.sh
 
 install:
   - make

.travis/check-git-clang-format-output.sh

@@ -0,0 +1,18 @@
+#!/bin/bash
+if [ "$TRAVIS_PULL_REQUEST" == "false" ] ; then
+  # Not in a pull request, so compare against parent commit
+  base_commit="HEAD^"
+  echo "Running clang-format against parent commit $(git rev-parse $base_commit)"
+else
+  base_commit="$TRAVIS_BRANCH"
+  echo "Running clang-format against branch $base_commit, with hash $(git rev-parse $base_commit)"
+fi
+output="$(.travis/git-clang-format --binary clang-format-3.8 --commit $base_commit --diff)"
+if [ "$output" == "no modified files to format" ] || [ "$output" == "clang-format did not modify any files" ] ; then
+  echo "clang-format passed."
+  exit 0
+else
+  echo "clang-format failed:"
+  echo "$output"
+  exit 1
+fi

.travis/git-clang-format

@@ -0,0 +1,485 @@
+#!/usr/bin/env python
+#
+#===- git-clang-format - ClangFormat Git Integration ---------*- python -*--===#
+#
+#                     The LLVM Compiler Infrastructure
+#
+# This file is distributed under the University of Illinois Open Source
+# License. See LICENSE.TXT for details.
+#
+#===------------------------------------------------------------------------===#
+
+r"""                                                                             
+clang-format git integration                                                     
+============================                                                     
+                                                                                 
+This file provides a clang-format integration for git. Put it somewhere in your  
+path and ensure that it is executable. Then, "git clang-format" will invoke      
+clang-format on the changes in current files or a specific commit.               
+                                                                                 
+For further details, run:                                                        
+git clang-format -h                                                              
+                                                                                 
+Requires Python 2.7                                                              
+"""               
+
+import argparse
+import collections
+import contextlib
+import errno
+import os
+import re
+import subprocess
+import sys
+
+usage = 'git clang-format [OPTIONS] [<commit>] [--] [<file>...]'
+
+desc = '''
+Run clang-format on all lines that differ between the working directory
+and <commit>, which defaults to HEAD.  Changes are only applied to the working
+directory.
+
+The following git-config settings set the default of the corresponding option:
+  clangFormat.binary
+  clangFormat.commit
+  clangFormat.extension
+  clangFormat.style
+'''
+
+# Name of the temporary index file in which save the output of clang-format.
+# This file is created within the .git directory.
+temp_index_basename = 'clang-format-index'
+
+
+Range = collections.namedtuple('Range', 'start, count')
+
+
+def main():
+  config = load_git_config()
+
+  # In order to keep '--' yet allow options after positionals, we need to
+  # check for '--' ourselves.  (Setting nargs='*' throws away the '--', while
+  # nargs=argparse.REMAINDER disallows options after positionals.)
+  argv = sys.argv[1:]
+  try:
+    idx = argv.index('--')
+  except ValueError:
+    dash_dash = []
+  else:
+    dash_dash = argv[idx:]
+    argv = argv[:idx]
+
+  default_extensions = ','.join([
+      # From clang/lib/Frontend/FrontendOptions.cpp, all lower case
+      'c', 'h',  # C
+      'm',  # ObjC
+      'mm',  # ObjC++
+      'cc', 'cp', 'cpp', 'c++', 'cxx', 'hpp',  # C++
+      # Other languages that clang-format supports
+      'proto', 'protodevel',  # Protocol Buffers
+      'js',  # JavaScript
+      'ts',  # TypeScript
+      ])
+
+  p = argparse.ArgumentParser(
+    usage=usage, formatter_class=argparse.RawDescriptionHelpFormatter,
+    description=desc)
+  p.add_argument('--binary',
+                 default=config.get('clangformat.binary', 'clang-format'),
+                 help='path to clang-format'),
+  p.add_argument('--commit',
+                 default=config.get('clangformat.commit', 'HEAD'),
+                 help='default commit to use if none is specified'),
+  p.add_argument('--diff', action='store_true',
+                 help='print a diff instead of applying the changes')
+  p.add_argument('--extensions',
+                 default=config.get('clangformat.extensions',
+                                    default_extensions),
+                 help=('comma-separated list of file extensions to format, '
+                       'excluding the period and case-insensitive')),
+  p.add_argument('-f', '--force', action='store_true',
+                 help='allow changes to unstaged files')
+  p.add_argument('-p', '--patch', action='store_true',
+                 help='select hunks interactively')
+  p.add_argument('-q', '--quiet', action='count', default=0,
+                 help='print less information')
+  p.add_argument('--style',
+                 default=config.get('clangformat.style', None),
+                 help='passed to clang-format'),
+  p.add_argument('-v', '--verbose', action='count', default=0,
+                 help='print extra information')
+  # We gather all the remaining positional arguments into 'args' since we need
+  # to use some heuristics to determine whether or not <commit> was present.
+  # However, to print pretty messages, we make use of metavar and help.
+  p.add_argument('args', nargs='*', metavar='<commit>',
+                 help='revision from which to compute the diff')
+  p.add_argument('ignored', nargs='*', metavar='<file>...',
+                 help='if specified, only consider differences in these files')
+  opts = p.parse_args(argv)
+
+  opts.verbose -= opts.quiet
+  del opts.quiet
+
+  commit, files = interpret_args(opts.args, dash_dash, opts.commit)
+  changed_lines = compute_diff_and_extract_lines(commit, files)
+  if opts.verbose >= 1:
+    ignored_files = set(changed_lines)
+  filter_by_extension(changed_lines, opts.extensions.lower().split(','))
+  if opts.verbose >= 1:
+    ignored_files.difference_update(changed_lines)
+    if ignored_files:
+      print 'Ignoring changes in the following files (wrong extension):'
+      for filename in ignored_files:
+        print '   ', filename
+    if changed_lines:
+      print 'Running clang-format on the following files:'
+      for filename in changed_lines:
+        print '   ', filename
+  if not changed_lines:
+    print 'no modified files to format'
+    return
+  # The computed diff outputs absolute paths, so we must cd before accessing
+  # those files.
+  cd_to_toplevel()
+  old_tree = create_tree_from_workdir(changed_lines)
+  new_tree = run_clang_format_and_save_to_tree(changed_lines,
+                                               binary=opts.binary,
+                                               style=opts.style)
+  if opts.verbose >= 1:
+    print 'old tree:', old_tree
+    print 'new tree:', new_tree
+  if old_tree == new_tree:
+    if opts.verbose >= 0:
+      print 'clang-format did not modify any files'
+  elif opts.diff:
+    print_diff(old_tree, new_tree)
+  else:
+    changed_files = apply_changes(old_tree, new_tree, force=opts.force,
+                                  patch_mode=opts.patch)
+    if (opts.verbose >= 0 and not opts.patch) or opts.verbose >= 1:
+      print 'changed files:'
+      for filename in changed_files:
+        print '   ', filename
+
+
+def load_git_config(non_string_options=None):
+  """Return the git configuration as a dictionary.
+
+  All options are assumed to be strings unless in `non_string_options`, in which
+  is a dictionary mapping option name (in lower case) to either "--bool" or
+  "--int"."""
+  if non_string_options is None:
+    non_string_options = {}
+  out = {}
+  for entry in run('git', 'config', '--list', '--null').split('\0'):
+    if entry:
+      name, value = entry.split('\n', 1)
+      if name in non_string_options:
+        value = run('git', 'config', non_string_options[name], name)
+      out[name] = value
+  return out
+
+
+def interpret_args(args, dash_dash, default_commit):
+  """Interpret `args` as "[commit] [--] [files...]" and return (commit, files).
+
+  It is assumed that "--" and everything that follows has been removed from
+  args and placed in `dash_dash`.
+
+  If "--" is present (i.e., `dash_dash` is non-empty), the argument to its
+  left (if present) is taken as commit.  Otherwise, the first argument is
+  checked if it is a commit or a file.  If commit is not given,
+  `default_commit` is used."""
+  if dash_dash:
+    if len(args) == 0:
+      commit = default_commit
+    elif len(args) > 1:
+      die('at most one commit allowed; %d given' % len(args))
+    else:
+      commit = args[0]
+    object_type = get_object_type(commit)
+    if object_type not in ('commit', 'tag'):
+      if object_type is None:
+        die("'%s' is not a commit" % commit)
+      else:
+        die("'%s' is a %s, but a commit was expected" % (commit, object_type))
+    files = dash_dash[1:]
+  elif args:
+    if disambiguate_revision(args[0]):
+      commit = args[0]
+      files = args[1:]
+    else:
+      commit = default_commit
+      files = args
+  else:
+    commit = default_commit
+    files = []
+  return commit, files
+
+
+def disambiguate_revision(value):
+  """Returns True if `value` is a revision, False if it is a file, or dies."""
+  # If `value` is ambiguous (neither a commit nor a file), the following
+  # command will die with an appropriate error message.
+  run('git', 'rev-parse', value, verbose=False)
+  object_type = get_object_type(value)
+  if object_type is None:
+    return False
+  if object_type in ('commit', 'tag'):
+    return True
+  die('`%s` is a %s, but a commit or filename was expected' %
+      (value, object_type))
+
+
+def get_object_type(value):
+  """Returns a string description of an object's type, or None if it is not
+  a valid git object."""
+  cmd = ['git', 'cat-file', '-t', value]
+  p = subprocess.Popen(cmd, stdout=subprocess.PIPE, stderr=subprocess.PIPE)
+  stdout, stderr = p.communicate()
+  if p.returncode != 0:
+    return None
+  return stdout.strip()
+
+
+def compute_diff_and_extract_lines(commit, files):
+  """Calls compute_diff() followed by extract_lines()."""
+  diff_process = compute_diff(commit, files)
+  changed_lines = extract_lines(diff_process.stdout)
+  diff_process.stdout.close()
+  diff_process.wait()
+  if diff_process.returncode != 0:
+    # Assume error was already printed to stderr.
+    sys.exit(2)
+  return changed_lines
+
+
+def compute_diff(commit, files):
+  """Return a subprocess object producing the diff from `commit`.
+
+  The return value's `stdin` file object will produce a patch with the
+  differences between the working directory and `commit`, filtered on `files`
+  (if non-empty).  Zero context lines are used in the patch."""
+  cmd = ['git', 'diff-index', '-p', '-U0', commit, '--']
+  cmd.extend(files)
+  p = subprocess.Popen(cmd, stdin=subprocess.PIPE, stdout=subprocess.PIPE)
+  p.stdin.close()
+  return p
+
+
+def extract_lines(patch_file):
+  """Extract the changed lines in `patch_file`.
+
+  The return value is a dictionary mapping filename to a list of (start_line,
+  line_count) pairs.
+
+  The input must have been produced with ``-U0``, meaning unidiff format with
+  zero lines of context.  The return value is a dict mapping filename to a
+  list of line `Range`s."""
+  matches = {}
+  for line in patch_file:
+    match = re.search(r'^\+\+\+\ [^/]+/(.*)', line)
+    if match:
+      filename = match.group(1).rstrip('\r\n')
+    match = re.search(r'^@@ -[0-9,]+ \+(\d+)(,(\d+))?', line)
+    if match:
+      start_line = int(match.group(1))
+      line_count = 1
+      if match.group(3):
+        line_count = int(match.group(3))
+      if line_count > 0:
+        matches.setdefault(filename, []).append(Range(start_line, line_count))
+  return matches
+
+
+def filter_by_extension(dictionary, allowed_extensions):
+  """Delete every key in `dictionary` that doesn't have an allowed extension.
+
+  `allowed_extensions` must be a collection of lowercase file extensions,
+  excluding the period."""
+  allowed_extensions = frozenset(allowed_extensions)
+  for filename in dictionary.keys():
+    base_ext = filename.rsplit('.', 1)
+    if len(base_ext) == 1 or base_ext[1].lower() not in allowed_extensions:
+      del dictionary[filename]
+
+
+def cd_to_toplevel():
+  """Change to the top level of the git repository."""
+  toplevel = run('git', 'rev-parse', '--show-toplevel')
+  os.chdir(toplevel)
+
+
+def create_tree_from_workdir(filenames):
+  """Create a new git tree with the given files from the working directory.
+
+  Returns the object ID (SHA-1) of the created tree."""
+  return create_tree(filenames, '--stdin')
+
+
+def run_clang_format_and_save_to_tree(changed_lines, binary='clang-format',
+                                      style=None):
+  """Run clang-format on each file and save the result to a git tree.
+
+  Returns the object ID (SHA-1) of the created tree."""
+  def index_info_generator():
+    for filename, line_ranges in changed_lines.iteritems():
+      mode = oct(os.stat(filename).st_mode)
+      blob_id = clang_format_to_blob(filename, line_ranges, binary=binary,
+                                     style=style)
+      yield '%s %s\t%s' % (mode, blob_id, filename)
+  return create_tree(index_info_generator(), '--index-info')
+
+
+def create_tree(input_lines, mode):
+  """Create a tree object from the given input.
+
+  If mode is '--stdin', it must be a list of filenames.  If mode is
+  '--index-info' is must be a list of values suitable for "git update-index
+  --index-info", such as "<mode> <SP> <sha1> <TAB> <filename>".  Any other mode
+  is invalid."""
+  assert mode in ('--stdin', '--index-info')
+  cmd = ['git', 'update-index', '--add', '-z', mode]
+  with temporary_index_file():
+    p = subprocess.Popen(cmd, stdin=subprocess.PIPE)
+    for line in input_lines:
+      p.stdin.write('%s\0' % line)
+    p.stdin.close()
+    if p.wait() != 0:
+      die('`%s` failed' % ' '.join(cmd))
+    tree_id = run('git', 'write-tree')
+    return tree_id
+
+
+def clang_format_to_blob(filename, line_ranges, binary='clang-format',
+                         style=None):
+  """Run clang-format on the given file and save the result to a git blob.
+
+  Returns the object ID (SHA-1) of the created blob."""
+  clang_format_cmd = [binary, filename]
+  if style:
+    clang_format_cmd.extend(['-style='+style])
+  clang_format_cmd.extend([
+      '-lines=%s:%s' % (start_line, start_line+line_count-1)
+      for start_line, line_count in line_ranges])
+  try:
+    clang_format = subprocess.Popen(clang_format_cmd, stdin=subprocess.PIPE,
+                                    stdout=subprocess.PIPE)
+  except OSError as e:
+    if e.errno == errno.ENOENT:
+      die('cannot find executable "%s"' % binary)
+    else:
+      raise
+  clang_format.stdin.close()
+  hash_object_cmd = ['git', 'hash-object', '-w', '--path='+filename, '--stdin']
+  hash_object = subprocess.Popen(hash_object_cmd, stdin=clang_format.stdout,
+                                 stdout=subprocess.PIPE)
+  clang_format.stdout.close()
+  stdout = hash_object.communicate()[0]
+  if hash_object.returncode != 0:
+    die('`%s` failed' % ' '.join(hash_object_cmd))
+  if clang_format.wait() != 0:
+    die('`%s` failed' % ' '.join(clang_format_cmd))
+  return stdout.rstrip('\r\n')
+
+
+@contextlib.contextmanager
+def temporary_index_file(tree=None):
+  """Context manager for setting GIT_INDEX_FILE to a temporary file and deleting
+  the file afterward."""
+  index_path = create_temporary_index(tree)
+  old_index_path = os.environ.get('GIT_INDEX_FILE')
+  os.environ['GIT_INDEX_FILE'] = index_path
+  try:
+    yield
+  finally:
+    if old_index_path is None:
+      del os.environ['GIT_INDEX_FILE']
+    else:
+      os.environ['GIT_INDEX_FILE'] = old_index_path
+    os.remove(index_path)
+
+
+def create_temporary_index(tree=None):
+  """Create a temporary index file and return the created file's path.
+
+  If `tree` is not None, use that as the tree to read in.  Otherwise, an
+  empty index is created."""
+  gitdir = run('git', 'rev-parse', '--git-dir')
+  path = os.path.join(gitdir, temp_index_basename)
+  if tree is None:
+    tree = '--empty'
+  run('git', 'read-tree', '--index-output='+path, tree)
+  return path
+
+
+def print_diff(old_tree, new_tree):
+  """Print the diff between the two trees to stdout."""
+  # We use the porcelain 'diff' and not plumbing 'diff-tree' because the output
+  # is expected to be viewed by the user, and only the former does nice things
+  # like color and pagination.
+  subprocess.check_call(['git', 'diff', old_tree, new_tree, '--'])
+
+
+def apply_changes(old_tree, new_tree, force=False, patch_mode=False):
+  """Apply the changes in `new_tree` to the working directory.
+
+  Bails if there are local changes in those files and not `force`.  If
+  `patch_mode`, runs `git checkout --patch` to select hunks interactively."""
+  changed_files = run('git', 'diff-tree', '-r', '-z', '--name-only', old_tree,
+                      new_tree).rstrip('\0').split('\0')
+  if not force:
+    unstaged_files = run('git', 'diff-files', '--name-status', *changed_files)
+    if unstaged_files:
+      print >>sys.stderr, ('The following files would be modified but '
+                           'have unstaged changes:')
+      print >>sys.stderr, unstaged_files
+      print >>sys.stderr, 'Please commit, stage, or stash them first.'
+      sys.exit(2)
+  if patch_mode:
+    # In patch mode, we could just as well create an index from the new tree
+    # and checkout from that, but then the user will be presented with a
+    # message saying "Discard ... from worktree".  Instead, we use the old
+    # tree as the index and checkout from new_tree, which gives the slightly
+    # better message, "Apply ... to index and worktree".  This is not quite
+    # right, since it won't be applied to the user's index, but oh well.
+    with temporary_index_file(old_tree):
+      subprocess.check_call(['git', 'checkout', '--patch', new_tree])
+    index_tree = old_tree
+  else:
+    with temporary_index_file(new_tree):
+      run('git', 'checkout-index', '-a', '-f')
+  return changed_files
+
+
+def run(*args, **kwargs):
+  stdin = kwargs.pop('stdin', '')
+  verbose = kwargs.pop('verbose', True)
+  strip = kwargs.pop('strip', True)
+  for name in kwargs:
+    raise TypeError("run() got an unexpected keyword argument '%s'" % name)
+  p = subprocess.Popen(args, stdout=subprocess.PIPE, stderr=subprocess.PIPE,
+                       stdin=subprocess.PIPE)
+  stdout, stderr = p.communicate(input=stdin)
+  if p.returncode == 0:
+    if stderr:
+      if verbose:
+        print >>sys.stderr, '`%s` printed to stderr:' % ' '.join(args)
+      print >>sys.stderr, stderr.rstrip()
+    if strip:
+      stdout = stdout.rstrip('\r\n')
+    return stdout
+  if verbose:
+    print >>sys.stderr, '`%s` returned %s' % (' '.join(args), p.returncode)
+  if stderr:
+    print >>sys.stderr, stderr.rstrip()
+  sys.exit(2)
+
+
+def die(message):
+  print >>sys.stderr, 'error:', message
+  sys.exit(2)
+
+
+if __name__ == '__main__':
+  main()

README.md

@@ -2,3 +2,6 @@
 
 Plasma is an experimental in-memory object manager. It is under development and
 not ready for general use.
+
+## clang-format
+Run .travis/git-clang-format to automatically format changes in the checkout.

src/event_loop.c

@@ -3,8 +3,8 @@
 #include <assert.h>
 #include <unistd.h>
 
-UT_icd item_icd = { sizeof(event_loop_item), NULL, NULL, NULL };
-UT_icd poll_icd = { sizeof(struct pollfd), NULL, NULL, NULL };
+UT_icd item_icd = {sizeof(event_loop_item), NULL, NULL, NULL};
+UT_icd poll_icd = {sizeof(struct pollfd), NULL, NULL, NULL};
 
 /* Initializes the event loop.
  * This function needs to be called before any other event loop function. */
@@ -18,15 +18,19 @@ void event_loop_init(event_loop *loop) {
  * which can be queried using event_loop_type and event_loop_id. The parameter
  * events is the same as in http://linux.die.net/man/2/poll.
  * Returns the index of the item in the event loop. */
-int64_t event_loop_attach(event_loop *loop, int type, data_connection* connection, int fd, int events) {
+int64_t event_loop_attach(event_loop *loop,
+                          int type,
+                          data_connection *connection,
+                          int fd,
+                          int events) {
   assert(utarray_len(loop->items) == utarray_len(loop->waiting));
   int64_t index = utarray_len(loop->items);
-  event_loop_item item = { .type = type };
+  event_loop_item item = {.type = type};
   if (connection) {
     item.connection = *connection;
   }
-  utarray_push_back(loop->items, &item );
-  struct pollfd waiting = { .fd = fd, .events = events };
+  utarray_push_back(loop->items, &item);
+  struct pollfd waiting = {.fd = fd, .events = events};
   utarray_push_back(loop->waiting, &waiting);
   return index;
 }
@@ -35,16 +39,18 @@ int64_t event_loop_attach(event_loop *loop, int type, data_connection* connectio
  * This invalidates all other indices into the event loop items, but leaves
  * the ids of the event loop items valid. */
 void event_loop_detach(event_loop *loop, int64_t index, int shall_close) {
-  struct pollfd *waiting_item = (struct pollfd*) utarray_eltptr(loop->waiting, index);
-  struct pollfd *waiting_back = (struct pollfd*) utarray_back(loop->waiting);
+  struct pollfd *waiting_item =
+      (struct pollfd *) utarray_eltptr(loop->waiting, index);
+  struct pollfd *waiting_back = (struct pollfd *) utarray_back(loop->waiting);
   if (shall_close) {
     close(waiting_item->fd);
   }
   *waiting_item = *waiting_back;
   utarray_pop_back(loop->waiting);
 
-  event_loop_item *items_item = (event_loop_item*) utarray_eltptr(loop->items, index);
-  event_loop_item *items_back = (event_loop_item*) utarray_back(loop->items);
+  event_loop_item *items_item =
+      (event_loop_item *) utarray_eltptr(loop->items, index);
+  event_loop_item *items_back = (event_loop_item *) utarray_back(loop->items);
   *items_item = *items_back;
   utarray_pop_back(loop->items);
 }
@@ -52,7 +58,8 @@ void event_loop_detach(event_loop *loop, int64_t index, int shall_close) {
 /* Poll the file descriptors associated to this event loop.
  * See http://linux.die.net/man/2/poll */
 int event_loop_poll(event_loop *loop) {
-  return poll((struct pollfd*) utarray_front(loop->waiting), utarray_len(loop->waiting), -1);
+  return poll((struct pollfd *) utarray_front(loop->waiting),
+              utarray_len(loop->waiting), -1);
 }
 
 /* Get the total number of file descriptors participating in the event loop. */
@@ -60,20 +67,25 @@ int64_t event_loop_size(event_loop *loop) {
   return utarray_len(loop->waiting);
 }
 
-/* Get the pollfd structure associated to a file descriptor participating in the event loop. */
+/* Get the pollfd structure associated to a file descriptor participating in the
+ * event loop. */
 struct pollfd *event_loop_get(event_loop *loop, int64_t index) {
-  return (struct pollfd*) utarray_eltptr(loop->waiting, index);
+  return (struct pollfd *) utarray_eltptr(loop->waiting, index);
 }
 
 /* Set the data connection information for participant in the event loop. */
-void event_loop_set_connection(event_loop *loop, int64_t index, const data_connection* conn) {
-  event_loop_item *item = (event_loop_item*) utarray_eltptr(loop->items, index);
+void event_loop_set_connection(event_loop *loop,
+                               int64_t index,
+                               const data_connection *conn) {
+  event_loop_item *item =
+      (event_loop_item *) utarray_eltptr(loop->items, index);
   item->connection = *conn;
 }
 
 /* Get the data connection information for participant in the event loop. */
-data_connection* event_loop_get_connection(event_loop *loop, int64_t index) {
-  event_loop_item *item = (event_loop_item*) utarray_eltptr(loop->items, index);
+data_connection *event_loop_get_connection(event_loop *loop, int64_t index) {
+  event_loop_item *item =
+      (event_loop_item *) utarray_eltptr(loop->items, index);
   return &item->connection;
 }
 

src/event_loop.h

@@ -12,25 +12,31 @@ typedef struct {
   int type;
   /* If type is data transfer, this contains information about the status
    * of the transfer. */
-	data_connection connection;
+  data_connection connection;
 } event_loop_item;
 
 typedef struct {
   /* Array of event_loop_items that hold information for connections. */
-  UT_array *items; 
+  UT_array *items;
   /* Array of file descriptors that are waiting, corresponding to items. */
-  UT_array *waiting; 
+  UT_array *waiting;
 } event_loop;
 
 /* Event loop functions. */
 void event_loop_init(event_loop *loop);
 void event_loop_free(event_loop *loop);
-int64_t event_loop_attach(event_loop *loop, int type, data_connection* connection, int fd, int events);
+int64_t event_loop_attach(event_loop *loop,
+                          int type,
+                          data_connection *connection,
+                          int fd,
+                          int events);
 void event_loop_detach(event_loop *loop, int64_t index, int shall_close);
 int event_loop_poll(event_loop *loop);
 int64_t event_loop_size(event_loop *loop);
 struct pollfd *event_loop_get(event_loop *loop, int64_t index);
-void event_loop_set_connection(event_loop *loop, int64_t index, const data_connection* conn);
+void event_loop_set_connection(event_loop *loop,
+                               int64_t index,
+                               const data_connection *conn);
 data_connection *event_loop_get_connection(event_loop *loop, int64_t index);
 
 #endif

src/example.c

@@ -1,7 +1,7 @@
 /* A simple example on how to use the plasma store
- * 
+ *
  * Can be called in the following way:
- * 
+ *
  * cd build
  * ./plasma_store -s /tmp/plasma_socket
  * ./example -s /tmp/plasma_socket -g
@@ -19,8 +19,8 @@ int main(int argc, char *argv[]) {
   int64_t size;
   void *data;
   int c;
-  plasma_id id = {{255, 255, 255, 255, 255, 255, 255, 255, 255, 255, 255, 255,
-		   255, 255, 255, 255, 255, 255, 255, 255}};
+  plasma_id id = {{255, 255, 255, 255, 255, 255, 255, 255, 255, 255,
+                   255, 255, 255, 255, 255, 255, 255, 255, 255, 255}};
   while ((c = getopt(argc, argv, "s:cfg")) != -1) {
     switch (c) {
     case 's':

src/fling.c

@@ -1,7 +1,9 @@
 #include "fling.h"
 
-void init_msg(struct msghdr *msg, struct iovec *iov,
-	      char *buf, size_t buf_len) {
+void init_msg(struct msghdr *msg,
+              struct iovec *iov,
+              char *buf,
+              size_t buf_len) {
   iov->iov_base = buf;
   iov->iov_len = 1;
 
@@ -13,7 +15,7 @@ void init_msg(struct msghdr *msg, struct iovec *iov,
   msg->msg_namelen = 0;
 }
 
-int send_fd(int conn, int fd, const char* payload, int size) {
+int send_fd(int conn, int fd, const char *payload, int size) {
   struct msghdr msg;
   struct iovec iov;
   char buf[CMSG_SPACE(sizeof(int))];
@@ -24,13 +26,13 @@ int send_fd(int conn, int fd, const char* payload, int size) {
   header->cmsg_level = SOL_SOCKET;
   header->cmsg_type = SCM_RIGHTS;
   header->cmsg_len = CMSG_LEN(sizeof(int));
-  *(int *)CMSG_DATA(header) = fd;
+  *(int *) CMSG_DATA(header) = fd;
 
   /* send file descriptor and payload */
   return sendmsg(conn, &msg, 0) != -1 && send(conn, payload, size, 0) == -1;
 }
 
-int recv_fd(int conn, char* payload, int size) {
+int recv_fd(int conn, char *payload, int size) {
   struct msghdr msg;
   struct iovec iov;
   char buf[CMSG_SPACE(sizeof(int))];
@@ -41,11 +43,14 @@ int recv_fd(int conn, char* payload, int size) {
 
   int found_fd = -1;
   int oh_noes = 0;
-  for (struct cmsghdr *header = CMSG_FIRSTHDR(&msg); header != NULL; header = CMSG_NXTHDR(&msg, header))
+  for (struct cmsghdr *header = CMSG_FIRSTHDR(&msg); header != NULL;
+       header = CMSG_NXTHDR(&msg, header))
     if (header->cmsg_level == SOL_SOCKET && header->cmsg_type == SCM_RIGHTS) {
-      int count = (header->cmsg_len - (CMSG_DATA(header) - (unsigned char *)header)) / sizeof(int);
+      int count =
+          (header->cmsg_len - (CMSG_DATA(header) - (unsigned char *) header)) /
+          sizeof(int);
       for (int i = 0; i < count; ++i) {
-        int fd = ((int *)CMSG_DATA(header))[i];
+        int fd = ((int *) CMSG_DATA(header))[i];
         if (found_fd == -1) {
           found_fd = fd;
         } else {

src/fling.h

@@ -15,20 +15,21 @@
 #include <sys/socket.h>
 #include <sys/un.h>
 
-/* This is neccessary for Mac OS X, see http://www.apuebook.com/faqs2e.html (10). */
+/* This is neccessary for Mac OS X, see http://www.apuebook.com/faqs2e.html
+ * (10). */
 #if !defined(CMSG_SPACE) && !defined(CMSG_LEN)
-  #define CMSG_SPACE(len) (__DARWIN_ALIGN32(sizeof(struct cmsghdr)) + __DARWIN_ALIGN32(len))
-  #define CMSG_LEN(len) (__DARWIN_ALIGN32(sizeof(struct cmsghdr)) + (len))
+#define CMSG_SPACE(len) \
+  (__DARWIN_ALIGN32(sizeof(struct cmsghdr)) + __DARWIN_ALIGN32(len))
+#define CMSG_LEN(len) (__DARWIN_ALIGN32(sizeof(struct cmsghdr)) + (len))
 #endif
 
-void init_msg(struct msghdr *msg, struct iovec *iov,
-              char *buf, size_t buf_len);
+void init_msg(struct msghdr *msg, struct iovec *iov, char *buf, size_t buf_len);
 
 /* Send a file descriptor "fd" and a payload "payload" of size "size"
  * over the socket "conn". Return 0 on success. */
-int send_fd(int conn, int fd, const char* payload, int size);
+int send_fd(int conn, int fd, const char *payload, int size);
 
 /* Receive a file descriptor and a payload of size up to "size" from a
  * socket "conn". The payload will be written to "payload" and the file
  * descriptor will be returned. Returns -1 on failure. */
-int recv_fd(int conn, char* payload, int size);
+int recv_fd(int conn, char *payload, int size);

src/plasma.h

@@ -7,15 +7,15 @@
 #include <string.h>
 
 #ifdef NDEBUG
-  #define LOG_DEBUG(M, ...)
+#define LOG_DEBUG(M, ...)
 #else
-  #define LOG_DEBUG(M, ...) \
-    fprintf(stderr, "[DEBUG] (%s:%d) " M "\n", __FILE__, __LINE__, ##__VA_ARGS__)
+#define LOG_DEBUG(M, ...) \
+  fprintf(stderr, "[DEBUG] (%s:%d) " M "\n", __FILE__, __LINE__, ##__VA_ARGS__)
 #endif
 
-#define LOG_ERR(M, ...) \
-  fprintf(stderr, "[ERROR] (%s:%d: errno: %s) " M "\n", \
-    __FILE__, __LINE__, errno == 0 ? "None" : strerror(errno), ##__VA_ARGS__)
+#define LOG_ERR(M, ...)                                                     \
+  fprintf(stderr, "[ERROR] (%s:%d: errno: %s) " M "\n", __FILE__, __LINE__, \
+          errno == 0 ? "None" : strerror(errno), ##__VA_ARGS__)
 
 #define LOG_INFO(M, ...) \
   fprintf(stderr, "[INFO] (%s:%d) " M "\n", __FILE__, __LINE__, ##__VA_ARGS__)
@@ -27,9 +27,7 @@ typedef struct {
 } plasma_object_info;
 
 /* Represents an object id hash, can hold a full SHA1 hash */
-typedef struct {
-  unsigned char id[20];
-} plasma_id;
+typedef struct { unsigned char id[20]; } plasma_id;
 
 enum plasma_request_type {
   /* Create a new object. */
@@ -72,10 +70,10 @@ typedef struct {
 } plasma_buffer;
 
 /* Connect to the local plasma store UNIX domain socket */
-int plasma_store_connect(const char* socket_name);
+int plasma_store_connect(const char *socket_name);
 
 /* Connect to a possibly remote plasma manager */
-int plasma_manager_connect(const char* addr, int port);
+int plasma_manager_connect(const char *addr, int port);
 
 void plasma_create(int store, plasma_id object_id, int64_t size, void **data);
 void plasma_get(int store, plasma_id object_id, int64_t *size, void **data);

src/plasma_client.c

@@ -25,10 +25,11 @@ void plasma_send(int fd, plasma_request *req) {
 
 void plasma_create(int conn, plasma_id object_id, int64_t size, void **data) {
   LOG_INFO("called plasma_create on conn %d with size %" PRId64, conn, size);
-  plasma_request req = { .type = PLASMA_CREATE, .object_id = object_id, .size = size };
+  plasma_request req = {
+      .type = PLASMA_CREATE, .object_id = object_id, .size = size};
   plasma_send(conn, &req);
   plasma_reply reply;
-  int fd = recv_fd(conn, (char*)&reply, sizeof(plasma_reply));
+  int fd = recv_fd(conn, (char *) &reply, sizeof(plasma_reply));
   assert(reply.type == PLASMA_OBJECT);
   assert(reply.size == size);
   *data = mmap(NULL, reply.size, PROT_READ | PROT_WRITE, MAP_SHARED, fd, 0);
@@ -39,19 +40,19 @@ void plasma_create(int conn, plasma_id object_id, int64_t size, void **data) {
 }
 
 void plasma_get(int conn, plasma_id object_id, int64_t *size, void **data) {
-  plasma_request req = { .type = PLASMA_GET, .object_id = object_id };
+  plasma_request req = {.type = PLASMA_GET, .object_id = object_id};
   plasma_send(conn, &req);
   plasma_reply reply;
   /* The following loop is run at most twice. */
-  int fd = recv_fd(conn, (char*)&reply, sizeof(plasma_reply));
+  int fd = recv_fd(conn, (char *) &reply, sizeof(plasma_reply));
   if (reply.type == PLASMA_FUTURE) {
-    int new_fd = recv_fd(fd, (char*)&reply, sizeof(plasma_reply));
+    int new_fd = recv_fd(fd, (char *) &reply, sizeof(plasma_reply));
     close(fd);
     fd = new_fd;
   }
   assert(reply.type == PLASMA_OBJECT);
   *data = mmap(NULL, reply.size, PROT_READ, MAP_SHARED, fd, 0);
-  if (*data  == MAP_FAILED) {
+  if (*data == MAP_FAILED) {
     LOG_ERR("mmap failed");
     exit(-1);
   }
@@ -59,11 +60,11 @@ void plasma_get(int conn, plasma_id object_id, int64_t *size, void **data) {
 }
 
 void plasma_seal(int fd, plasma_id object_id) {
-  plasma_request req = { .type = PLASMA_SEAL, .object_id = object_id };
+  plasma_request req = {.type = PLASMA_SEAL, .object_id = object_id};
   plasma_send(fd, &req);
 }
 
-int plasma_store_connect(const char* socket_name) {
+int plasma_store_connect(const char *socket_name) {
   assert(socket_name);
   struct sockaddr_un addr;
   int fd;
@@ -73,11 +74,12 @@ int plasma_store_connect(const char* socket_name) {
   }
   memset(&addr, 0, sizeof(addr));
   addr.sun_family = AF_UNIX;
-  strncpy(addr.sun_path, socket_name, sizeof(addr.sun_path)-1);
-  /* Try to connect to the Plasma store. If unsuccessful, retry several times. */
+  strncpy(addr.sun_path, socket_name, sizeof(addr.sun_path) - 1);
+  /* Try to connect to the Plasma store. If unsuccessful, retry several times.
+   */
   int connected_successfully = 0;
   for (int num_attempts = 0; num_attempts < 50; ++num_attempts) {
-    if (connect(fd, (struct sockaddr*)&addr, sizeof(addr)) == 0) {
+    if (connect(fd, (struct sockaddr *) &addr, sizeof(addr)) == 0) {
       connected_successfully = 1;
       break;
     }
@@ -94,7 +96,7 @@ int plasma_store_connect(const char* socket_name) {
 
 #define h_addr h_addr_list[0]
 
-int plasma_manager_connect(const char* ip_addr, int port) {
+int plasma_manager_connect(const char *ip_addr, int port) {
   int fd = socket(PF_INET, SOCK_STREAM, 0);
   if (fd < 0) {
     LOG_ERR("could not create socket");
@@ -112,17 +114,22 @@ int plasma_manager_connect(const char* ip_addr, int port) {
   bcopy(manager->h_addr, &addr.sin_addr.s_addr, manager->h_length);
   addr.sin_port = htons(port);
 
-  int r = connect(fd, (struct sockaddr*) &addr, sizeof(addr));
+  int r = connect(fd, (struct sockaddr *) &addr, sizeof(addr));
   if (r < 0) {
-    LOG_ERR("could not establish connection to manager with id %s:%d", &ip_addr[0], port);
+    LOG_ERR("could not establish connection to manager with id %s:%d",
+            &ip_addr[0], port);
     exit(-1);
   }
   return fd;
 }
 
-void plasma_transfer(int manager, const char* addr, int port, plasma_id object_id) {
-  plasma_request req = {.type = PLASMA_TRANSFER, .object_id = object_id, .port = port};
-  char* end = NULL;
+void plasma_transfer(int manager,
+                     const char *addr,
+                     int port,
+                     plasma_id object_id) {
+  plasma_request req = {
+      .type = PLASMA_TRANSFER, .object_id = object_id, .port = port};
+  char *end = NULL;
   for (int i = 0; i < 4; ++i) {
     req.addr[i] = strtol(end ? end : addr, &end, 10);
     /* skip the '.' */

src/plasma_manager.c

@@ -34,7 +34,8 @@ typedef struct {
 /* Initialize the plasma manager. This function initializes the event loop
  * of the plasma manager, and stores the address 'store_socket_name' of
  * the local plasma store socket. */
-void init_plasma_manager(plasma_manager_state *s, const char* store_socket_name) {
+void init_plasma_manager(plasma_manager_state* s,
+                         const char* store_socket_name) {
   s->loop = malloc(sizeof(event_loop));
   event_loop_init(s->loop);
   s->store_socket_name = store_socket_name;
@@ -45,36 +46,47 @@ void init_plasma_manager(plasma_manager_state *s, const char* store_socket_name)
  * the data header to the other object manager. */
 void initiate_transfer(plasma_manager_state* s, plasma_request* req) {
   int store_conn = plasma_store_connect(s->store_socket_name);
-  plasma_buffer buf = { .object_id = req->object_id, .writable = 0 };
+  plasma_buffer buf = {.object_id = req->object_id, .writable = 0};
   plasma_get(store_conn, req->object_id, &buf.size, &buf.data);
-  
+
   char ip_addr[16];
-  snprintf(ip_addr, 32, "%d.%d.%d.%d",
-                    req->addr[0], req->addr[1],
-                    req->addr[2], req->addr[3]);
+  snprintf(ip_addr, 32, "%d.%d.%d.%d", req->addr[0], req->addr[1], req->addr[2],
+           req->addr[3]);
 
   int fd = plasma_manager_connect(&ip_addr[0], req->port);
-  data_connection conn = { .type = DATA_CONNECTION_WRITE,  .store_conn = store_conn, .buf = buf, .cursor = 0 };
+  data_connection conn = {.type = DATA_CONNECTION_WRITE,
+                          .store_conn = store_conn,
+                          .buf = buf,
+                          .cursor = 0};
   event_loop_attach(s->loop, CONNECTION_DATA, &conn, fd, POLLOUT);
 
-  plasma_request manager_req = { .type = PLASMA_DATA, .object_id = req->object_id, .size = buf.size };
+  plasma_request manager_req = {
+      .type = PLASMA_DATA, .object_id = req->object_id, .size = buf.size};
   plasma_send(fd, &manager_req);
 }
 
 /* Start reading data from another object manager.
  * Initializes the object we are going to write to in the
  * local plasma store and then switches the data socket to reading mode. */
-void start_reading_data(int64_t index, plasma_manager_state* s, plasma_request* req) {
+void start_reading_data(int64_t index,
+                        plasma_manager_state* s,
+                        plasma_request* req) {
   int store_conn = plasma_store_connect(s->store_socket_name);
-  plasma_buffer buf = { .object_id = req->object_id, .size = req->size, .writable = 1 };
+  plasma_buffer buf = {
+      .object_id = req->object_id, .size = req->size, .writable = 1};
   plasma_create(store_conn, req->object_id, req->size, &buf.data);
-  data_connection conn = { .type = DATA_CONNECTION_READ, .store_conn = store_conn, .buf = buf, .cursor = 0 };
+  data_connection conn = {.type = DATA_CONNECTION_READ,
+                          .store_conn = store_conn,
+                          .buf = buf,
+                          .cursor = 0};
   event_loop_set_connection(s->loop, index, &conn);
 }
 
 /* Handle a command request that came in through a socket (transfering data,
  * or accepting incoming data). */
-void process_command(int64_t id, plasma_manager_state* state, plasma_request* req) {
+void process_command(int64_t id,
+                     plasma_manager_state* state,
+                     plasma_request* req) {
   switch (req->type) {
   case PLASMA_TRANSFER:
     LOG_INFO("transfering object to manager with port %d", req->port);
@@ -91,63 +103,66 @@ void process_command(int64_t id, plasma_manager_state* state, plasma_request* re
 }
 
 /* Handle data or command event incoming on socket with index "index". */
-void read_from_socket(plasma_manager_state* state, struct pollfd *waiting, int64_t index, plasma_request* req) {
+void read_from_socket(plasma_manager_state* state,
+                      struct pollfd* waiting,
+                      int64_t index,
+                      plasma_request* req) {
   ssize_t r, s;
-  data_connection *conn = event_loop_get_connection(state->loop, index);
+  data_connection* conn = event_loop_get_connection(state->loop, index);
   switch (conn->type) {
-    case DATA_CONNECTION_HEADER:
-      r = read(waiting->fd, req, sizeof(plasma_request));
-      if (r == -1) {
-        LOG_ERR("read error");
-      } else if (r == 0) {
-        LOG_INFO("connection with id %" PRId64 " disconnected", index);
-        event_loop_detach(state->loop, index, 1);
+  case DATA_CONNECTION_HEADER:
+    r = read(waiting->fd, req, sizeof(plasma_request));
+    if (r == -1) {
+      LOG_ERR("read error");
+    } else if (r == 0) {
+      LOG_INFO("connection with id %" PRId64 " disconnected", index);
+      event_loop_detach(state->loop, index, 1);
+    } else {
+      process_command(index, state, req);
+    }
+    break;
+  case DATA_CONNECTION_READ:
+    LOG_DEBUG("polled DATA_CONNECTION_READ");
+    r = read(waiting->fd, conn->buf.data + conn->cursor, BUFSIZE);
+    if (r == -1) {
+      LOG_ERR("read error");
+    } else if (r == 0) {
+      LOG_INFO("end of file");
+    } else {
+      conn->cursor += r;
+    }
+    if (r == 0) {
+      LOG_DEBUG("reading on channel %" PRId64 " finished", index);
+      plasma_seal(conn->store_conn, conn->buf.object_id);
+      close(conn->store_conn);
+      event_loop_detach(state->loop, index, 1);
+    }
+    break;
+  case DATA_CONNECTION_WRITE:
+    LOG_DEBUG("polled DATA_CONNECTION_WRITE");
+    s = conn->buf.size - conn->cursor;
+    if (s > BUFSIZE)
+      s = BUFSIZE;
+    r = write(waiting->fd, conn->buf.data + conn->cursor, s);
+    if (r != s) {
+      if (r > 0) {
+        LOG_ERR("partial write on fd %d", waiting->fd);
       } else {
-        process_command(index, state, req);
+        LOG_ERR("write error");
+        exit(-1);
       }
-      break;
-    case DATA_CONNECTION_READ:
-      LOG_DEBUG("polled DATA_CONNECTION_READ");
-      r = read(waiting->fd, conn->buf.data + conn->cursor, BUFSIZE);
-      if (r == -1) {
-        LOG_ERR("read error");
-      } else if (r == 0) {
-        LOG_INFO("end of file");
-      } else {
-        conn->cursor += r;
-      }
-      if (r == 0) {
-        LOG_DEBUG("reading on channel %" PRId64 " finished", index);
-        plasma_seal(conn->store_conn, conn->buf.object_id);
-        close(conn->store_conn);
-        event_loop_detach(state->loop, index, 1);
-      }
-      break;
-    case DATA_CONNECTION_WRITE:
-      LOG_DEBUG("polled DATA_CONNECTION_WRITE");
-      s = conn->buf.size - conn->cursor;
-      if (s > BUFSIZE)
-        s = BUFSIZE;
-      r = write(waiting->fd, conn->buf.data + conn->cursor, s);
-      if (r != s) {
-        if (r > 0) {
-          LOG_ERR("partial write on fd %d", waiting->fd);
-        } else {
-          LOG_ERR("write error");
-          exit(-1);
-        }
-      } else {
-        conn->cursor += r;
-      }
-      if (r == 0) {
-        LOG_DEBUG("writing on channel %" PRId64 " finished", index);
-        close(conn->store_conn);
-        event_loop_detach(state->loop, index, 1);
-      }
-      break;
-    default:
-      LOG_ERR("invalid connection type");
-      exit(-1);
+    } else {
+      conn->cursor += r;
+    }
+    if (r == 0) {
+      LOG_DEBUG("writing on channel %" PRId64 " finished", index);
+      close(conn->store_conn);
+      event_loop_detach(state->loop, index, 1);
+    }
+    break;
+  default:
+    LOG_ERR("invalid connection type");
+    exit(-1);
   }
 }
 
@@ -163,7 +178,7 @@ void run_event_loop(int sock, plasma_manager_state* s) {
       exit(-1);
     }
     for (int i = 0; i < event_loop_size(s->loop); ++i) {
-      struct pollfd *waiting = event_loop_get(s->loop, i);
+      struct pollfd* waiting = event_loop_get(s->loop, i);
       if (waiting->revents == 0)
         continue;
       if (waiting->fd == sock) {
@@ -176,7 +191,7 @@ void run_event_loop(int sock, plasma_manager_state* s) {
           }
           break;
         }
-        data_connection conn = { .type = DATA_CONNECTION_HEADER };
+        data_connection conn = {.type = DATA_CONNECTION_HEADER};
         event_loop_attach(s->loop, CONNECTION_DATA, &conn, new_socket, POLLIN);
         LOG_INFO("new connection with id %" PRId64, event_loop_size(s->loop));
       } else {
@@ -186,7 +201,9 @@ void run_event_loop(int sock, plasma_manager_state* s) {
   }
 }
 
-void start_server(const char *store_socket_name, const char* master_addr, int port) {
+void start_server(const char* store_socket_name,
+                  const char* master_addr,
+                  int port) {
   struct sockaddr_in name;
   int sock = socket(PF_INET, SOCK_STREAM, 0);
   if (sock < 0) {
@@ -203,7 +220,7 @@ void start_server(const char *store_socket_name, const char* master_addr, int po
     close(sock);
     exit(-1);
   }
-  setsockopt(sock, SOL_SOCKET, SO_REUSEADDR, &on, sizeof (on));
+  setsockopt(sock, SOL_SOCKET, SO_REUSEADDR, &on, sizeof(on));
   if (bind(sock, (struct sockaddr*) &name, sizeof(name)) < 0) {
     LOG_ERR("could not bind socket");
     exit(-1);
@@ -220,9 +237,9 @@ void start_server(const char *store_socket_name, const char* master_addr, int po
 
 int main(int argc, char* argv[]) {
   /* Socket name of the plasma store this manager is connected to. */
-  char *store_socket_name = NULL;
+  char* store_socket_name = NULL;
   /* IP address of this node. */
-  char *master_addr = NULL;
+  char* master_addr = NULL;
   /* Port number the manager should use. */
   int port;
   int c;
@@ -243,11 +260,15 @@ int main(int argc, char* argv[]) {
     }
   }
   if (!store_socket_name) {
-    LOG_ERR("please specify socket for connecting to the plasma store with -s switch");
+    LOG_ERR(
+        "please specify socket for connecting to the plasma store with -s "
+        "switch");
     exit(-1);
   }
   if (!master_addr) {
-    LOG_ERR("please specify ip address of the current host in the format 123.456.789.10 with -m switch");
+    LOG_ERR(
+        "please specify ip address of the current host in the format "
+        "123.456.789.10 with -m switch");
     exit(-1);
   }
   start_server(store_socket_name, master_addr, port);

src/plasma_manager.h

@@ -7,11 +7,7 @@
 /* The buffer size in bytes. Data will get transfered in multiples of this */
 #define BUFSIZE 4096
 
-enum connection_type {
-  CONNECTION_REDIS,
-  CONNECTION_LISTENER,
-  CONNECTION_DATA
-};
+enum connection_type { CONNECTION_REDIS, CONNECTION_LISTENER, CONNECTION_DATA };
 
 enum data_connection_type {
   /* Connection to send commands and metadata to the manager. */

src/plasma_store.c

@@ -9,7 +9,6 @@
  * It keeps a hash table that maps object_ids (which are 20 byte long,
  * just enough to store and SHA1 hash) to memory mapped files. */
 
-
 #include <stdio.h>
 #include <stdlib.h>
 #include <unistd.h>
@@ -30,7 +29,7 @@
 
 typedef struct {
   /* Event loop for the plasma store. */
-  event_loop *loop;
+  event_loop* loop;
 } plasma_store_state;
 
 void init_state(plasma_store_state* s) {
@@ -103,41 +102,42 @@ void create_object(int conn, plasma_request* req) {
     LOG_ERR("could not create shared memory buffer");
     exit(-1);
   }
-  object_table_entry *entry = malloc(sizeof(object_table_entry));
+  object_table_entry* entry = malloc(sizeof(object_table_entry));
   memcpy(&entry->object_id, &req->object_id, 20);
   entry->info.size = req->size;
   /* TODO(pcm): set the other fields */
   entry->fd = fd;
   HASH_ADD(handle, open_objects, object_id, sizeof(plasma_id), entry);
-  plasma_reply reply = { PLASMA_OBJECT, req->size };
+  plasma_reply reply = {PLASMA_OBJECT, req->size};
   send_fd(conn, fd, (char*) &reply, sizeof(plasma_reply));
 }
 
 /* Get an object from the hash table. */
 void get_object(int conn, plasma_request* req) {
-  object_table_entry *entry;
+  object_table_entry* entry;
   HASH_FIND(handle, sealed_objects, &req->object_id, sizeof(plasma_id), entry);
   if (entry) {
-    plasma_reply reply = { PLASMA_OBJECT, entry->info.size };
+    plasma_reply reply = {PLASMA_OBJECT, entry->info.size};
     send_fd(conn, entry->fd, (char*) &reply, sizeof(plasma_reply));
   } else {
     LOG_INFO("object not in hash table of sealed objects");
     int fd[2];
     socketpair(AF_UNIX, SOCK_STREAM, 0, fd);
-    object_notify_entry *notify_entry = malloc(sizeof(object_notify_entry));
+    object_notify_entry* notify_entry = malloc(sizeof(object_notify_entry));
     memcpy(&notify_entry->object_id, &req->object_id, 20);
     notify_entry->conn[notify_entry->num_waiting] = fd[0];
     notify_entry->num_waiting += 1;
-    HASH_ADD(handle, objects_notify, object_id, sizeof(plasma_id), notify_entry);
-    plasma_reply reply = { PLASMA_FUTURE, -1 };
+    HASH_ADD(handle, objects_notify, object_id, sizeof(plasma_id),
+             notify_entry);
+    plasma_reply reply = {PLASMA_FUTURE, -1};
     send_fd(conn, fd[1], (char*) &reply, sizeof(plasma_reply));
   }
 }
 
 /* Seal an object that has been created in the hash table. */
 void seal_object(int conn, plasma_request* req) {
-  LOG_INFO("sealing object"); // TODO(pcm): add object_id here
-  object_table_entry *entry;
+  LOG_INFO("sealing object");  // TODO(pcm): add object_id here
+  object_table_entry* entry;
   HASH_FIND(handle, open_objects, &req->object_id, sizeof(plasma_id), entry);
   if (!entry) {
     return; /* TODO(pcm): return error */
@@ -148,11 +148,12 @@ void seal_object(int conn, plasma_request* req) {
   HASH_ADD(handle, sealed_objects, object_id, sizeof(plasma_id), entry);
   /* Inform processes that the object is ready now. */
   object_notify_entry* notify_entry;
-  HASH_FIND(handle, objects_notify, &req->object_id, sizeof(plasma_id), notify_entry);
+  HASH_FIND(handle, objects_notify, &req->object_id, sizeof(plasma_id),
+            notify_entry);
   if (!notify_entry) {
     return;
   }
-  plasma_reply reply = { PLASMA_OBJECT, size };
+  plasma_reply reply = {PLASMA_OBJECT, size};
   for (int i = 0; i < notify_entry->num_waiting; ++i) {
     send_fd(notify_entry->conn[i], fd, (char*) &reply, sizeof(plasma_reply));
     close(notify_entry->conn[i]);
@@ -190,7 +191,7 @@ void run_event_loop(int socket) {
       exit(-1);
     }
     for (int i = 0; i < event_loop_size(state.loop); ++i) {
-      struct pollfd *waiting = event_loop_get(state.loop, i);
+      struct pollfd* waiting = event_loop_get(state.loop, i);
       if (waiting->revents == 0)
         continue;
       if (waiting->fd == socket) {
@@ -230,7 +231,7 @@ void start_server(char* socket_name) {
     exit(-1);
   }
   int on = 1;
-  if (setsockopt(fd, SOL_SOCKET, SO_REUSEADDR, (char*)&on, sizeof(on)) < 0) {
+  if (setsockopt(fd, SOL_SOCKET, SO_REUSEADDR, (char*) &on, sizeof(on)) < 0) {
     LOG_ERR("setsockopt failed");
     close(fd);
     exit(-1);
@@ -244,15 +245,15 @@ void start_server(char* socket_name) {
   struct sockaddr_un addr;
   memset(&addr, 0, sizeof(addr));
   addr.sun_family = AF_UNIX;
-  strncpy(addr.sun_path, socket_name, sizeof(addr.sun_path)-1);
+  strncpy(addr.sun_path, socket_name, sizeof(addr.sun_path) - 1);
   unlink(socket_name);
-  bind(fd, (struct sockaddr*)&addr, sizeof(addr));
+  bind(fd, (struct sockaddr*) &addr, sizeof(addr));
   listen(fd, 5);
   run_event_loop(fd);
 }
 
 int main(int argc, char* argv[]) {
-  char *socket_name = NULL;
+  char* socket_name = NULL;
   int c;
   while ((c = getopt(argc, argv, "s:")) != -1) {
     switch (c) {