search

OctoMap / octomap_mapping

ROS stack for mapping with OctoMap, contains octomap_server package
http://www.ros.org/wiki/octomap_mapping
342 stars 280 forks source link

Change iterator in OctomapServer #16

Closed wxmerkt closed 9 years ago

wxmerkt commented 9 years ago

Reading through the source code, I have noticed that the end of the iterator iterating over the occupied_cells was specified by the length of the free_cells KeySet. I think this should be replaced by the occupied_cells length as completed in this commit/PR. Or is there perhaps another underlying reason for this choice?

ahornung commented 9 years ago

Thanks, that indeed looks like a copy / paste error.