[MPlayer-dev-eng] [PATCH] vd_null to output black frames

[Alexander Strasser]

Hi,

Reimar Döffinger wrote:
> On Wed, Oct 31, 2012 at 03:41:53PM +0800, Xidorn Quan wrote:
> > On Tue, Oct 16, 2012 at 2:26 PM, Reimar Döffinger wrote:
> > > On 15 Oct 2012, at 21:48, Alexander Strasser <eclipse7 at gmx.net> wrote:
> > > >
> > > >  Done and committed.
> > >
> > > Thanks for taking care of it.
> > > Btw. it might make sense to point to the special-case null and black
> > > strcmp code from the codecs.conf entries, since it is related to why a
> > > codec entry without any fourccs even makes sense.
> > > Alternatively, having a flag in the codec structure to indicate such
> > > "dummy" codecs might make it more obvious, too.
> > >
> > 
> > The patch attached adds a flag for dummy codecs. Is it ok?
> 
> I had actually thought making it a flag in the codec struct in the vd_*.c file,
> but I think I actually like your idea better :-)
> So, looks fine to me.

  I was thinking if "dummy" is expressive enough as a name for a configuration
flag. Would "ignore_fourcc" be easier to understand for someone reading codecs.conf?

  I do not particularly care about the naming but I do like the patch. So please
do not understand this as a blocking issue. Just wanted to find out what other
developers think.

  Alexander