Added hacks to handle new and old formats of labeling coordinates so that previous storage files can still be loaded and compared so that the tests pass.
StaticOptimizationTarget could not find the corresponding spline for the motion of the coordinate because of the new naming scheme.  A hack was introduced to locate the spline.
Also made Coordinate::getSpeedName() return a const reference.

testControllers tolerance was tightened on fiber-lengths but loosened on activations. The standard and the results from OpenSim 3.2 but 3.2 passed, but just barely. The current solution and 3.2 are virtually identical but different enough so that the comparison with the standard was failing.
Merge branch 'master' into CoordinateStateVariableNames
RRA uses the old InverseDynamics (analysis) and that required the same "fix" as StaticOptimizationTarget to associate the motion to the correct coordinate.

Also added a check to see if rra.run() returns false and then to throw an exception. I did not make run() throw the exception because it looked like some effort was put in to have the method return false and not throw - probably the GUI depends on this instead of handling the Exception.

Simplified hack in CMC_TaskSet to use access the coordinate that is already available (instead of using model.getComponent() to find it)
Convert tabs to spaces.
Merge branch 'master' into CoordinateStateVariableNames
Moved the search for matching column index in a storage to Storage::getStateIndex(). Addresses @chisdembia's point that both the source (this) and the other storage should use the same index matching scheme. The previous approach on worked if the the outdated file was the source (this) which worked with our standard first convention. Now it does not matter.

Also added a comment for Coordinate::getSpeedName().
Merge pull request #271 from opensim-org/CoordinateStateVariableNames

Coordinate state variable names fixes #135