On 3/30/2015 11:45 PM, Michael Niedermayer wrote: > pkt->pts = x265pic_out.pts; > pkt->dts = x265pic_out.dts; > + switch (x265pic_out.sliceType) { > + case X265_TYPE_IDR: > + case X265_TYPE_I: avctx->coded_frame->pict_type = AV_PICTURE_TYPE_I; > break; > + case X265_TYPE_P: avctx->coded_frame->pict_type = AV_PICTURE_TYPE_P; > break; > + case X265_TYPE_B: avctx->coded_frame->pict_type = AV_PICTURE_TYPE_B; > break; > + }
LGTM if you format it like this (for consistency in the file): pkt->pts = x265pic_out.pts; pkt->dts = x265pic_out.dts; switch (x265pic_out.sliceType) { case X265_TYPE_I: avctx->coded_frame->pict_type = AV_PICTURE_TYPE_I; break; case X265_TYPE_P: avctx->coded_frame->pict_type = AV_PICTURE_TYPE_P; break; case X265_TYPE_B: avctx->coded_frame->pict_type = AV_PICTURE_TYPE_B; break; } Also I assume it is set to AV_PICTURE_TYPE_NONE by default? - Derek _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel