Perhaps someone could add this to games as well ??  Please?

/PS

-----Ursprungligt meddelande-----
Fr�n: Dirk Meyer
Till: [EMAIL PROTECTED]
Skickat: 2003-12-08 19:26
�mne: [Freevo-devel] Re: FXD handler bugfixes

Matthieu Weber wrote:
> Here is a patch that fixes the fxdhandler, videoitem and __init__ of
the
> video subsystem. Unless the DTD for video fxd files has changed, the
> fxdhandler implementation does not respect the DTD (and none of my fxd
> work anymore).

Sorry. In the beginning I didn't even want to rewrite it, only add the
new fxd parser stuff. Since I don't use most of the fxd features, some
may got lost.

> Moreover, the videoitem patch provides a dirty fix for the following
> problem: when playing a video with a subtitle file specified in an FXD
> file, the subtitle filename is enclosed in "". But this means that the
> "" become part of the filename (so mplayer doesn't find the file), and
> as the command line passed to mplayer is split on spaces, it doesn't
> protect a filename containing spaces to be split, thus messing up the
> command line of mplayer completely. The fix just removes the "", thus
> forbidding the filenames containing spaces.

All this "" stuff is too dirty. That's why I switched to a list in
mplayer.py and xine.py (at some point before adding the filename, the
string will be transformed to an argv list).

The subtitle file is also no 'mplayer' option, xine may also use
it. So maybe you should add the filename of the subtitle to the
mplayer_options and add a variable subtitle_file to VideoItem. Than
the player could handle that. Does this sounds ok or am I missing
something? 


Dischi

-- 
Just remember, if the world didn't suck, we'd all fall off.


-------------------------------------------------------
This SF.net email is sponsored by: SF.net Giveback Program.
Does SourceForge.net help you be more productive?  Does it
help you create better code?  SHARE THE LOVE, and help us help
YOU!  Click Here: http://sourceforge.net/donate/
_______________________________________________
Freevo-devel mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/freevo-devel


-------------------------------------------------------
This SF.net email is sponsored by: SF.net Giveback Program.
Does SourceForge.net help you be more productive?  Does it
help you create better code?  SHARE THE LOVE, and help us help
YOU!  Click Here: http://sourceforge.net/donate/
_______________________________________________
Freevo-devel mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/freevo-devel

Reply via email to