Hi Barend.

On 16/05/2014 09:06 μμ, Barend Gehrels wrote:
>
> In include/boost/geometry/algorithms/distance.hpp:
>
> > +
> > +
> > +template <typename Geometry1, typename Geometry2>
> > +struct distance
> > +{
> > +    template <typename Strategy>
> > +    static inline typename result_of::distance<Geometry1, Geometry2, Strategy>::type
> > +    apply(
> > +        const Geometry1& geometry1,
> > +        const Geometry2& geometry2,
> > +        Strategy const& strategy)
> > +    {
> > +        return resolve_strategy::distance::apply(geometry1, geometry2, strategy);
> > +    }
> > +};
> > +
>
> So yes, this review becomes a bit messy, I see that this is already 
> outdated so processed?
>
> Anyway, please keep template at the method-level and /not/ at the 
> function level. Sorry if you already changed this. We prefer to have 
> it at the method level (for Menelaos: for your information).
>
FYI, Samuel has kept it as it was.

@Barend: I will keep that in mind. In fact having templates at the 
method level made my life easy/easier when I started working with 
distances. So no worries, I back this choice, and will do henceforth.

-m.

> As also remarked by Bruno and now agreed by Menelaos.
>
> —
> Reply to this email directly or view it on GitHub 
> <https://github.com/boostorg/geometry/pull/26/files#r12751683>.
>