Re: [FFmpeg-devel] [PATCH] avfilter: add metadata filters

2016-02-10 Thread Tobias Rapp
On 10.02.2016 11:40, Paul B Mahol wrote: On 2/10/16, Tobias Rapp wrote: On 10.02.2016 10:59, Paul B Mahol wrote: On 2/10/16, Tobias Rapp wrote: On 10.02.2016 10:01, Paul B Mahol wrote: On 2/10/16, Tobias Rapp wrote: On 06.02.2016 23:30, Paul B Mahol wrote: On 2/6/16, Paul B Mahol wrote:

Re: [FFmpeg-devel] [PATCH] avfilter: add metadata filters

2016-02-10 Thread Paul B Mahol
On 2/10/16, Tobias Rapp wrote: > On 10.02.2016 10:59, Paul B Mahol wrote: >> On 2/10/16, Tobias Rapp wrote: >>> On 10.02.2016 10:01, Paul B Mahol wrote: On 2/10/16, Tobias Rapp wrote: > On 06.02.2016 23:30, Paul B Mahol wrote: >> On 2/6/16, Paul B Mahol wrote: >>> Hi, >>> >

Re: [FFmpeg-devel] [PATCH] avfilter: add metadata filters

2016-02-10 Thread Tobias Rapp
On 10.02.2016 10:59, Paul B Mahol wrote: On 2/10/16, Tobias Rapp wrote: On 10.02.2016 10:01, Paul B Mahol wrote: On 2/10/16, Tobias Rapp wrote: On 06.02.2016 23:30, Paul B Mahol wrote: On 2/6/16, Paul B Mahol wrote: Hi, patch attached. Improved version attached. [...] + +static int s

Re: [FFmpeg-devel] [PATCH] avfilter: add metadata filters

2016-02-10 Thread Paul B Mahol
On 2/10/16, Tobias Rapp wrote: > On 10.02.2016 10:01, Paul B Mahol wrote: >> On 2/10/16, Tobias Rapp wrote: >>> On 06.02.2016 23:30, Paul B Mahol wrote: On 2/6/16, Paul B Mahol wrote: > Hi, > > patch attached. > Improved version attached. [...] + >>>

Re: [FFmpeg-devel] [PATCH] avfilter: add metadata filters

2016-02-10 Thread Tobias Rapp
On 10.02.2016 10:01, Paul B Mahol wrote: On 2/10/16, Tobias Rapp wrote: On 06.02.2016 23:30, Paul B Mahol wrote: On 2/6/16, Paul B Mahol wrote: Hi, patch attached. Improved version attached. [...] + +static int string(const char *value1, const char *value2, size_t length) +{ +return

Re: [FFmpeg-devel] [PATCH] avfilter: add metadata filters

2016-02-10 Thread Paul B Mahol
On 2/10/16, Tobias Rapp wrote: > On 06.02.2016 23:30, Paul B Mahol wrote: >> On 2/6/16, Paul B Mahol wrote: >>> Hi, >>> >>> patch attached. >>> >> >> Improved version attached. >> >> [...] >> + >> +static int string(const char *value1, const char *value2, size_t length) >> +{ >> +return !strn

Re: [FFmpeg-devel] [PATCH] avfilter: add metadata filters

2016-02-10 Thread Tobias Rapp
On 06.02.2016 23:30, Paul B Mahol wrote: On 2/6/16, Paul B Mahol wrote: Hi, patch attached. Improved version attached. [...] + +static int string(const char *value1, const char *value2, size_t length) +{ +return !strncmp(value1, value2, length); +} If I understand correctly this func

Re: [FFmpeg-devel] [PATCH] avfilter: add metadata filters

2016-02-09 Thread Paul B Mahol
On 2/6/16, Paul B Mahol wrote: > On 2/6/16, Paul B Mahol wrote: >> Hi, >> >> patch attached. >> > > Improved version attached. > Gonna push this soon. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] avfilter: add metadata filters

2016-02-06 Thread Paul B Mahol
On 2/6/16, Paul B Mahol wrote: > Hi, > > patch attached. > Improved version attached. From 46283dd6b38ce8f1346249707986212546026e39 Mon Sep 17 00:00:00 2001 From: Paul B Mahol Date: Sat, 6 Feb 2016 11:19:45 +0100 Subject: [PATCH] avfilter: add metadata filters Signed-off-by: Paul B Mahol ---

Re: [FFmpeg-devel] [PATCH] avfilter: add metadata filters

2016-02-06 Thread Paul B Mahol
On 2/6/16, Moritz Barsnick wrote: > On Sat, Feb 06, 2016 at 12:47:25 +0100, Paul B Mahol wrote: >> patch attached. > > Nice. (Untested.) > > > >> +Manipulate frames metadata. > Either: > frame metadata [using this as a generic term] > or: > frames' metadata > > I prefer the former. > >

Re: [FFmpeg-devel] [PATCH] avfilter: add metadata filters

2016-02-06 Thread Moritz Barsnick
On Sat, Feb 06, 2016 at 12:47:25 +0100, Paul B Mahol wrote: > patch attached. Nice. (Untested.) > +Manipulate frames metadata. Either: frame metadata [using this as a generic term] or: frames' metadata I prefer the former. > +If both @code{value} and @code{key} is set, select fram