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.

    • Both the above have been done r22804 [Tully]

Meeting agenda

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.

  • {X} Major issues that need to be resolved

Wiki: pluginlib/Reviews/2009-08-20 API Review (last edited 2009-08-25 01:40:56 by TullyFoote)