fixed. It does look better this way.


On Thu, May 15, 2014 at 11:51 AM, Ioan A Sucan <notifications@github.com>wrote:

> In include/moveit/setup_assistant/tools/moveit_config_data.h:
>
> > + * Planning parameters which may be set in the config files
> > + */
> > + struct OmplPlanningParameter{
> > +   std::string name; // name of parameter
> > +   std::string value; // value parameter will receive (but as a string)
> > +   std::string comment; // comment briefly describing what this parameter does
> > + };
> > +
> > +/** \brief This class describes the OMPL planners by name, type, and parameter list, used to create the ompl_planning.yaml file */
> > + class OMPLPlannerDescription{
> > + public:
> > +/** \brief Constructor
> > + *  @param name: name of planner
> > + *  @parameter type: type of planner
> > + */
> > +   OMPLPlannerDescription(const std::string  & name, const std::string  & type){
>
> And we use & next to the variable name in MoveIt.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/ros-planning/moveit_setup_assistant/pull/85/files#r12700030>
> .
>