[FFmpeg-devel] v210 decoder patch

[Michael Niedermayer]

Hi

On Tue, Jul 17, 2007 at 02:26:25PM -0400, Francois Oligny-Lemieux wrote:
> Hi,
> 
> I came across a HD .avi file that I couldn't play back using usual software
> (VLC, ffmpeg, Quicktime), so I created a function in ffmpeg to convert this
> v210 pixel format to YUV422.
> I'm trying to submit this as a patch so ffmpeg can read these HD AVI files
> with riff == "v210".
> 
> Let me know if you like this patch and if there is something wrong with it.

* yes, if you add a PIX_FMT then you must add support to swscale to convert to
  and from that
* convertion in a decoder must not loose information, so convertion to 8bit
  is not ok
* ideally someone should add generic pix_fmt support to swscale so that any
  packing of bits into 1-3 planes can be read and written with a generic
  routine and we dont need to add a new converter for each format, see
  the mailinglist for this, there where some disscussions about it but
  sadly patches drifted off into the direction of a bloated API full of
  stuff unrelated to what was actually needed

so your options are either
A. continue where the generic PIX_FMT support got stuck and then just add
   2-5 lines of code to make lavc magically support v210
B. add v210 support to swscale, add a new PIX_FMT, update the raw decoder&
   demuxer

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

It is dangerous to be right in matters on which the established authorities
are wrong. -- Voltaire
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070717/fdd13e24/attachment.pgp>