neo_in_matrix@… — Sun, 29 Aug 2010 15:57:16 GMT

BTW, I have #define BOOST_FILESYSTEM_VERSION 3 so that I am using the newer version 3 of filesystem.hpp.

neo_in_matrix@… — Sun, 29 Aug 2010 16:00:42 GMT

Yeah... my first message is garbled by this bug system.

Let's say the code is like this:

filesystem::path path1("\\abc\\");
filesystem::path path2("\\xyz\\");
filesystem::path path3 = path1 / path2;
//path3 will be \abc\\xyz\, which has redundant path seps.

status changed; resolution set

Beman Dawes — Tue, 30 Dec 2014 23:06:21 GMT

status new → closed
resolution → wontfix

See http://cplusplus.github.io/filesystem-ts/working-draft.html#path-appends

The ISO C++ filesystem TS says:

Appends path::preferred_separator to pathname unless:

an added separator would be redundant, or
would change a relative path to an absolute path [Note: An empty path is relative. — end note], or
p.empty(), or
*p.native().cbegin() is a directory separator.

Then appends p.native() to pathname.

Boost.Filesystem follows the TS here. The TS does not add a separator if it would be redundant, but if native() ends with a separator and the append argument begins with a separator, both are retained.

You are free to report this to the C++ committee as defect, and if they decide to change the spec then Boost.Filesystem will follow their lead. But I'm not aware of any reason for adding a gratuitous difference between Boost and other implementation.

Thanks for your interest in Boost.Filesystem,

--Beman

PS: Test case added to verify the library works as expected per your test case.

Boost C++ Libraries: Ticket #4611: path::operator / does not remove redundant separators

status changed; resolution set