Why not "return 0" instead of "return true" in CheckPathAllConstraints when _checkpathvelocityconstraintsfn is empty

[ompugao]

if return 1 on default, true is casted into (int)1, which means that it always claims that the specified path segment has some problem on CFO_CheckEnvCollisions? Am I wrong?

https://github.com/rdiankov/openrave/blob/production/include/openrave/planner.h#L271

// it is quite rare to keep _checkpathvelocityconstraintsfn empty though.