On Sat, Jun 4, 2016 at 7:28 AM, Luca Barbato <[email protected]> wrote: > On 04/06/16 13:22, [email protected] wrote: >> From: typx <[email protected]> >> >> According to avcodec.h, avcodec_decode_video2 should return the number of >> bytes used if a frame was decoded. >> The current implementation returns size - used size of all the subframes. >> This fixes the VLC's bug https://trac.videolan.org/vlc/ticket/16836. >> --- >> libavcodec/vp9.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/libavcodec/vp9.c b/libavcodec/vp9.c >> index 45f0771..c11e9b8 100644 >> --- a/libavcodec/vp9.c >> +++ b/libavcodec/vp9.c >> @@ -1251,7 +1251,7 @@ static int vp9_decode_packet(AVCodecContext *avctx, >> void *frame, >> data += sz; >> size -= sz; >> } >> - return size; >> + return avpkt->size; >> } >> } >> >> > > The packet is completely consumed there, so sure, looks fine for me.
What about on line 1262? Is a "return size;" enough? -- Vittorio _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
