Alanaktion / transmisson-remote-gui

Automatically exported from code.google.com/p/transmisson-remote-gui
GNU General Public License v2.0
1 stars 0 forks source link

GetAppConfigDir returns the incorrect data for non-ASCII symbols #199

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Main.pas (r231)

row 558:
      FHomeDir:=IncludeTrailingPathDelimiter(GetAppConfigDir(False));

Replace it please with this:

      FHomeDir:=IncludeTrailingPathDelimiter(GetAppConfigDirUTF8(False));

Thanks.

Original issue reported on code.google.com by Alex7...@gmail.com on 5 Mar 2010 at 11:25

GoogleCodeExporter commented 9 years ago
[deleted comment]
GoogleCodeExporter commented 9 years ago
Not all so is simple as it has appeared.
It was necessary to correct very much.
I have made a patch.

Original comment by Alex7...@gmail.com on 7 Mar 2010 at 5:05

Attachments:

GoogleCodeExporter commented 9 years ago
I don't understand why it is necessary to convert all paths to utf8 for internal
storage and then convert them back to the system encoding before doing any file
system operation.

Currently all paths are retrieved and stored in the system encoding.

Original comment by j...@cp-lab.com on 17 Mar 2010 at 1:07

GoogleCodeExporter commented 9 years ago
Probably I compiled the project with wrong options :(
What parametres of compilation need to be set for Lazarus?
Your exe-file from the distribution kit works perfectly.
But if to copile the project from source codes, it not so.

ps: fpc 2.4.1, lazarus 0.9.29.23824

Original comment by Alex7...@gmail.com on 17 Mar 2010 at 8:54

GoogleCodeExporter commented 9 years ago
No special options are needed.
Recently I switched to fpc 2.4.x and Lazarus 0.9.28.x
Probably there is a problem with your Lazarus version. Try to use Lazarus 
0.9.28.x
and the latest transgui svn.

Original comment by j...@cp-lab.com on 18 Mar 2010 at 1:25

GoogleCodeExporter commented 9 years ago

Original comment by j...@cp-lab.com on 26 Mar 2010 at 1:48