To run this in the future, install astyle (sudo apt-get install astyle),
then enter the following from the top-level project directory:

astyle --options=astyle.options --recursive "./Source/*.cpp" "./Source/*.h"

This will convert tabs to 4 spaces and ensure that brackets are on
their own lines.

Use the following command to update the copyright year in all files,
starting in the Source/ directory:

find . -type f -print0 | xargs -0
   sed -i 's/Copyright (C) 2012/Copyright (C) 2013/g'

Works like a charm!

Added some comments and removed deprecated code.

MainWindow now points to the ProcessorGraph and AudioComponent
with ScopedPointers. deleteAndZero must no longer be called
in the destructor.

Generic changes:

- Genuinely unused variables (-Wunused-variable): delete
  them. Sometimes add a comment if the variable is for an extension of
  the code that isn't done yet.

- Variables that became unused due to commented-out
  code (-Wunused-variable): comment out their definitions. Dirty, but
  that's because commenting out code instead of deleting it is
  dirty.

  Many of these are due to commented-out printlines that should have
  been log messages that only get emitted at high verbosity level.

Specific changes:

- SpikeObject.cpp (-Wunused-value): move initialization of loop
  variable into first for loop clause to silence the warning.

Signed-off-by: Marti Bolivar <mbolivar@leaflabs.com>

This appears to be the source of the inconsistencies, at least on OS X.

Now, when the user clicks on a tab in the DataViewport, the corresponding Editor becomes
active. Likewise, when the user selects an Editor with a Visualizer, the corresponding
tab becomes active in the DataViewport.