On 2014-02-23 18:53:32 +0100, Tim Walker wrote:
> 
> On 23 Feb 2014, at 18:40, Janne Grunau <[email protected]> wrote:
> 
> > On 2014-02-23 17:21:10 +0000, Derek Buitenhuis wrote:
> >> On 2/23/2014 3:20 PM, Janne Grunau wrote:
> >>> This doesn't look right, AV_PIX_FMT_YUV420P is a valid pixel format for
> >>> 'x265_max_bit_depth == 12' and I can't see how x265 could deduce that
> >>> otherwise.
> >> 
> >> No, the *internal* depth is not the same as *input* depth. The input depth
> >> is set in the picture struct, and is separate.
> > 
> > I fail to see there the input depth is set unless x265_encoder_encode()
> > deduces that automagically from stride and width.
> > 
> > Janne
> 
> Already in git:
> 
> x265pic.bitDepth = av_pix_fmt_desc_get(avctx->pix_fmt)->comp[0].depth_minus1 
> + 1;

not in git but in the v2 patch I missed/ignored, sorry for the noise

Janne
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to