Re: [FFmpeg-devel] [PATCH] avfilter: slice processing for geq

2018-01-04 Thread Michael Niedermayer
On Thu, Jan 04, 2018 at 07:52:17AM +, Marc-Antoine ARNAUD wrote: > Sorry, I have upgrade and re-generate patches. will apply thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB The real ebay dictionary, page 1 "Used only once"- "Some unspecified

Re: [FFmpeg-devel] [PATCH] avfilter: slice processing for geq

2018-01-03 Thread Marc-Antoine ARNAUD
Sorry, I have upgrade and re-generate patches. Le mer. 3 janv. 2018 à 23:47, Moritz Barsnick a écrit : > On Wed, Jan 03, 2018 at 12:51:06 +, Marc-Antoine ARNAUD wrote: > > Changelog | 10 + > > MAINTAINERS |

Re: [FFmpeg-devel] [PATCH] avfilter: slice processing for geq

2018-01-03 Thread Moritz Barsnick
On Wed, Jan 03, 2018 at 12:51:06 +, Marc-Antoine ARNAUD wrote: > Changelog | 10 + > MAINTAINERS |9 +- > Makefile|9 +- > compat/cuda/dynlink_loader.h| 18 +- >

Re: [FFmpeg-devel] [PATCH] avfilter: slice processing for geq

2017-11-30 Thread Michael Niedermayer
On Thu, Nov 30, 2017 at 01:35:52PM +, Marc-Antoine ARNAUD wrote: > Le jeu. 30 nov. 2017 à 01:51, Michael Niedermayer > a écrit : > > > On Wed, Nov 29, 2017 at 11:28:40AM +, Marc-Antoine ARNAUD wrote: > > > Le mer. 22 nov. 2017 à 17:54, Michael Niedermayer

Re: [FFmpeg-devel] [PATCH] avfilter: slice processing for geq

2017-11-30 Thread Marc-Antoine ARNAUD
Le jeu. 30 nov. 2017 à 01:51, Michael Niedermayer a écrit : > On Wed, Nov 29, 2017 at 11:28:40AM +, Marc-Antoine ARNAUD wrote: > > Le mer. 22 nov. 2017 à 17:54, Michael Niedermayer > > > a écrit : > > > > > On Wed, Nov 22, 2017 at 10:24:30AM

Re: [FFmpeg-devel] [PATCH] avfilter: slice processing for geq

2017-11-29 Thread Michael Niedermayer
On Wed, Nov 29, 2017 at 11:28:40AM +, Marc-Antoine ARNAUD wrote: > Le mer. 22 nov. 2017 à 17:54, Michael Niedermayer > a écrit : > > > On Wed, Nov 22, 2017 at 10:24:30AM +, Marc-Antoine ARNAUD wrote: > > > New patch version which fixe the last remark. > > > > > >

Re: [FFmpeg-devel] [PATCH] avfilter: slice processing for geq

2017-11-29 Thread Marc-Antoine ARNAUD
Le mer. 22 nov. 2017 à 17:54, Michael Niedermayer a écrit : > On Wed, Nov 22, 2017 at 10:24:30AM +, Marc-Antoine ARNAUD wrote: > > New patch version which fixe the last remark. > > > > > > Le ven. 10 nov. 2017 à 00:47, Michael Niedermayer > >

Re: [FFmpeg-devel] [PATCH] avfilter: slice processing for geq

2017-11-22 Thread Michael Niedermayer
On Wed, Nov 22, 2017 at 10:24:30AM +, Marc-Antoine ARNAUD wrote: > New patch version which fixe the last remark. > > > Le ven. 10 nov. 2017 à 00:47, Michael Niedermayer > a écrit : > > > On Thu, Nov 09, 2017 at 10:22:23AM +, Marc-Antoine ARNAUD wrote: > > >

Re: [FFmpeg-devel] [PATCH] avfilter: slice processing for geq

