Re: [FFmpeg-devel] [PATCH] avcodec/librav1e: free the RaPacket after using it

2019-11-10 Thread James Almer
On 11/10/2019 10:21 PM, Derek Buitenhuis wrote: > On 11/11/2019 01:19, James Almer wrote: >> Fixes leaks. >> >> Signed-off-by: James Almer >> --- >> Missed this detail when reviewing. It's a remnant of the version where the >> RaPacket was attached to the AVPacket. > > Woops, yeah. I remember /th

Re: [FFmpeg-devel] [PATCH] avcodec/librav1e: free the RaPacket after using it

2019-11-10 Thread Derek Buitenhuis
On 11/11/2019 01:19, James Almer wrote: > Fixes leaks. > > Signed-off-by: James Almer > --- > Missed this detail when reviewing. It's a remnant of the version where the > RaPacket was attached to the AVPacket. Woops, yeah. I remember /thinking/ I needed too and then forgot. LGTM. - Derek _

[FFmpeg-devel] [PATCH] avcodec/librav1e: free the RaPacket after using it

2019-11-10 Thread James Almer
Fixes leaks. Signed-off-by: James Almer --- Missed this detail when reviewing. It's a remnant of the version where the RaPacket was attached to the AVPacket. libavcodec/librav1e.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/libavcodec/librav1e.c b/libavcodec/librav1e.c