commit 4f3dc0e9e7295617344de968fe20c496f0f0922a
author Eric Fiselier <eric@efcs.ca> Mon Mar 26 06:23:55 2018 +0000
committer Eric Fiselier <eric@efcs.ca> Mon Mar 26 06:23:55 2018 +0000
tree b47d724f57aaca4ff5034efacd0cb7617cad5d45
parent 0cea3a5796a2fe43484e114a6bb2d53a6a4d4055

Implement filesystem::perm_options specified in NB comments.

The NB comments for filesystem changed permissions and added
a new enum `perm_options` which control how the permissions
are applied.

This implements than NB resolution

llvm-svn: 328476
Cr-Mirrored-From: sso://chromium.googlesource.com/_direct/external/github.com/llvm/llvm-project
Cr-Mirrored-Commit: 4d334c4cdd4a58dc930cbc2ed8abd77f1a1cd4bb

include/experimental/filesystem

diff --git a/include/experimental/filesystem b/include/experimental/filesystem
index cdfe9e2..47cc0c5 100644
--- a/include/experimental/filesystem
+++ b/include/experimental/filesystem
@@ -68,6 +68,7 @@
 
     enum class file_type;
     enum class perms;
+    enum class perm_options;
     enum class copy_options;
     enum class directory_options;
 
@@ -178,8 +179,11 @@
     void last_write_time(const path& p, file_time_type new_time,
                                  error_code& ec) _NOEXCEPT;
 
-    void permissions(const path& p, perms prms);
-    void permissions(const path& p, perms prms, error_code& ec) _NOEXCEPT;
+    void permissions(const path& p, perms prms,
+                     perm_options opts=perm_options::replace);
+    void permissions(const path& p, perms prms, error_code& ec) noexcept;
+    void permissions(const path& p, perms prms, perm_options opts,
+                     error_code& ec);
 
     path read_symlink(const path& p);
     path read_symlink(const path& p, error_code& ec);
@@ -290,10 +294,6 @@
     sticky_bit   = 01000,
     mask         = 07777,
     unknown      = 0xFFFF,
-
-    add_perms        = 0x10000,
-    remove_perms     = 0x20000,
-    symlink_nofollow = 0x40000
 };
 
 _LIBCPP_INLINE_VISIBILITY
@@ -324,6 +324,41 @@
 inline perms& operator^=(perms& _LHS, perms _RHS)
 { return _LHS = _LHS ^ _RHS; }
 
+enum class _LIBCPP_ENUM_VIS perm_options : unsigned char {
+  replace  = 1,
+  add      = 2,
+  remove   = 4,
+  nofollow = 8
+};
+
+_LIBCPP_INLINE_VISIBILITY
+inline _LIBCPP_CONSTEXPR perm_options operator&(perm_options _LHS, perm_options _RHS)
+{ return static_cast<perm_options>(static_cast<unsigned>(_LHS) & static_cast<unsigned>(_RHS)); }
+
+_LIBCPP_INLINE_VISIBILITY
+inline _LIBCPP_CONSTEXPR perm_options operator|(perm_options _LHS, perm_options _RHS)
+{ return static_cast<perm_options>(static_cast<unsigned>(_LHS) | static_cast<unsigned>(_RHS)); }
+
+_LIBCPP_INLINE_VISIBILITY
+inline _LIBCPP_CONSTEXPR perm_options operator^(perm_options _LHS, perm_options _RHS)
+{ return static_cast<perm_options>(static_cast<unsigned>(_LHS) ^ static_cast<unsigned>(_RHS)); }
+
+_LIBCPP_INLINE_VISIBILITY
+inline _LIBCPP_CONSTEXPR perm_options operator~(perm_options _LHS)
+{ return static_cast<perm_options>(~static_cast<unsigned>(_LHS)); }
+
+_LIBCPP_INLINE_VISIBILITY
+inline perm_options& operator&=(perm_options& _LHS, perm_options _RHS)
+{ return _LHS = _LHS & _RHS; }
+
+_LIBCPP_INLINE_VISIBILITY
+inline perm_options& operator|=(perm_options& _LHS, perm_options _RHS)
+{ return _LHS = _LHS | _RHS; }
+
+_LIBCPP_INLINE_VISIBILITY
+inline perm_options& operator^=(perm_options& _LHS, perm_options _RHS)
+{ return _LHS = _LHS ^ _RHS; }
+
 enum class _LIBCPP_ENUM_VIS copy_options : unsigned short
 {
     none               = 0,
@@ -1286,7 +1321,7 @@
 void __last_write_time(const path& p, file_time_type new_time,
         error_code *ec=nullptr);
 _LIBCPP_FUNC_VIS
-void __permissions(const path& p, perms prms, error_code *ec=nullptr);
+void __permissions(const path&, perms, perm_options, error_code* = nullptr);
 _LIBCPP_FUNC_VIS
 path __read_symlink(const path& p, error_code *ec=nullptr);
 _LIBCPP_FUNC_VIS
@@ -1664,13 +1699,20 @@
 }
 
 inline _LIBCPP_INLINE_VISIBILITY
-void permissions(const path& __p, perms __prms) {
-    __permissions(__p, __prms);
+void permissions(const path& __p, perms __prms,
+                 perm_options __opts = perm_options::replace) {
+    __permissions(__p, __prms, __opts);
 }
 
 inline _LIBCPP_INLINE_VISIBILITY
-void permissions(const path& __p, perms __prms, error_code& __ec)  {
-    __permissions(__p, __prms, &__ec);
+void permissions(const path& __p, perms __prms, error_code& __ec) _NOEXCEPT {
+    __permissions(__p, __prms, perm_options::replace, &__ec);
+}
+
+inline _LIBCPP_INLINE_VISIBILITY
+void permissions(const path& __p, perms __prms, perm_options __opts,
+                 error_code& __ec)  {
+    __permissions(__p, __prms, __opts, &__ec);
 }
 
 inline _LIBCPP_INLINE_VISIBILITY