[FFmpeg-devel] [PATCH] v210 decoder

[Michael Niedermayer]

On Tue, Feb 17, 2009 at 08:45:22PM +0100, Reimar D?ffinger wrote:
> On Tue, Feb 17, 2009 at 08:05:38PM +0100, Michael Niedermayer wrote:
> > On Tue, Feb 17, 2009 at 07:38:03PM +0100, Reimar D?ffinger wrote:
> > > On Tue, Feb 17, 2009 at 06:53:08PM +0100, Michael Niedermayer wrote:
> > > > On Tue, Feb 17, 2009 at 06:26:01PM +0100, Reimar D?ffinger wrote:
> > > > > this is a decoder for
> > > > > http://samples.mplayerhq.hu/V-codecs/v210/v210_720p.avi
> > > > > it is not actually a proper codec, it is just little-endian
> > > > > UYVY with 10 bits per component and aligned to 32 bit every 3
> > > > > components.
> > > > > The last part makes it a bit awkward to deal with, which is also the
> > > > > reason why a favour a decoder instead of adding a pixfmt, IMHO if
> > > > > we want to support 10-bit pixfmts the planar or 444 variants would
> > > > > be enough and less painful to do...
> > > > 
> > > > I can accept codecs instead of pixfmts to deal with obscure packing
> > > > of pixels.
> > > > But i dont plan to accept code that always truncates 10 bit to 8
> > > 
> > > Well, from my point of view that's silly because hardly anyone cares the
> > > slightest about those extra 2 bits and thus I expect nobody in the foreseeable
> > > future will write pixfmt support for them (I just see someone in 2007
> > > gave up at that point), but I don't really care, though I do find it a
> > 
> > why dont you move your 10->8 convertion into swscale? that still then
> > allows transcoding to other containers supporting v210 ...
> 
> I will consider it the moment someone points me to a nice step-by-step
> tutorial to adding stuff to swscale, or makes it similarly easy to add a

ok here is the step by step tutorial
svn di -r20426:r20427
the hunks 3 and 5 you dont need, they are optional special converters
also the change to isSupportedOut() you dont need
above will add a new input format

also another example for adding an input format
svn di -r20604:r20605


[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I hate to see young programmers poisoned by the kind of thinking
Ulrich Drepper puts forward since it is simply too narrow -- Roman Shaposhnik
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090218/b0a1a292/attachment.pgp>