[MPlayer-dev-eng] [PATCH] sub_select fix

[Joey Parrish]

On Fri, Sep 24, 2004 at 03:02:51PM -0500, Joey Parrish wrote:
> On Fri, Sep 24, 2004 at 07:50:59PM +0200, Moritz Bunkus wrote:
> > Heya,
> > 
> > > Can you provide me a link to download SRT subs to test?
> > 
> > Attached to this file.
> > 
> > > Can you also provide a gdb backtrace?
> > 
> > Sure. I've attached the log from...
> > 
> > ./mplayer -v -dvd-device /ftp/.rip/dvds/KILLBILL_VOL2/ -vo x11 -zoom \
> > -sub /ftp/.rip/mkv/samples/vsshort-en.srt dvd://1
> > 
> > (No, don't argue about the -vo please, this is my father's computer with
> > a very limited Linux setup.)
> > 
> > Here's the backtrace:
> > 
> > main (argc=10, argv=0xbffffb44) at mplayer.c:3752
> > 3752              tmp = subdata->filename;
> > (gdb) bt
> > #0  main (argc=10, argv=0xbffffb44) at mplayer.c:3752
> > 
> > Not very long ;)
> 
> Fixed it.  Also fixed a problem with ogg and mkv not working with
> -slang.  Attached new patch.

Committed.  The version I committed also documents the new function
using Doxygen.

Let the cola flow!

--Joey

-- 
"Of the seven dwarves, the only one who shaved was Dopey.
That should tell us something about the wisdom of shaving."