[FFmpeg-devel] [PATCH]Support more tiff variants in mov

[Michael Niedermayer]

On Wed, Apr 18, 2012 at 01:07:24PM +0000, Carl Eugen Hoyos wrote:
> Michael Niedermayer <michaelni <at> gmx.at> writes:
> 
> > > +    avctx->bits_per_coded_sample = 
> > > av_get_bits_per_pixel(&av_pix_fmt_descriptors[avctx->pix_fmt]);
> > 
> > is this correct for the yuv formats too ?
> 
> It is only known to be correct for PAL8 (which does not work 
> correctly in mov), RGB24 (which is default anyway), RGBA and 
> monob, gray needs a special value (0x28), RGB48 and RGB64 
> are reported not to work.
> 
> Is it better to set bits_per_coded_sample explicitely and 
> only for the variants that are known to work with QuickTime?

i would suggest to set it to the most correct values known and leave
it 0 for the rest

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The real ebay dictionary, page 3
"Rare item" - "Common item with rare defect or maybe just a lie"
"Professional" - "'Toy' made in china, not functional except as doorstop"
"Experts will know" - "The seller hopes you are not an expert"
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20120418/30e087a7/attachment.asc>