Re: [boost] [filesystem] Request for comments on proposed relative() function

Subject: Re: [boost] [filesystem] Request for comments on proposed relative() function
From: Beman Dawes (bdawes_at_[hidden])
Date: 2014-05-14 09:24:35

On Wed, May 14, 2014 at 3:02 AM, Rob Stewart <robertstewart_at_[hidden]>wrote:

> On May 9, 2014 4:23:00 PM EDT, Daniel Pfeifer <daniel_at_[hidden]>
> wrote:
> >2014-05-08 21:02 GMT+02:00 Bjorn Reese <breese_at_[hidden]>:
> >> On 05/08/2014 07:19 PM, Beman Dawes wrote:
> >>
> >>>   Creates a path from the trailing elements of p that are relative
> >>> to base.
> >>
> >>
> >> It may be more user-friendly to add that base must be a prefix of p.
> >
> >It might be even more user-friendly to allow this use case:
> >
> >BOOST_TEST(fs::relative("/abc", "/abc/def") == path("../def"));
>
> The first element in the range p, that's not in base, is the empty set, so
> p cannot be made relative to base in your example. If you reverse the
> arguments, then the result would be "def".
>

+1

--Beman