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.
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]
To be filled out by proposer based on comments gathered during API review period
Package status change mark change manifest)
Action items that need to be taken.
Major issues that need to be resolved