move_base_msgs/Reviews/2009-10-06_Code_Review

Package Developer:

Examiner:

Stu

• I tried to find a link to look at MoveBase.action and failed. The only way for me to see it was to roscd in my source tree.

  • Added this to the wiki page
• Mention somewhere how the pose maps into the 2D plane.
  • Added a description of this

Present at code review:

Notes from examiner

Examiner: Enter your notes on each section below. See ExaminerChecklist for more detailed instructions.

• Divide your concerns up into bullet lists that can be run through in order in the meeting, and concretely decided about. Include enough details / relevant information for people at meeting to quickly decide on the issue (e.g. file and line number, description of race condition, or list unclear names)

• Use error symbol to call out things that are clear errors

• Use warning symbol to call out things that you're worried about

• Use check mark to mark areas that look good.

• Use the below bullets as a basic guide, but feel free to add new issues or remove issues that are not relevant (e.g. command-line options for a library)
• Feel free to delete this instruction section when you're done

Documentation

• API
• ROS API
• command-line
• tutorials / examples

Naming

• headers
• libraries
• exported symbols
• ROS API

Source Code

• header files
• source files
• data / configuration / xml files

Manifest / Build System

• build
• dependencies
• manifest data

Testing

• all tests pass
• unit tests
• rostests

Conclusions

This section will be filled out during the group code review meeting, and will include

• Any action items that need to be taken before clearing

• Change in official package status

---