Hi Christian, On 10 January 2013 09:31, Christian Schulte <christian.schu...@onera.fr> wrote: > I just tried another solution, which works and is maybe, let's say, more > correct from a coding point of view. Indeed, we want to floor a division of > two floats, so we should logically use floorf instead of floor, because in > math.h floor is only double. That's also the reason we don't have the error > with cmath because we have a float overload of the double floor(double). > > What do you think about this solution ? (I will still try to see which are > the consequences of replacing math.h by cmath, but it will be a little > longer :-) )
Using floorf would be appropriate if the function returns a float and the parameters internally will be float. If we do properly embrace cmath then we can probably take the opportunity to clean up the more than just this particular function in include/osg/Math. Robert. _______________________________________________ osg-users mailing list osg-users@lists.openscenegraph.org http://lists.openscenegraph.org/listinfo.cgi/osg-users-openscenegraph.org
