Doc. no. | N4212 |
Date: | 2014-10-11 |
Project: | Programming Language C++ |
Reply to: | Beman Dawes <bdawes at acm dot org> |
Revised 2014-10-11 at 18:10:45 UTC
Reference ISO/IEC TS 18822
Also see:
This document contains only SG3 issues which have been closed
by the File System Study Group as duplicates or not defects. That is,
issues which have a status of Dup or
NAD. See the
Section: X 4.14 [fs.def.parent] Status: NAD Submitter: CH-3 Opened: 2014-01-20 Last modified: 2014-07-05
View all issues with NAD status.
Discussion:
The concept of a parent directory for dot or dotdot exists, but the definition doesn't apply.
Suggested action:
Remove the paragraph. This concept does not apply to dot and dot-dot. Add a definition for dot and dot-dot.
[2014-02-07, Beman Dawes comments]
Suggest it is editorial and should be passed to the project editor.
[ 2014-02-11 Issaquah: Beman to provide wording for review next meeting. Also see related issue 5. ]
[22 May 2014 Beman Dawes comments:]
I've now reviewed this issue carefully and believe it is NAD. "parent" is used in four places in the WP, and so deserves a definition. The current definition is copied word-for-word and in its entirety from the POSIX definition. I believe strongly that the File System TS needs to stay in alignment with POSIX on this matter, and that the best way to do that is simply to use the POSIX wording.
[17 Jun 2014 Rapperswil LWG closes as NAD. No concensus for change.]
Proposed resolution:
Section: X 6 [fs.filesystem.synopsis] Status: NAD Submitter: FI-4 Opened: 2014-01-20 Last modified: 2014-06-27
View all other issues in 6 [fs.filesystem.synopsis].
View all issues with NAD status.
Discussion:
It is unclear why copy, copy_file and copy_symlink have different return types, and why the attribute-version of create_directory has a different return type than the create_directory that takes no attributes. The status/error reporting in these functions seems inconsistent.
Resolution: Make the status/error reporting consistent or add a note explaining why it's different.[ 2014-02-11 Issaquah: NAD. LWG/SG-3 reviewed each function and return type, and found that since they have different functionality different return types are warranted. create_directory has an inconsistent return type between the synopsis and the description. This has subsequently been corrected editorially. ]
Proposed resolution:
Section: X 6 & 15 Status: NAD Future Submitter: GB-1 Opened: 2014-01-20 Last modified: 2014-07-05
View all issues with NAD Future status.
Discussion:
There is no relative() operation, to complement both absolute() and canonical()
The TS introduces relative paths.
They are defined in section 4.18 relative path [fs.def.relative-path]
A decomposition method relative_path() is described in section 8.4.9 path decomposition [path.decompose]
Two query methods to determine if a path either has_relative_path() or is_relative() described in 8.4.10 path query [path.query]
However there is no way to create a relative path as a path relative to another. Methods are provided to create absolute and canonical paths.
In section 15.1 Absolute [fs.op.absolute]:
path absolute(const path& p, const path& base=current_path());
and in section 15.2 Canonical [fs.op.canonical]
path canonical(const path& p, const path& base = current_path());
path canonical(const path& p, error_code& ec);
path canonical(const path& p, const path& base, error_code& ec);
By providing a operations to achieve absolute and canonical paths there is no impediment to providing a similar operation relative() that attempts to return a new path relative to some base path.
For example:
path relative(const path& p, const path& to = current_path());
path relative(const path& p, error_code& ec);
path relative(const path& p, const path& to, error_code& ec);
This would return a path, if possible, that is relative to to. The implementation can make use of absolute() and canonical() to determine the relative path, if it exists.
The File System TS is based on the boost::filesystem library and it too suffers from this anomaly. There are open tickets for this in Boost Trac:
and it is the subject of several posts on StackOverflow for example:
http://stackoverflow.com/questions/10167382/boostfilesystem-get-relative-path
http://stackoverflow.com/questions/5772992/get-relative-path-from-two-absolute-paths
Other languages typically provide a similar function. For example python provides:
os.path.relpath(path[, start])
Return a relative filepath to path either from the current directory or from an optional start directory. This is a path computation: the filesystem is not accessed to confirm the existence or nature of path or start. start defaults to os.curdir.
[2014-02-07, Beman Dawes comments]
A relative()
function is useful and much requested.
I've seen such a function provided by users and have written it myself in app code.
It is one of those things I've been meaning to do for years, and have just never gotten around to.
That said, my mild preference is to treat this as "NAD, Future" for File System TS1, but treat it as a priority for TS2.
[ 2014-02-11 Issaquah ]
The LWG/SG-3 voted strongly in favor of adding this functionality, and doing so in this TS. That implies quite a bit of work before the next meeting to validate that the proposed interface works as desired for various platforms. There was general agreement not to hold FS STS1 if this functionality isn't ready when the rest of the TS is ready.
[2014-05-19 Beman Dawes supplied wording. The design benefited from discussions with Jamie Allsop, who was the source of the original NB comment. Thanks to Bjorn Reese for corrections and suggestions. Although there was also discussion and experimentation with additional relative functions that took into account symlinks and normalization, these are not proposed here since even the proponents of such functions were unsure of appropriate semantics.]
[17 Jun 2014 Rapperswil LWG closes as NAD, Future. Although there is strong concensus for eventually providing both lexical and existence based flavors of relative() functionality, discussion of the many possible design choices led to the conclusion that more research and actual user experience is necessary before moving forward. Interested parties should submit papers.]
Original proposed resolution:
Modify header <filesystem> synopsis, 6 [fs.filesystem.synopsis], by adding the operational functions after canonical:
path relative(const path& p, const path& to = current_path()); path relative(const path& p, error_code& ec); path relative(const path& p, const path& to, error_code& ec);
Insert the section:
15.3 Relative [fs.op.relative]
path relative(const path& p, const path& to = current_path()); path relative(const path& p, error_code& ec); path relative(const path& p, const path& to, error_code& ec);Overview: Return a relative path of p to the current directory or from an optional to path.
Returns: A relative path such that canonical(to)/relative(p,to) == canonical(p), otherwise path(). If canonical(to) == canonical(p) the path path(".") is returned. For the overload without a to argument, to is current_path(). Signatures with argument ec return path() if an error occurs.
Throws: As specified in Error reporting.
Remarks: !exists(p) or !exists(to) or !is_directory(to) is an error.
and bump all following sections up by 0.1. Update the contents and any cross-references accordingly.
Question: Should Returns be specified in terms of equivalence? For example: equivalent( canonical(to)/relative(p,to), canonical(p) )
Question: Should canonical(to) == canonical(p) return path(".") or path()? Why?
Question: Should to be spelled start?
Proposed resolution:
To 6 Header <experimental/filesystem> synopsis [fs.filesystem.synopsis], add:
path lexically_relative(const path& p, const path& base);
At the end of 8.6 path non-member functions [path.non-member], add
8.6.3
path
lexically_relative function [path.lexically.relative]path lexically_relative(const path& p, const path& base);Creates a path from the trailing elements of
p
that are lexically relative tobase
, which must be a prefix ofp
.Effects: If the number of elements in [
p.begin(), p.end()
) is less than or equal to the number of elements in [base.begin(), base.end()
), or if any element in [base.begin(), base.end()
) is not equal to the corresponding element in [p.begin(), p.end()
), throw an exception of typefilesystem_error
.Remarks: Equality or inequality are determined by
path::operator==
orpath::operator!=
respectively.Returns: An object of class
path
containing the first element ofp
that does not have a corresponding element inbase,
followed by the subsequent elements ofp
appended as if bypath::operator/=
.Throws:
filesystem_error
.[Note: Behavior is determined by the lexical value of the elements of
p
andbase
- the external file system is not accessed. The case where an element ofbase
is not equal to corresponding element ofp
is treated as an error to avoid returning an incorrect result in the event of symlinks. --end note]A possible implementation would be:
auto mm = std::mismatch( p.begin(), p.end(), base.begin(), base.end()); if (mm.first == p.end() || mm.second != base.end()) { throw filesystem_error( "p does not begin with base, so can not be made relative to base", p, base, error_code(errc::invalid_argument, generic_category())); } path tmp(*mm.first++); for (; mm.first != p.end(); ++mm.first) tmp /= *mm.first; return tmp;
Section: X [fs.filesystem.synopsis], [fs.op.file_size] Status: NAD Future Submitter: CH-8 Opened: 2014-01-20 Last modified: 2014-06-27
View all issues with NAD Future status.
Discussion:
uintmax_t is specified to hold at least 64 bit. This is not enough for sizes beyond 4 [sic] exabytes.
Specify whether an implementation must provide a uintmax_t that can hold the maximum possible space and file size values.
[2014-02-06: Jeffery Yasskin points out 64-bits unsigned actually has a maximum value of "16 exabytes, not 4"]
[2014-02-07: Beman Dawes suggests: This should be NAD. Such ultra-large files are the province of enterprise-wide filesystems such as requested by IBM and others for a follow-on SG3 TS. That would be the best vehicle to address this concern IMO.]
[ 2014-02-11 Issaquah: NAD Future. ]
Proposed resolution:
Section: X 7 [fs.err.report] etc. Status: NAD Submitter: CH 9 Opened: 2014-01-20 Last modified: 2014-06-27
View all issues with NAD status.
Discussion:
The specification of the actual error conditions for the functions that specify Throws: As specified in Error reporting. is missing.
Add those specifications.
[2014-02-07, Beman Dawes comments]
The actual error codes, and thus the error conditions, are determined by the operating system, and thus operating system dependent.
[2014-02-11 Issaquah]
There is no consensus for a change. LWG/SG3 requested a separate issue be opened to clarify 7 [fs.err.report]. See issue 55.Proposed resolution:
Section: X 8.4.5 [path.modifiers] Status: NAD Submitter: CH-12 Opened: 2014-01-20 Last modified: 2014-06-27
View all issues with NAD status.
Discussion:
As we have move semantics, member swap functions shouldn't be necessary any more.
[2014-02-12 LWG/SG-3 Issaquah ]
No consensus for change. STL pointed out that swap should be noexcept and will submit a separate issue.Proposed resolution:
Remove swap().
Section: X 15 Status: NAD Submitter: CH-14 Opened: 2014-01-20 Last modified: 2014-05-24
View all issues with NAD status.
Discussion:
Since create_symlink(), create_hardlink(), and create_directory() exist, there's no reason not to have a create_regular_file() function.
Consider adding a function create_regular_file() with the behaviour of the POSIX touch command.
[Beman comments]
create_regular_file() and touch() should be different functions since their behavior would differ if the file already exists; touch() updates the last write date.
I have often wanted create_regular_file() but never got around to adding it.
While touch is quite useful from the command line, I'm less sure of its usefulness as a library function. It is trivial for a user to implement.
Whether or not it is appropriate to add operational functions this late in the PDTS process is questionable.
[2014-02-13 LWG/SG-3 Issaquah: No consensus for change at this time.]
Proposed resolution:
Section: X 15.13 [fs.op.equivalent] Status: NAD Submitter: CH-16 Opened: 2014-01-20 Last modified: 2014-05-24
View all issues with NAD status.
Discussion:
Equivalence is a volatile property.
Consider adding a note that equivalence cannot be determined race-free.
[2014-02-13 LWG/SG-3 Issaquah: No consensus for change. Section 2.1 description of races is sufficient.]
Proposed resolution:
Section: X 15.25 [fs.op.last_write_time] Status: NAD Submitter: GB-15 Opened: 2014-01-20 Last modified: 2014-05-24
View all other issues in 15.25 [fs.op.last_write_time].
View all issues with NAD status.
Discussion:
The constraint on last_write_time is too weak: It is noted that the postcondition of last_write_time(p) == new_time is not specified since it might not hold for file systems with coarse time granularity.
However, might it be possible to have a postcondition that last_write_time(p) <= new_time ?Add postcondition: last_write_time(p) <= new_time
[2014-02-09, Beman Dawes comments:]
That assumes the file system rounds down. We don't know which direction a file system rounds. Nice try, but this one looks NAD to me.[2014-02-13 LWG/SG-3 Issaquah: No consensus for change.]
Proposed resolution:
Section: X 15.28 [fs.op.remove] Status: NAD Submitter: CH-17 Opened: 2014-01-20 Last modified: 2014-05-24
View all issues with NAD status.
Discussion:
The specification can be read to require the existence test. As this introduces a race, the existence test must not happen.
Change to: "Effects: p is removed as if by POSIX remove()."
[2014-02-13 LWG/SG-3 Issaquah: Insufficient consensus for change. Vote for NAD: 9 0 0 2 1.]
Proposed resolution:
This wording is relative to SG3 working draft.
Change X 15.28 [fs.op.remove] as indicated:
Effects:
If exists(symlink_status(p,ec)), itp is removed as if by POSIX remove().
Section: X 15.30 [fs.op.rename] Status: NAD Submitter: CH-18 Opened: 2014-01-20 Last modified: 2014-06-27
View all issues with NAD status.
Discussion:
POSIX guarantees some kind of atomicity for rename().
Clarify that POSIX' rename() guarantee "If the rename() function fails for any reason other than [EIO], any file named by new shall be unaffected." holds for C++ as well.
[2014-02-10 Beman Dawes]
Section 2.1, POSIX conformance, [fs.conform.9945] specifies the POSIX conformance requirements for TS implementations in carefully crafted and specific detail. Repeating a portion of the POSIX standard's specification for a particular TS function would do great harm as it would bring into question all of the portions of the POSIX specification for the function that were not repeated. Furthermore, all the caveats and other details of the 2.1 specification would have to be analyzed and possibly appended; it ties the hands of implementors if they are not given latitude to deviate as needed when working with non-POSIX operating systems. I strongly recommend NAD for this issue.[2014-02-13 LWG/SG-3 Issaquah: No consensus for change.]
Proposed resolution:
Section: X 12.3 [directory_entry.obs], X 15.12 [fs.op.exists], X 15.16 [fs.op.is_block_file], X 15.17 [fs.op.is_char_file], X 15.18 [fs.op.is_directory], X 15.19 [fs.op.is_empty], X 15.20 [fs.op.is_fifo], X 15.21 [fs.op.is_other], X 15.22 [fs.op.is_regular_file], X 15.23 [fs.op.is_socket], X 15.24 [fs.op.is_symlink], X 15.33 [fs.op.status], X 15.35 [fs.op.symlink_status], X 15.38 [fs.op.unique_path] Status: NAD Submitter: P.J. Plauger Opened: 2014-01-30 Last modified: 2014-07-05
View all other issues in 12.3 [directory_entry.obs].
View all issues with NAD status.
Discussion:
exists(const path&) should be noexcept (drop error_code version).
is_*(const path&) should be noexcept (drop error_code version).
status(const path&) should be noexcept (drop error_code version).
symlink_status(const path&) should be noexcept (drop error_code version).
file_status::status() should be noexcept (drop error_code version).
file_status::symlink_status() should be noexcept (drop error_code version).
unique_path(const path&) should be noexcept (drop error_code version).
[2014-02-08: Daniel comments]
unique_path(const path&) cannot be declared as noexcept, because it returns an object during whose construction a memory allocation request may fail, see the rationale provided in 37.
exists(const path&) and the is_*(const path&) functions cannot be noexcept, because they are specified in terms of status(const path&), which again may throw an exception, which is explicitly described in the Effects (filesystem_error), because the non-throwing function (status(const path&, error_code&)) may fail to satisfy the request. symlink_status(const path&) may throw an exception for similar reasons that status(const path&) could fail. The reference to file_status::status()/symlink_status() looks like a typo to me (there are no such functions in file_status), presumably directory_entry::status()/symlink_status() was meant. In this case I see no reason how these could be marked as noexcept, because these functions all may fail and may throw an exception. Based on this interpretation of the issue discussion I recommend to resolve this issue as NAD.[Beman Dawes 2014-02-27]
Issues 37, 38, 41, and 49 are concerned with signatures which should or should not be noexcept. I will provide unified proposed wording for these issues, possibly in a separate paper.[17 Jun 2014 Rapperswil LWG closes as NAD. Working paper correct as written.]
Proposed resolution:
Section: X 6 [fs.filesystem.synopsis] Status: NAD Editorial Submitter: P.J. Plauger Opened: 2014-01-30 Last modified: 2014-06-27
View all issues with NAD Editorial status.
Discussion:
permissions function is missing from the summary list.
Proposed resolution:
[2014-02-07: Beman Dawes comments: Fixed as Editorial.]
Section: X 8 [class.path] Status: NAD Submitter: P.J. Plauger Opened: 2014-01-30 Last modified: 2014-07-05
View all other issues in 8 [class.path].
View all issues with NAD status.
Discussion:
class path should have defaulted constructors/destructor/assignments.
[2014-02-26 Beman Dawes comments]
Suggest NAD. Earlier versions did have defaulted constructors/destructor/assignments, but they were removed at the request of Alberto Ganesh Barbati (c++std-filesys January 8, 2013):Speaking of =default, we have to be careful not over-constrain the specification. I mean, if we just specify the intended meaning of those function with proper wording, the implementation is allowed to use =default in case it provides an equivalent behaviour, but if we put =default in the specification, the implementation is required to use it. However, we have to remember that "for exposition only" data members may not be an exhaustive list and that 17.5.2.3/2 allows implementations to provide an equivalent behaviour using different members for which "default" construction/copy/assignment may not be appropriate. =default is what we want for tuple, atomics, etc. It might be appropriate for simple types like file_status, but, for more complex types like path itself, I'd remove it and add proper wording.
[17 Jun 2014 Rapperswil LWG closes as NAD. Ganesh's analysis is correct. WP correct as written.]
Proposed resolution:
Section: X 8 [class.path] Status: Dup Submitter: P.J. Plauger Opened: 2014-01-30 Last modified: 2014-06-27
View all other issues in 8 [class.path].
View all issues with Dup status.
Discussion:
path::compare(const string&) should be path::compare(const string_type&).
[2014-02-08 Daniel comments]
This issue is a duplicate of 50. The suggested wording of that issue would resolve this issue here as well.
[2014-02-13 LWG/SG-3 Issaquah: Agrees with Daniel.]
Proposed resolution:
Section: X 8.4.7 [path.generic.obs] Status: NAD Submitter: P.J. Plauger Opened: 2014-01-30 Last modified: 2014-05-24
View all issues with NAD status.
Discussion:
Do we really need generic*?
[2014-02-08 Daniel comments]
These functions should exist for more than one reason.
First, the generic pathname format is a well-defined concept for the path specification and defining just a single way into a path but not out of it looks like an incomplete design. More importantly, the existence of these functions have demonstrated to be quite useful in practice, because the generic pathname format concept is popular for many tools such as maven or the some configuration files for the Eclipse IDE do understand this syntax uniformly on all platforms and it allows to generate or modify such files using C++ code based on filesystem::path. The practical problem of the non-portable root-names doesn't matter in such contexts, because the serialized path names are in general relative names or depend on initial parts that are determined by properties or environment variables. In other words: I'm recommending NAD for this issue.[2014-02-13 LWG/SG-3 Issaquah: Withdrawn by submitter.]
Proposed resolution:
Section: X 13 [class.directory_iterator], X 14 [class.rec.dir.itr] Status: Dup Submitter: Stephan T. Lavavej Opened: 2014-02-03 Last modified: 2014-06-27
View all other issues in 13 [class.directory_iterator].
View all issues with Dup status.
Discussion:
directory_iterator and recursive_directory_iterator are constant iterators, but their pointer/reference typedefs are wrong (std::iterator defaults to providing modifiable ones).
[2014-02-08 Daniel comments]
I noticed the same problem when trying to resolve 52. The currently suggested wording for that issue should fix the here mentioned problem as well.
I recommend to solve this issue as "Resolved by the proposed wording for 52".[2014-02-13 LWG/SG-3 Issaquah: Daniel's resolution accepted.]
Proposed resolution:
Section: X all Status: NAD Future Submitter: Google Opened: 2014-01-20 Last modified: 2014-07-05
View all issues with NAD Future status.
Discussion:
We have two primary concerns with the interface as specified:
(a) its interface repeats the mistake of V7 Unix in 1979 by exposing access checking (and similarly file creation) independently from opening and mutating the file, and
(b) it provides no realistic means of testing a software library which uses the standard interface for accessing the filesystem under fault scenarios.
Due to the extent of (a), TOCTTOU [1] security vulnerabilities are guaranteed, if not during access checking[2], during other common operations such as temporary file creation[3].
Due to (b) it is impossible to portably test libraries using the proposed interface against critical correctness and security edge cases.
[1]: TOCTTOU: Time-of-check-to-time-of-use. Operating system integrity in OS/VS2
[2]: Fixing Races for Fun and Profit: How to use access(2)
[3]: Checking for Race Conditions in File Accesses
[Beman Dawes: 10 Feb 2014: Suggested response: NAD, Future]We share your concerns and look forward to receiving specific proposals to address them. Whether they will addressed by a revision of TS 18822 or a new TS will be decided as proposals progress through the committee process. See How To Submit a Proposal.
[17 Jun 2014 Rapperswil LWG agrees NAD, Future with rationale as stated above.]
Proposed resolution:
Section: X 10 [fs.enum] Status: NAD Editorial Submitter: Daniel Krügler Opened: 2014-03-01 Last modified: 2014-07-05
View all issues with NAD Editorial status.
Discussion:
copy_options
is declared as enum class
type that is a bismask type, but the
specification repeatedly uses expressions that are invalid for scoped
enums, such as:
!(options)
because there is no contextual conversion to bool, not even the || operator in:
((options & copy_options::recursive) || !(options))
Affected are basically all formulations in the form:
"if options & copy_options::create_symlinks
[..]"
because all rely on contextual conversion to bool. The only other specifically
mention scoped enumeration in the standard that is also a bit mask type is the
launch
enum and
the wording there always uses forms such as:
"if policy & launch::deferred
is non-zero"
which better acknowledges the fact that the obtained values does not necessarily
undergo an implicit conversion.
I think the current wording in the file system spec. must be changed, especially for
invalid expressions of the form:
((options & copy_options::recursive) || !(options))
A similar problem arises in the usage of the bitmask type perms
for the
expression:
((prms & add_perms) && (prms & remove_perms))
The only way how to describe this with a scoped enum is the lengthier form
((prms & perms::add_perms) != perms::none && (prms & perms::remove_perms) !=
perms::none)
thus fixing several problems:
add_perms
and remove_perms
is incorrect[20 May 2014 Beman Dawes provides proposed wording. Fixing invalid C++ is editorial, but treating this as an issue ensures more people review the proposed changes.]
[17 Jun 2014 Rapperswil LWG requests issue be handled as editorial.]
Proposed resolution:
Change 15.3 Copy [fs.op.copy]:
Before the first use of
f
andt
:
- If
(options & copy_options::create_symlinks) != copy_options::none || (options & copy_options::skip_symlinks) != copy_options::none
, thenauto f = symlink_status(from)
and if neededauto t = symlink_status(to)
.- Otherwise,
auto f = status(from)
and if neededauto t = status(to)
.Report an error as specified in Error reporting (7) if:
!exists(f)
, orequivalent(from, to)
, oris_other(f) || is_other(t)
, oris_directory(f) && is_regular_file(t)
.If
is_symlink(f)
, then:
- If
(options & copy_options::skip_symlinks) != copy_options::none
, then return.- Otherwise if
!exists(t)
, thencopy_symlink(from, to, options)
.- Otherwise report an error as specified in Error reporting (7).
Otherwise if
is_regular_file(f)
, then:
- If
(options & copy_options::directories_only) != copy_options::none
, then return.- Otherwise if
(options & copy_options::create_symlinks ) != copy_options::none
, then create a symbolic link to the source file.- Otherwise if
(options & copy_options::create_hard_links) != copy_options::none
, then create a hard link to the source file.- Otherwise if
is_directory(t)
, thencopy_file(from, to/from.filename(), options)
.- Otherwise,
copy_file(from, to, options)
.
Otherwise if
is_directory(f) && ((options & copy_options::recursive) != copy_options::none ||
then:!(options == copy_options::none))
- If
!exists(t)
, thencreate_directory(to, from)
.- Then, iterate over the files in
from
, as if byfor (directory_entry& x : directory_iterator(from))
, and for each iterationcopy(x.path(), to/x.path().filename(), options | copy_options::unspecified)
.
Change 15.4 Copy file [fs.op.copy_file]:
If
exists(to) &&
report a file already exists error as specified in Error reporting (7).
!(options & (copy_options::skip_existing | copy_options::overwrite_existing | copy_options::update_existing)) == copy_options::noneIf
!exists(to) || (options & copy_options::overwrite_existing) != copy_options::none || ((options & copy_options::update_existing) != copy_options::none && last_write_time(from) > last_write_time(to)) ||
copy the contents and attributes of the file!(options & (copy_options::skip_existing | copy_options::overwrite_existing | copy_options::update_existing)) == copy_options::nonefrom
resolves to the fileto
resolves to.
Change 15.26 Permissions [fs.op.permissions]:
Requires:
!((prms & perms::add_perms) != perms::none && (prms & perms::remove_perms) != perms::none)
.
Section: X 6 [fs.filesystem.synopsis] Status: NAD Submitter: Beman Dawes Opened: 2014-04-12 Last modified: 2014-07-05
View all other issues in 6 [fs.filesystem.synopsis].
View all issues with NAD status.
Discussion:
bool equivalent(const path& p1, const path& p2);
has always thrown a exception if neither file exists,
with rationale that if they don't exist, it isn't possible to tell
if two paths are equivalent. Dave Abrahams has reported that this
is counter-intuitive and hard to teach.
An alternative if neither path exists would be to return true if they are lexically equal (operator==), otherwise return false.
This was not a national body comment, and Dave is the only one I can recall ever complaining about the current behavior. On the other hand, any complaint from Dave deserves serious consideration.
[17 Jun 2014 Rapperswil LWG considers this NAD. Mixing lexical and existence based behavior is not desirable.]
Proposed resolution: