On 12/02/15 14:31, Hendrik Leppkes wrote: > On Thu, Feb 12, 2015 at 10:54 AM, Anton Khirnov <[email protected]> wrote: >> --- >> avconv.c | 1 - >> 1 file changed, 1 deletion(-) >> >> diff --git a/avconv.c b/avconv.c >> index 8e76863..6349cae 100644 >> --- a/avconv.c >> +++ b/avconv.c >> @@ -1276,7 +1276,6 @@ static int decode_video(InputStream *ist, AVPacket >> *pkt, int *got_output) >> >> decoded_frame->pts = guess_correct_pts(&ist->pts_ctx, >> decoded_frame->pkt_pts, >> decoded_frame->pkt_dts); >> - pkt->size = 0; >> >> if (ist->st->sample_aspect_ratio.num) >> decoded_frame->sample_aspect_ratio = ist->st->sample_aspect_ratio; > > > While this change itself is not wrong, I would like to question the > requirement for it. > All video decoders to-date can get by without this, and it greatly > simplifies the API - plus practically no implementation will support > it. > > So, if possible at all, can the mfx code handle this internally and we > not start requiring this handling for the video decode API as well? >
Adding a queue inside it as per the previous iteration is an option, not sure why we should queue up instead of calling more than once. lu _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
