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? - Hendrik _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
