ROSIntegration icon indicating copy to clipboard operation
ROSIntegration copied to clipboard

std_msgs/Header time vs. stamp in docs

Open samkys opened this issue 4 years ago • 2 comments

Is there intent behind having the std_msgs/Header time name be time instead of stamp?

Where name is time at line 20: https://github.com/code-iai/ROSIntegration/blob/master/Source/ROSIntegration/Public/std_msgs/Header.h

And in docs where name is stamp: http://docs.ros.org/en/melodic/api/std_msgs/html/msg/Header.html

Thanks!

samkys avatar Nov 17 '20 22:11 samkys

Hi @samkys If there was an intent in that naming, i can't really remember what it was 😃

From my side it would be worth a try to rename it and see if other things break.

Sanic avatar Nov 30 '20 22:11 Sanic

Cool, next time I'm working on that I'll make a change and see what happens.

samkys avatar Dec 11 '20 19:12 samkys