[quote="Ingo_Lutkebohle, post:2, topic:6371, full:true"]
My 2 cents: Either use _ as the word separator throughout or CamelCase, but please not both.
[/quote]
Says the guy with handle `Ingo_Lutkebohle` :yum:
Jokes aside, from the google groups discussion I don't really understand why the decision to enforce stricter rules in the first place was reached, but I haven't really been following the developments as much.
In my opinion minimizing the migration effort by far outweights the consistency concerns. Consistency can be improved by having some style guide and linters to educate users and maybe even enforce the rules for new core messages. At the same time we should still allow exceptions, in particular also to accommodate non-public messages people might have. So for that reason I'm in favor to have the same rules as for ROS1, unless there is a very strong technical reason to change it (making constants easier to distinguish from fields is nice, but not a very strong reason but rather only a little bonus IMHO, especially since most IDE's have syntax highlighting for that sort of thing).
For core messages like `CameraInfo`, I personally would only change them if there is a strong reason to do so and if there is consensus among a larger number of actual users of these messages that the new version is actually an improvement. For some suggestion of one person of a more consistent / understandable naming, there are probably 5 people that disagree or have their own alternative suggestions.
---
[Visit Topic](
https://discourse.ros.org/t/relaxing-ros2-topic-service-field-name-restrictions/6371/3) or reply to this email to respond.
If you do not want to receive messages from ros-users please use the unsubscribe link below. If you use the one above, you will stop all of ros-users from receiving updates.
______________________________________________________________________________
ros-users mailing list
ros-users@lists.ros.org
http://lists.ros.org/mailman/listinfo/ros-users
Unsubscribe: <
http://lists.ros.org/mailman//options/ros-users>
(text/plain)