Re: [FFmpeg-devel] [PATCH] avcodec/wcmv: use ff_reget_buffer()

2019-09-04 Thread James Almer
On 8/30/2019 3:26 AM, Paul B Mahol wrote: > It still exits early when no blocks are encoded. Reverted that and pushed. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link abov

Re: [FFmpeg-devel] [PATCH] avcodec/wcmv: use ff_reget_buffer()

2019-08-30 Thread Paul B Mahol
On 8/30/19, James Almer wrote: > On 8/30/2019 3:45 AM, Paul B Mahol wrote: >> Here is sample: https://0x0.st/z49v.avi > > Thanks. Can you make a fate test for it? Here is much smaller file. https://0x0.st/z4f5.avi > >> >> On 8/30/19, Paul B Mahol wrote: >>> It still exits early when no blocks

Re: [FFmpeg-devel] [PATCH] avcodec/wcmv: use ff_reget_buffer()

2019-08-30 Thread James Almer
On 8/30/2019 3:45 AM, Paul B Mahol wrote: > Here is sample: https://0x0.st/z49v.avi Thanks. Can you make a fate test for it? > > On 8/30/19, Paul B Mahol wrote: >> It still exits early when no blocks are encoded. >> >> On 8/30/19, James Almer wrote: >>> Signed-off-by: James Almer >>> --- >>>

Re: [FFmpeg-devel] [PATCH] avcodec/wcmv: use ff_reget_buffer()

2019-08-30 Thread James Almer
On 8/30/2019 3:26 AM, Paul B Mahol wrote: > It still exits early when no blocks are encoded. That's unrelated to this patch. I can look into reverting 976dae8b32 and preventing data copying when the frame doesn't change in a separate patchset. > > On 8/30/19, James Almer wrote: >> Signed-off-by

Re: [FFmpeg-devel] [PATCH] avcodec/wcmv: use ff_reget_buffer()

2019-08-29 Thread Paul B Mahol
Here is sample: https://0x0.st/z49v.avi On 8/30/19, Paul B Mahol wrote: > It still exits early when no blocks are encoded. > > On 8/30/19, James Almer wrote: >> Signed-off-by: James Almer >> --- >> Untested. There doesn't seem to be a fate test, and i have no samples >> at hand. >> >> libavcod

Re: [FFmpeg-devel] [PATCH] avcodec/wcmv: use ff_reget_buffer()

2019-08-29 Thread Paul B Mahol
It still exits early when no blocks are encoded. On 8/30/19, James Almer wrote: > Signed-off-by: James Almer > --- > Untested. There doesn't seem to be a fate test, and i have no samples > at hand. > > libavcodec/wcmv.c | 23 +-- > 1 file changed, 9 insertions(+), 14 deletio

[FFmpeg-devel] [PATCH] avcodec/wcmv: use ff_reget_buffer()

2019-08-29 Thread James Almer
Signed-off-by: James Almer --- Untested. There doesn't seem to be a fate test, and i have no samples at hand. libavcodec/wcmv.c | 23 +-- 1 file changed, 9 insertions(+), 14 deletions(-) diff --git a/libavcodec/wcmv.c b/libavcodec/wcmv.c index 0d60b9fe1f..68142b45fd 100644 -