Also take const& for VectorD range constructor.

MSVC uses this constructor when copying IMP::cgal::internal::VectorWithIndex
objects, so take a reference to avoid it trying to do a copy within
the copy constructor (and so running out of stack).