[FFmpeg-devel] VP5 interlaced mode, any hints?

Aurelien Jacobs aurel
Mon Dec 3 14:09:44 CET 2007


Reimar D?ffinger wrote:

> Hello,
> On Mon, Dec 03, 2007 at 02:05:47AM +0100, Aurelien Jacobs wrote:
> > Vitalijus Slavinskas wrote:
> [...]
> > > http://samples.mplayerhq.hu/V-codecs/VP5/vp5_artefacts.nsv
> > > is actually not NSV as is,it's actually NSV in ultravox bitstream
> > > ( http://ultravox.aol.com/ )
> > 
> > Attached patch adds support for NSV in ultravox bitstream.
> > It plays vp5_artefacts.nsv perfectly.
> 
> MPlayer implements it in the stream layer, but that's only for http.
> IMO it does not belong into the demuxer, it is a wrapper above the
> demuxer layer really, so something like a stream filter would be more
> appropriate.

Indeed, it's supposed to be a network protocol. Maybe it could be
implemented as a URLProtocol.
But on the other hand, it's also used directly in some files.
And it seems to be used only for NSV. So writting a more generic
URLProtocol or something wouldn't be much more useful than
de-warping inside the NSV demuxer. This also won't incite people
to use this ugly wrapping with anything else than NSV.

> On the other hand that makes it hard to actually make use of the
> metadata later on...

This is one more good reason to keep de-warping inside the NSV demuxer.

IOW: keep the hack as close as possible to what needs a hack.

Anyone against my patch ?

Aurel




More information about the ffmpeg-devel mailing list