It's been reported that, indeed, the patch previously does not work. I still cannot usefully test this, but I'll make one more attempt at simply a more thorough version of the above, addressing both points. It does translate / to \\ and it also wraps several more filepath-related Win32 API callsites in File.cpp and FileReader.cpp.
It still doesn't work on WINE 1.6.2; the test suite creates the following error console messages: [----------] 1 test from testarchive [ RUN ] testarchive.test_archive U::gNP: test/data/out/gtest.h => \\?\test\data\out\gtest.h U::gNP: test/data/ => \\?\test\data\ U::gNP: test/data/out/ => \\?\test\data\out\ U::gNP: test/data/out/gtest.h => \\?\test\data\out\gtest.h F::F: test/data/out/gtest.h \\?\test\data\out\gtest.h U::gNP: test/data/out/gtest.h => \\?\test\data\out\gtest.h Thrown: FileException: Path not found. test/testarchive.cpp:17: Failure Failed FileException: Path not found. [ FAILED ] testarchive.test_archive (3 ms) [----------] 1 test from testarchive (5 ms total) U::gNP == Util::getNtPath and F::F is File::File. Removing the Util::getNtPath call from F::F does, in fact, allow the test suite to pass under WINE 1.6, but as far as I can tell, based on MSDN documentation, that filename translation is correct. Without the ability to test on my end, this is probably about as far as I can go. ** Patch added: "Doesn't actually work under WINE 1.6.2" https://bugs.launchpad.net/dcplusplus/+bug/238482/+attachment/4441592/+files/ntpaths_v2.patch -- You received this bug notification because you are a member of Dcplusplus-team, which is subscribed to DC++. https://bugs.launchpad.net/bugs/238482 Title: Create support for paths longer than 255 characters Status in DC++: Confirmed Bug description: I'm getting this error on 2 files with long file-/pathnames (whole path is bigger than 255 characters). At least I think the length of the path causes the problem, as I still can can access those 2 files with other programs. DC++ version: 0.699 To manage notifications about this bug go to: https://bugs.launchpad.net/dcplusplus/+bug/238482/+subscriptions _______________________________________________ Mailing list: https://launchpad.net/~linuxdcpp-team Post to : linuxdcpp-team@lists.launchpad.net Unsubscribe : https://launchpad.net/~linuxdcpp-team More help : https://help.launchpad.net/ListHelp
