This is the existing prototype.  I agree 
```
const visualization_msgs::InteractiveMarkerFeedback &
```
would be better, but changing it would break legacy code.

I plan to change this later by replacing these callback functions with virtual methods in the GenericInteraction class, so this will be improved later.