Yes, that's a bit cryptic. Also, not sure this is the right way to use
level, as it's not very well documented. I just needed a way to run the
code to check for 0 values to load robot defaults on startup. Another
possibility is just to add some boolean attribute to make this more clear
an avoid some misunderstanding about the level argument.


On Mon, Jan 27, 2014 at 8:17 PM, Reed Hedges <notifications@github.com>wrote:

> In RosAria.cpp:
>
> >    robot->lock();
> > -  ros::NodeHandle n_("~");
> > -  if (n_.hasParam("TicksMM"))
> > -  {
> > -    n_.getParam( "TicksMM", TicksMM);
> > -    ROS_INFO("Setting TicksMM from ROS Parameter: %d", TicksMM);
> > -    robot->comInt(93, TicksMM);
> > -  }
> > -  else
> > -  {
> > +  if (level == ~0)
>
> Using ~0 confused me at first since I thought 'level' was some kind of
> cardinal integer. Looking at dynamic_reconfigure code 'level' is actually a
> bitmask, so this makes sense. I may add a comment here explaining it more.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/amor-ros-pkg/rosaria/pull/23/files#r9203933>
> .
>