boost::geometry::model::point single argument constructor should be explicit

[Jared Russell <jaredrussell341@…>]

In boost/geometry/geometries/point.hpp, the function:

point::point(CoordinateType const & v0, CoordinateType const & v1 = 0, CoordinateType const & v2 = 0)

has default arguments. Because there is a single argument case, we can use this function for implicit conversion. This is probably not the desired behavior. As a result, any numerical type can be implicitly converted to a point whether you meant to or not. Adding the "explicit" keyword resolves the issue.

Example:

namespace bgm = boost::geometry::model;
namespace bgc = boost::geometry::cs;

bgm::point<double, 2, bgc::cartesian> ll(0.0, 0.0); // lower left
bgm::point<double, 2, bgc::cartesian> ur(100.0, 100.0); // upper right

bgm::box<bgm::point<double, 2, bgc::cartesian> > myBox(11,ur); // Oops!  literal 11, not "ll".  But it will compile!

[Barend Gehrels]

Agreed, thanks. Done. Will be released in 1.55