aMule Forum

Please login or register.

Login with username, password and session length
Advanced search  

News:

We're back! (IN POG FORM)

Pages: 1 2 [3]

Author Topic: settings: new option for filesystem (filenames) encoding  (Read 20978 times)

Kry

  • Ex-developer
  • Retired admin
  • Hero Member
  • *****
  • Karma: -665
  • Offline Offline
  • Posts: 5795
Re: settings: new option for filesystem (filenames) encoding
« Reply #30 on: January 03, 2008, 02:06:51 PM »

we don't lose portability, we use fopen and then associate the FILE* to a wxFFile object and use it like we're doing now.
Logged

phoenix

  • Evil respawning bird from aMule Dev Team
  • Developer
  • Hero Member
  • *****
  • Karma: 44
  • Offline Offline
  • Posts: 2503
  • The last shadow you'll ever see
Re: settings: new option for filesystem (filenames) encoding
« Reply #31 on: January 30, 2008, 04:08:00 AM »

Everybody, please test tomorrow svn tarball. Xaignar has probably fixed this issue.
Logged

mr_hyde

  • Global Moderator
  • Full Member
  • *****
  • Karma: 12
  • Offline Offline
  • Posts: 105
Re: settings: new option for filesystem (filenames) encoding
« Reply #32 on: January 30, 2008, 11:13:48 AM »

Just tried with amule and amuled, Incoming and Temp on a NTFS partition (mounted by ntfs-3g). Everything seems to work correctly!
Thanks again for your marvellous job!

EDIT:  I forgot to report my locale settings: en_US.utf8 and ntfs-3g partitions mounted with options "locale=en_US.utf8"

Bye,
  Mr Hyde
« Last Edit: January 30, 2008, 10:09:47 PM by mr_hyde »
Logged

macias

  • Full Member
  • ***
  • Karma: 0
  • Offline Offline
  • Posts: 110
Re: settings: new option for filesystem (filenames) encoding
« Reply #33 on: February 05, 2008, 05:05:53 PM »

I tested 20080205 version and comparing it to patched (ConvAmule) 20071226 I would say it is a regression -- no matter what I do (*), I don't see Polish characters.

(*) LANG=en_EN or LANG=pl_PL.ISO-8859-2  (I use ISO-8859-2 characters in filenames)

In 20071216 (but only patched!) Polish characters are displayed 100% correctly.
Logged

wuischke

  • Developer
  • Hero Member
  • *****
  • Karma: 183
  • Offline Offline
  • Posts: 4292
Re: settings: new option for filesystem (filenames) encoding
« Reply #34 on: February 05, 2008, 06:49:36 PM »

If I'm not mistaken, this is intended behaviour.

We use only Unicode now, so you'll get a misdisplay, but at least all the filenames should be written correctly to disk. It would be preferable to use Unicode on your system, too, but iirc you stated some reasons not to use Unicode earlier.

Keep in mind that I haven't read the code after the changes yet, so I could be wrong about what I say.
Logged

macias

  • Full Member
  • ***
  • Karma: 0
  • Offline Offline
  • Posts: 110
Re: settings: new option for filesystem (filenames) encoding
« Reply #35 on: February 05, 2008, 07:40:17 PM »

We use only Unicode now, so you'll get a misdisplay, but at least all the filenames should be written correctly to disk.

It is impossible by definition? Unicode can be properly written if the FS is Unicode. If it is not, and any conversion is not done there is no change (i.e. filenames will be written incorrectly).

imho amule should take LANG into account because it is quite good indicator how the system is set.
Logged

phoenix

  • Evil respawning bird from aMule Dev Team
  • Developer
  • Hero Member
  • *****
  • Karma: 44
  • Offline Offline
  • Posts: 2503
  • The last shadow you'll ever see
Re: settings: new option for filesystem (filenames) encoding
« Reply #36 on: February 05, 2008, 08:41:41 PM »

macias,

Xaignar is now looking at your issue and it will be fixed. Please test and report your results after he says it is committed.

Cheers!
Logged

Xaignar

  • Admin and Code Junky
  • Hero Member
  • *****
  • Karma: 19
  • Offline Offline
  • Posts: 1103
Re: settings: new option for filesystem (filenames) encoding
« Reply #37 on: February 05, 2008, 09:35:14 PM »

We use only Unicode now, so you'll get a misdisplay, but at least all the filenames should be written correctly to disk.
It is impossible by definition?
No, the problem with not displaying the Polish chars is because of a bug in the demangling routines used for printing. The filename strings used when actually accessing the filesystem are not affected by this problem. In any case, I've commited a fix, so that these routines should now use the system locale when possible, instead of ISO-8859-1, which was the previously hardcoded fallback.
Logged

macias

  • Full Member
  • ***
  • Karma: 0
  • Offline Offline
  • Posts: 110
Re: settings: new option for filesystem (filenames) encoding
« Reply #38 on: February 06, 2008, 06:33:44 PM »

No, the problem with not displaying the Polish chars is because of a bug in the demangling routines used for printing. The filename strings used when actually accessing the filesystem are not affected by this problem. In any case, I've commited a fix, so that these routines should now use the system locale when possible, instead of ISO-8859-1, which was the previously hardcoded fallback.

I downloaded 20080206 SVN and all filenames with Polish characters are now displayed 100% properly. THANK YOU!
Logged

Xaignar

  • Admin and Code Junky
  • Hero Member
  • *****
  • Karma: 19
  • Offline Offline
  • Posts: 1103
Re: settings: new option for filesystem (filenames) encoding
« Reply #39 on: February 06, 2008, 07:21:59 PM »

I downloaded 20080206 SVN and all filenames with Polish characters are now displayed 100% properly. THANK YOU!

Glad to hear it, and appologies for the oversight on my part.
Logged
Pages: 1 2 [3]