Skip to content

Conversation

sarahmonod
Copy link

and changed the double quotes to single for consistency.

and changed the double quotes to single for consistency.
@sarahmonod
Copy link
Author

This wiki page should also be updated: http://wiki.ros.org/ROS/Tutorials/WritingPublisherSubscriber%28python%29

@dirk-thomas
Copy link
Member

Thank you for paying attention to details!

dirk-thomas added a commit that referenced this pull request Feb 5, 2016
Fixed minor misleading comment
@dirk-thomas dirk-thomas merged commit d6d11f3 into ros:jade-devel Feb 5, 2016
@dirk-thomas
Copy link
Member

Also cherry-picked to the indigo-devel branch: 565a77f

@dirk-thomas
Copy link
Member

The mentioned wiki page seems to not contains the buggy docblock though?

@sarahmonod
Copy link
Author

Check out 1.2.1, The file contents look close to: then there are the same few mistakes.

@dirk-thomas
Copy link
Member

The second code block in 1.2.1 looks like this for me?

  ...
  10     # In ROS, nodes are uniquely named. If two nodes with the same
  11     # node are launched, the previous one is kicked off. The
  12     # anonymous=True flag means that rospy will choose a unique
  13     # name for our 'listener' node so that multiple listeners can
  14     # run simultaneously.
  15     rospy.init_node('listener', anonymous=True)
  ...

@sarahmonod
Copy link
Author

Weird. It was different for me yesterday, but now it seems fixed.

Although the quotes still seem inconsistent to me, I guess this is very minor and doesn't really require fixing :-P

@dirk-thomas
Copy link
Member

If you see anything which needs to be updated please feel free to edit the wiki page.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants