[MPlayer-dev-eng] [PATCH] Interlaced vo_png support

Rich Felker dalias at aerifal.cx
Tue May 2 22:09:05 CEST 2006


On Tue, May 02, 2006 at 08:53:13PM +0200, Ivo wrote:
> On Tuesday 02 May 2006 20:16, Diego Biurrun wrote:
> > On Tue, May 02, 2006 at 10:43:52AM -0400, Rich Felker wrote:
> > > On Tue, May 02, 2006 at 03:35:09PM +0300, Jan Knutar wrote:
> > > > On Tuesday 02 May 2006 03:30, Rich Felker wrote:
> > > > > Totally unrelated. "interlaced" has a completely different meaning
> > > > > for png, it's a stupid 'incremental loading' thing for web use. One
> > > > > of the many hideous bloated features in png.
> > > >
> > > > However, -lavdopts lowres isn't a hideous bloated feature? ;-)
> > >
> > > Unrelated and -lavdopts lowres is just a few lines of code.
> >
> > As is the patch for interlaced PNG support.  Can we please evaluate the
> > code on its merits?
> 
> It actually is not about the incremental loading thingy Rich talks about, 
> but it writes separate files for each field of interlaced content. I don't 
> see why this should be added, as RC pointed out, one can use -vf tfields 
> -vo png. The resulting files are the same.

Indeed. Sorry for not RTFP'ing. I guess I should never assume that
patches are sane..

Rich




More information about the MPlayer-dev-eng mailing list