Looks like -1.0 is being used as a dummy NULL value for arm_velocity_. Is that right? If so it should be at least documented, ideally by using a constant (may need to be a #define for use in constructor initialization list) for the -1.0 value.