Re: [FFmpeg-devel] [PATCH v4] avcodec: Cineform HD Decoder

2016-01-28 Thread Michael Niedermayer
On Thu, Jan 28, 2016 at 08:53:00PM +, Kieran Kunhya wrote: > Decodes YUV 4:2:2 10-bit and RGB 12-bit files. > Older files with more subbands, skips, Bayer, alpha not supported. > Alpha requires addition of GBRAP12 pixel format. [...] > +static inline int dequant_and_decompand(int level, int qu

Re: [FFmpeg-devel] [PATCH v4] avcodec: Cineform HD Decoder

2016-01-28 Thread Moritz Barsnick
Cosmetic nerd nit: > +av_log(avctx, AV_LOG_DEBUG, "large len %x \n", ((tagu & 0xff) << > 16) | data); > +} else if (tag == 20) { > +av_log(avctx, AV_LOG_DEBUG, "Width %"PRIu16" \n", data); ^ You're logging