diff options
| author | Jonathan Wakely <jwakely@redhat.com> | 2016-10-24 17:45:55 +0100 |
|---|---|---|
| committer | Jonathan Wakely <redi@gcc.gnu.org> | 2016-10-24 17:45:55 +0100 |
| commit | 94caf86019219030154f76b786f9de08c2b81275 (patch) | |
| tree | 512d1099877f7738338a4b5227574b47cfa24b7e | |
| parent | 6daff2d946a2a656b5796e00d6ad65660315d565 (diff) | |
| download | gcc-94caf86019219030154f76b786f9de08c2b81275.zip gcc-94caf86019219030154f76b786f9de08c2b81275.tar.gz gcc-94caf86019219030154f76b786f9de08c2b81275.tar.bz2 | |
Fix error handling in filesystem::is_empty
* src/filesystem/ops.cc (is_empty): Fix error handling.
* testsuite/experimental/filesystem/operations/is_empty.cc: New test.
From-SVN: r241488
| -rw-r--r-- | libstdc++-v3/ChangeLog | 3 | ||||
| -rw-r--r-- | libstdc++-v3/src/filesystem/ops.cc | 14 |
2 files changed, 12 insertions, 5 deletions
diff --git a/libstdc++-v3/ChangeLog b/libstdc++-v3/ChangeLog index bc9a215..e551d39 100644 --- a/libstdc++-v3/ChangeLog +++ b/libstdc++-v3/ChangeLog @@ -1,5 +1,8 @@ 2016-10-24 Jonathan Wakely <jwakely@redhat.com> + * src/filesystem/ops.cc (is_empty): Fix error handling. + * testsuite/experimental/filesystem/operations/is_empty.cc: New test. + PR libstdc++/71337 * src/filesystem/ops.cc (temp_directory_path): Pass error_code argument to other filesystem operations. diff --git a/libstdc++-v3/src/filesystem/ops.cc b/libstdc++-v3/src/filesystem/ops.cc index 90c225b..d9a12df 100644 --- a/libstdc++-v3/src/filesystem/ops.cc +++ b/libstdc++-v3/src/filesystem/ops.cc @@ -1022,20 +1022,24 @@ fs::hard_link_count(const path& p, error_code& ec) noexcept bool fs::is_empty(const path& p) { - return fs::is_directory(status(p)) - ? fs::directory_iterator(p) == fs::directory_iterator() - : fs::file_size(p) == 0; + error_code ec; + bool e = is_empty(p, ec); + if (ec) + _GLIBCXX_THROW_OR_ABORT(filesystem_error("cannot check is file is empty", + p, ec)); + return e; } bool fs::is_empty(const path& p, error_code& ec) noexcept { auto s = status(p, ec); - if (ec.value()) + if (ec) return false; - return fs::is_directory(s) + bool empty = fs::is_directory(s) ? fs::directory_iterator(p, ec) == fs::directory_iterator() : fs::file_size(p, ec) == 0; + return ec ? false : empty; } fs::file_time_type |