Re: [MPlayer-dev-eng] [PATCH] mencoder slang vobsub fix
- Date: Fri, 5 Sep 2014 18:30:26 -0400
- From: Edward Batutis <edward@xxxxxxxxxxx>
- Subject: Re: [MPlayer-dev-eng] [PATCH] mencoder slang vobsub fix
Thanks for your reply.
I am using mencoder as part of a mediatomb application that recodes video
for tablet devices, so mencoder is still a useful thing. Sorry to hear it
isn't being maintained, but that's the way it goes I guess.
I need to do the "if" and update dvdsub_id because I am following the
pattern used farther down in the source file. Also, the code that uses the
value won't do the right thing unless I set it.
In fact, when testing the application, I noticed that if I didn't set it, I
didn't fix the bug.
On Thu, Sep 4, 2014 at 2:41 PM, Reimar DÃffinger <Reimar.Doeffinger@xxxxxx>
> On Tue, Mar 11, 2014 at 01:12:28PM -0400, Edward Batutis wrote:
> > Hi,
> > This is my very first patch for mplayer/mencoder.
> And promptly your patch fell through the cracks :(
> Very sorry about that!
> Note that mencoder isn't properly maintained anymore,
> which should explain why things don't work in it
> that work in MPlayer, but also is the reason for my main comment:
> > + if(dvdsub_lang && dvdsub_id==-1)
> This is quite a bit different from what MPlayer does,
> increasing the risk it will break in the future
> (I suspect it also means they will behave different
> when they are given a certain set of options).
> Why can't you just do
> here like MPlayer does (without the if and without
> changing dvdsub_id)?
MPlayer-dev-eng mailing list