Why take a ConstPtr& here for a ROS message and just a & for the callback above? I'd rather we take the ref to the msg in both cases.