[MPlayer-cvslog] r29730 - in trunk: cfg-common-opts.h mencoder.c mplayer.c

[Reimar Döffinger]

On Mon, Sep 28, 2009 at 10:13:29AM -0400, The Wanderer wrote:
> On 09/28/2009 07:32 AM, reimar wrote:
> 
> > Author: reimar
> > Date: Mon Sep 28 13:32:27 2009
> > New Revision: 29730
> > 
> > Log:
> > Add a -nosub option that allows disabling auto-selected subtitles as e.g. the
> > mkv demuxer supports.
> 
> Could this be extended (or, for that matter, another option be added) to
> allow disabling previously-explicitly-selected subtitles?

If this options doesn't do that, please provide a bug report (well,
actually just testing and telling me the result should be ok).

> This option seems like it could be the correct one to disable subtitles
> that way, syntactically speaking, but from a quick test it doesn't do
> that. Is there a reason why it shouldn't?

Probably that I missed a dvdsub_id < 0 that should be a dvdsub_id == -1
or that there is some other buggy code that makes -slang override -sid.
For that later case, try if -sid 200 or so reliably disables
subtitles.