[Bug libstdc++/83891] std::filesystem::path::is_absolute for Windows

2018-05-18 Thread redi at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83891

--- Comment #4 from Jonathan Wakely  ---
This is still wrong, for Windows it should be:

  return has_root_name() && has_root_directory();

But I'll fix it as part of PR 78870

[Bug libstdc++/83891] std::filesystem::path::is_absolute for Windows

2018-05-15 Thread redi at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83891

Jonathan Wakely  changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED
   Target Milestone|--- |8.2

--- Comment #3 from Jonathan Wakely  ---
Fixed for 8.2, thanks for the report.

[Bug libstdc++/83891] std::filesystem::path::is_absolute for Windows

2018-05-15 Thread redi at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83891

--- Comment #2 from Jonathan Wakely  ---
Author: redi
Date: Tue May 15 13:51:03 2018
New Revision: 260260

URL: https://gcc.gnu.org/viewcvs?rev=260260=gcc=rev
Log:
PR libstdc++/83891 fix path::is_absolute() for non-POSIX targets

The correct definition seems to be has_root_directory() for all systems
we care about.

PR libstdc++/83891
* include/bits/fs_path.h (path::is_absolute()): Use same definition
for all operating systems.
* include/experimental/bits/fs_path.h (path::is_absolute()): Likewise.
* testsuite/27_io/filesystem/path/query/is_absolute.cc: New.
* testsuite/27_io/filesystem/path/query/is_relative.cc: Fix comment.
* testsuite/experimental/filesystem/path/query/is_absolute.cc: New.

Added:
   
branches/gcc-8-branch/libstdc++-v3/testsuite/27_io/filesystem/path/query/is_absolute.cc
   
branches/gcc-8-branch/libstdc++-v3/testsuite/experimental/filesystem/path/query/is_absolute.cc
Modified:
branches/gcc-8-branch/libstdc++-v3/ChangeLog
branches/gcc-8-branch/libstdc++-v3/include/bits/fs_path.h
branches/gcc-8-branch/libstdc++-v3/include/experimental/bits/fs_path.h
   
branches/gcc-8-branch/libstdc++-v3/testsuite/27_io/filesystem/path/query/is_relative.cc

[Bug libstdc++/83891] std::filesystem::path::is_absolute for Windows

2018-05-15 Thread redi at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83891

--- Comment #1 from Jonathan Wakely  ---
Author: redi
Date: Tue May 15 13:50:49 2018
New Revision: 260259

URL: https://gcc.gnu.org/viewcvs?rev=260259=gcc=rev
Log:
PR libstdc++/83891 fix path::is_absolute() for non-POSIX targets

The correct definition seems to be has_root_directory() for all systems
we care about.

PR libstdc++/83891
* include/bits/fs_path.h (path::is_absolute()): Use same definition
for all operating systems.
* include/experimental/bits/fs_path.h (path::is_absolute()): Likewise.
* testsuite/27_io/filesystem/path/query/is_absolute.cc: New.
* testsuite/27_io/filesystem/path/query/is_relative.cc: Fix comment.
* testsuite/experimental/filesystem/path/query/is_absolute.cc: New.

Added:
trunk/libstdc++-v3/testsuite/27_io/filesystem/path/query/is_absolute.cc
   
trunk/libstdc++-v3/testsuite/experimental/filesystem/path/query/is_absolute.cc
Modified:
trunk/libstdc++-v3/ChangeLog
trunk/libstdc++-v3/include/bits/fs_path.h
trunk/libstdc++-v3/include/experimental/bits/fs_path.h
trunk/libstdc++-v3/testsuite/27_io/filesystem/path/query/is_relative.cc

[Bug libstdc++/83891] std::filesystem::path::is_absolute for Windows

2018-05-14 Thread redi at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83891

Jonathan Wakely  changed:

   What|Removed |Added

 Status|UNCONFIRMED |NEW
   Last reconfirmed||2018-05-14
 Ever confirmed|0   |1