API review

Proposer: your name here

Present at review:

• List reviewers

Question / concerns / comments

Enter your thoughts on the API and any questions / concerns you have here. Please sign your name. Anything you want to address in the API review should be marked down here before the start of the meeting.

• Josh

  • Why does ClassLoader::getBaseClassType() take a name? Isn't ClassLoader initialized with the base class name?

    • [Tully] This will return the base class for any class declared in xml
    • [Eitan] Actually, it shouldn't take a name as it will be the same for all derived classes. This can/will change easily.

  • What does loading a class mean? Is that explicitly loading the plugin for the specified class? If so I would be tempted to name loadClass something like loadPluginForClass or loadLibraryForClass.

    • [Tully] The latter seems the best. ticketed https://prdev.willowgarage.com/trac/personalrobots/ticket/2531

  • Both the above have been done r22804 [Tully]

Meeting agenda

To be filled out by proposer based on comments gathered during API review period

Conclusion

Package status change mark change manifest)

• Action items that need to be taken.

• Major issues that need to be resolved

---