On 23 Feb 2014, at 19:06, Janne Grunau <[email protected]> wrote:
> 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 Sorry, I looked at the wrong git repo. Tim _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
