[MPlayer-dev-eng] [PATCH] Make frameno.avi optional

Guillaume POIRIER poirierg at gmail.com
Mon Sep 19 15:45:16 CEST 2005


Hi,

On 9/19/05, Ivo <ivop at euronet.nl> wrote:
> On Monday 19 September 2005 13:22, Oded Shimon wrote:
> > I'd like to atleast make this very obsolete feature optional (I'm in
> > favor of ripping it out of MEncoder altogether, MEncoder's source is
> > hackish enough as it is...).
> > With this patch, you have -frameno-file where you can now actually
> > specify the frameno filename, and also MEncoder doesn't show the very
> > annoying and very stupid 'file not found frameno.avi' at startup.
> > I'm not sure if to document this option at all, as it'll encourage people
> > to actually use it.
> >
> > Thoughts? I'll commit soon if no-one objects, and that's a threat. (It
> > breaks any scripts which assumed MEncoder automatically uses frameno.avi)
> 
> Why not just change the mp_msg call to MSGL_V and leave the rest as it is? I
> see no reason to change the behaviour of obsolete functionality.

Seconded. Even if that function doesn't work so well, some ppl like it
and use it at their own risk. Not maintaining compatibility is what
flooded the ML with the question related to the new FMP4 fourcc, which
was very annoying.

Guillaume

-- 
Reading doesn't hurt, really!
  -- Dominik 'Rathann' Mierzejewski




More information about the MPlayer-dev-eng mailing list