2017-11-22 Thread Derek Buitenhuis
On 11/22/2017 2:38 PM, Marc-Antoine ARNAUD wrote: > It made sense, but this commit don't touch this part of the code. > It can made more sense to add an another path to "prevent bad memory > allocation in geq filter". What do you think ? Ah, you are correct. A 2nd patch would be nice, yes, but

Re: [FFmpeg-devel] [PATCH] avfilter: slice processing for geq

2017-11-22 Thread Marc-Antoine ARNAUD
It made sense, but this commit don't touch this part of the code. It can made more sense to add an another path to "prevent bad memory allocation in geq filter". What do you think ? Regards, Le mer. 22 nov. 2017 à 15:19, Derek Buitenhuis a écrit : > On 11/22/2017

Re: [FFmpeg-devel] [PATCH] avfilter: slice processing for geq

2017-11-22 Thread Derek Buitenhuis
On 11/22/2017 10:24 AM, Marc-Antoine ARNAUD wrote: > New patch version which fixe the last remark. [...] > +char depth_string[8]; > +snprintf(depth_string, sizeof(depth_string), "%d", (1 1); > +geq->expr_str[A] = av_strdup(depth_string); Missing return value

Re: [FFmpeg-devel] [PATCH] avfilter: slice processing for geq

2017-11-22 Thread Marc-Antoine ARNAUD
New patch version which fixe the last remark. Le ven. 10 nov. 2017 à 00:47, Michael Niedermayer a écrit : > On Thu, Nov 09, 2017 at 10:22:23AM +, Marc-Antoine ARNAUD wrote: > > Please find the merged patch in attachement. > > > > Thanks > > > > Le mer. 8 nov. 2017 à

Re: [FFmpeg-devel] [PATCH] avfilter: slice processing for geq

2017-11-09 Thread Michael Niedermayer
On Thu, Nov 09, 2017 at 10:22:23AM +, Marc-Antoine ARNAUD wrote: > Please find the merged patch in attachement. > > Thanks > > Le mer. 8 nov. 2017 à 17:12, Paul B Mahol a écrit : > > > On 11/8/17, Marc-Antoine ARNAUD wrote: > > > This patch

Re: [FFmpeg-devel] [PATCH] avfilter: slice processing for geq

2017-11-09 Thread Marc-Antoine ARNAUD
Please find the merged patch in attachement. Thanks Le mer. 8 nov. 2017 à 17:12, Paul B Mahol a écrit : > On 11/8/17, Marc-Antoine ARNAUD wrote: > > This patch will fix the stride issue. > > Is it valid for you ? > > > > Does it required to

Re: [FFmpeg-devel] [PATCH] avfilter: slice processing for geq

2017-11-08 Thread Paul B Mahol
On 11/8/17, Marc-Antoine ARNAUD wrote: > This patch will fix the stride issue. > Is it valid for you ? > > Does it required to merge these 2 patches ? (and remove base64 encoding on > the first one) Please merge those two patches, base64 encoding should not be

Re: [FFmpeg-devel] [PATCH] avfilter: slice processing for geq

2017-11-08 Thread Marc-Antoine ARNAUD
This patch will fix the stride issue. Is it valid for you ? Does it required to merge these 2 patches ? (and remove base64 encoding on the first one) Thx Marc-Antoine Le ven. 3 nov. 2017 à 03:13, Michael Niedermayer a écrit : > On Thu, Nov 02, 2017 at 02:04:33PM +,

Re: [FFmpeg-devel] [PATCH] avfilter: slice processing for geq

2017-11-02 Thread Michael Niedermayer
On Thu, Nov 02, 2017 at 02:04:33PM +, Marc-Antoine ARNAUD wrote: > > vf_geq.c | 126 > --- > 1 file changed, 89 insertions(+), 37 deletions(-) > b0379f3d7f1d9660e209fe491f48fd7f70113615 >