linuxdcpp-team team mailing list archive

[cologic <238482@xxxxxxxxxxxxxxxxxx>]

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