aMule Forum

English => en_Bugs => Topic started by: chemical on February 12, 2008, 11:47:46 AM

Title: Nodsu's Patch for Preview in Remote GUI
Post by: chemical on February 12, 2008, 11:47:46 AM

Quote

        Nodsu:
                * Patch to enable preview in remote GUI.

I really appreciate this patch as I always use a windows remote GUI to a linux amuled running on my router. This patch seems to work fine, however theres a caveat preventing the success for me:

Amulegui on my windows system obviously prefixes the filename of the partfile with a slash. This might be correct on unices, on Windows however it leads to the following construct:

H:\amule\temp/05.part

This string is handed over to VLC which doesn't accept the path. Would it be possible to check for Win32 during compilation and use a backslash instead of a slash then?

Thanks!
chemical

Title: Re: Nodsu's Patch for Preview in Remote GUI
Post by: Kry on February 12, 2008, 12:19:26 PM

There's a wx function for that.

Title: Re: Nodsu's Patch for Preview in Remote GUI
Post by: chemical on February 12, 2008, 06:01:47 PM

Maybe ... hm. How should.. How am I supposed hmm. hm. Nevermind. Thanks anyway.

Title: Re: Nodsu's Patch for Preview in Remote GUI
Post by: Xaignar on February 12, 2008, 09:18:27 PM

That's should be fixed in current SVN, as part of different commit:

Code: [Select]

- wxString fileWithoutMet;
- fileWithoutMet <<
- thePrefs::GetTempDir() << wxT("/") <<
- file->GetPartMetFileName().RemoveExt().GetRaw();
+ wxString fileWithoutMet = file->GetFullName().RemoveExt().GetRaw();

Title: Re: Nodsu's Patch for Preview in Remote GUI
Post by: chemical on February 12, 2008, 09:23:14 PM

Are you sure that file->GetFullName() returns the path I set as a Temp-Directory in the remote GUI? This directory is different than the one thats set up in the daemon.

Obviously thePrefs::GetTempDir() returned the right one, I don't know about the GetFullName().