From: Nathan H. <nj...@nj...> - 2013-09-29 00:37:51
|
On Sat, Sep 28, 2013 at 10:28:08PM +0200, Johan Engelen wrote: > Hi all, > Point::isZero is using float point comparison instead of are_near. > Any objections against changing this to are_near? > The new code: > bool isZero() const { > return ( are_near(_pt[X],0) && are_near(_pt[Y],0) ); > } I think isZero should test for zero, but I'm happy for you to change code to use pt.areNear(Point()) or add nearZero() or something njh |