[all-commits] [llvm/llvm-project] e69c65: [libcxx] Test accessing a directory on windows tha...

Martin Storsjö via All-commits all-commits at lists.llvm.org
Thu Mar 11 11:08:22 PST 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: e69c65d5c45557e140b13237448581d28bbd5846
      https://github.com/llvm/llvm-project/commit/e69c65d5c45557e140b13237448581d28bbd5846
  Author: Martin Storsjö <martin at martin.st>
  Date:   2021-03-11 (Thu, 11 Mar 2021)

  Changed paths:
    M libcxx/src/filesystem/directory_iterator.cpp
    M libcxx/test/std/input.output/filesystems/class.directory_entry/directory_entry.cons/path.pass.cpp
    M libcxx/test/std/input.output/filesystems/class.directory_iterator/directory_iterator.members/ctor.pass.cpp
    M libcxx/test/std/input.output/filesystems/class.rec.dir.itr/rec.dir.itr.members/ctor.pass.cpp
    M libcxx/test/std/input.output/filesystems/fs.op.funcs/fs.op.exists/exists.pass.cpp
    M libcxx/test/support/filesystem_test_helper.h

  Log Message:
  -----------
  [libcxx] Test accessing a directory on windows that gives "access denied" errors

Fix handling of skip_permission_denied on windows; after converting
the return value of GetLastError() to a standard error_code, ec.value()
is in the standard errc range, not a native windows error code. This
was missed in 156180727d6c347eda3ba749730707acb8a48093.

The directory "C:\System Volume Information" does seem to exist and
have these properties on most relevant contempory setups.

Differential Revision: https://reviews.llvm.org/D98166




More information about the All-commits mailing list