Created on 2014-05-30.00:00:00 last changed 89 months ago
Proposed resolution:
This wording is relative to N4606, and assumes that the PR for 2732 is applied.
path& operator/=(const path& p);
-?- Requires:
!p.has_root_name()
.-2- Effects: Appends
path::preferred_separator
to pathname unless:
- — an added directory-separator would be redundant, or
- — an added directory-separator would change a relative path to an absolute path [Note: An empty path is relative. — end note], or
- —
p.empty()
is true, or- —
*p.native().cbegin()
is a directory-separator.Then appends
p.native()
to pathname.-3- Returns:
*this
.
path operator/(const path& lhs, const path& rhs);
-13-
ReturnsEffects: Equivalent toreturn path(lhs) /= rhs;
.
[ 2016-08-03 Chicago ]
After discussion on 2732, it was determined that the PR for that issue should be applied to this issue before it is accepted. That PR changes all the path appends to go through operator/=, so only one requires element remains necessary.
Fri AM: Moved to Tentatively Ready
[ 2016-07-03, Daniel comments ]
The same wording area is touched by LWG 2732.
Previous resolution [SUPERSEDED]:
This wording is relative to N4594.
Change [fs.path.append] path appends as indicated:
path& operator/=(const path& p);
-?- Requires:
!p.has_root_name()
.-2- Effects: Appends
path::preferred_separator
to pathname unless:
an added directory-separator would be redundant, or
an added directory-separator would change a relative path to an absolute path [Note: An empty path is relative. — end note], or
p.empty()
is true, or
*p.native().cbegin()
is a directory-separator.Then appends
p.native()
to pathname.-3- Returns:
*this
.template <class Source>
path& operator/=(const Source& source);
template <class Source>
path& append(const Source& source);
template <class InputIterator>
path& append(InputIterator first, InputIterator last);-?- Requires:
-4- Effects: Appends!source.has_root_name()
or!*first.has_root_name()
, respectively.path::preferred_separator
to pathname, converting format and encoding if required ([fs.path.cvt]), unless:
an added directory-separator would be redundant, or
an added directory-separator would change a relative path to an absolute path, or
source.empty()
is true, or
*source.native().cbegin()
is a directory-separator.Then appends the effective range of
-5- Returns:source
([fs.path.req]) or the range[first, last)
topathname
, converting format and encoding if required ([fs.path.cvt]).*this
.Change [fs.path.nonmember] path non-member functions as indicated:
path operator/(const path& lhs, const path& rhs);
-?- Requires:
!rhs.has_root_name()
.-13- Returns:
path(lhs) /= rhs
.
[ Apr 2016 Issue updated to address the C++ Working Paper. Previously addressed File System TS ]
[ 2016-06-13, Beman provides wording and rationale ]
Rationale: The purpose of the append operations is to provide a simple concatenation facility for users
wishing to extend a path by appending one or more additional elements, and to do so without worrying about the
details of when a separator is needed. In that context it makes no sense to provide an argument that has a
root-name. The simplest solution is simply to require !p.has_root_name()
.
The other suggested solutions IMO twist the functions into something harder to reason about
yet any advantages for users are purely speculative. The concatenation functions can
be used instead for corner cases.
[ 2016-02, Jacksonville ]
Beman to provide wording.
[ 17 Jun 2014 Rapperswil LWG will investigate issue at a subsequent meeting. ]
In a recent discussion on the Boost developers mailing list, the semantics of operator /
and other append operations were questioned:
p1 / p2
is required to concatenate the
lexical representation of p1
and p2
, inserting a
preferred separator as needed.
This means that, for example, "c:\x" / "d:\y"
gives
"c:\x\d:\y"
, and that "c:\x" / "\\server\share"
gives "c:\x\\server\share"
. This is rarely, if ever, useful.
An alternative interpretation of p1 / p2
could be that it yields a
path that is the approximation of what p2
would mean if interpreted
in an environment in which p1
is the starting directory.
Under this interpretation, "c:\x" / "d:\y"
gives "d:\y"
,
which is more likely to match what was intended.
I am not saying that this second interpretation is the right one, but I do say
that we have reasons to suspect that the first one (lexical concatenation using
a separator) may not be entirely correct.
This leads me to think that the behavior of p1 / p2
, when p2
has a root, needs to be left implementation-defined, so that implementations are
not required to do the wrong thing, as above.
This change will not affect the ordinary use case in which p2
is a
relative, root-less, path.
History | |||
---|---|---|---|
Date | User | Action | Args |
2017-07-30 20:15:43 | admin | set | status: wp -> c++17 |
2016-11-14 03:59:28 | admin | set | status: pending -> wp |
2016-11-14 03:55:22 | admin | set | status: ready -> pending |
2016-08-06 20:44:18 | admin | set | status: new -> ready |
2016-08-03 20:47:51 | admin | set | messages: + msg8374 |
2016-07-03 12:47:29 | admin | set | messages: + msg8224 |
2016-06-14 10:32:15 | admin | set | messages: + msg8174 |
2016-06-14 10:32:15 | admin | set | messages: + msg8173 |
2016-04-10 22:23:37 | admin | set | messages: + msg8038 |
2016-03-07 04:46:57 | admin | set | messages: + msg8007 |
2016-01-28 01:00:35 | admin | set | messages: + msg7917 |
2014-05-30 00:00:00 | admin | create |