Re: [FFmpeg-devel] [PATCH 01/17] lavfi: add ff_inlink_acknowledge_status().

2017-01-12 Thread wm4
On Thu, 12 Jan 2017 15:04:04 +0100 Nicolas George wrote: > Le tridi 23 nivôse, an CCXXV, wm4 a écrit : > > Well then why didn't you point these out when I asked? Oh I know, these > > comments didn't really give a coherent overview. > > You do not know that, you only read the patches looking fo

Re: [FFmpeg-devel] [PATCH 01/17] lavfi: add ff_inlink_acknowledge_status().

2017-01-12 Thread Nicolas George
Le tridi 23 nivôse, an CCXXV, wm4 a écrit : > Well then why didn't you point these out when I asked? Oh I know, these > comments didn't really give a coherent overview. You do not know that, you only read the patches looking for ways to criticize and nag me. Never to do constructive comments. > I

Re: [FFmpeg-devel] [PATCH 01/17] lavfi: add ff_inlink_acknowledge_status().

2017-01-12 Thread wm4
On Thu, 12 Jan 2017 14:49:51 +0100 Nicolas George wrote: > Le tridi 23 nivôse, an CCXXV, wm4 a écrit : > > In my opinion, such an overview was necessary along such a huge > > change - if only to make sure reviewers can properly review the new > > API before giving their ok. > > That "overview"

Re: [FFmpeg-devel] [PATCH 01/17] lavfi: add ff_inlink_acknowledge_status().

2017-01-12 Thread Nicolas George
Le tridi 23 nivôse, an CCXXV, wm4 a écrit : > In my opinion, such an overview was necessary along such a huge > change - if only to make sure reviewers can properly review the new > API before giving their ok. That "overview" is already present on the mailing-list and in the comments of the patche

Re: [FFmpeg-devel] [PATCH 01/17] lavfi: add ff_inlink_acknowledge_status().

2017-01-12 Thread wm4
On Thu, 12 Jan 2017 14:22:56 +0100 Nicolas George wrote: > Le tridi 23 nivôse, an CCXXV, wm4 a écrit : > > Do you have any explanation as to why you pushed this patchset without > > doing what was requested? > > What do you mean exactly? > In my opinion, such an overview was necessary along

Re: [FFmpeg-devel] [PATCH 01/17] lavfi: add ff_inlink_acknowledge_status().

2017-01-12 Thread Nicolas George
Le tridi 23 nivôse, an CCXXV, wm4 a écrit : > Do you have any explanation as to why you pushed this patchset without > doing what was requested? What do you mean exactly? -- Nicolas George signature.asc Description: Digital signature ___ ffmpeg-dev

Re: [FFmpeg-devel] [PATCH 01/17] lavfi: add ff_inlink_acknowledge_status().

2017-01-12 Thread wm4
On Mon, 9 Jan 2017 11:48:50 +0100 Nicolas George wrote: > Le decadi 20 nivôse, an CCXXV, wm4 a écrit : > > Can we have a document in doc/ or a doxygen section that gives an > > overview how the "new" internal API works? > > I already stated several times this was planned. Do you have any expl

Re: [FFmpeg-devel] [PATCH 01/17] lavfi: add ff_inlink_acknowledge_status().

2017-01-09 Thread Nicolas George
Le decadi 20 nivôse, an CCXXV, wm4 a écrit : > Can we have a document in doc/ or a doxygen section that gives an > overview how the "new" internal API works? I already stated several times this was planned. BTW, « API-vomit » is not acceptable IMHO. -- Nicolas George _

Re: [FFmpeg-devel] [PATCH 01/17] lavfi: add ff_inlink_acknowledge_status().

2017-01-09 Thread wm4
On Thu, 29 Dec 2016 15:33:47 +0100 Nicolas George wrote: > Also introduce libavfilter/filters.h for all functions needed > to implement filters. > > Signed-off-by: Nicolas George Can we have a document in doc/ or a doxygen section that gives an overview how the "new" internal API works? __

Re: [FFmpeg-devel] [PATCH 01/17] lavfi: add ff_inlink_acknowledge_status().

2016-12-29 Thread Michael Niedermayer
On Thu, Dec 29, 2016 at 03:33:47PM +0100, Nicolas George wrote: > Also introduce libavfilter/filters.h for all functions needed > to implement filters. > > Signed-off-by: Nicolas George > --- > libavfilter/avfilter.c | 14 ++ > libavfilter/filters.h | 37

[FFmpeg-devel] [PATCH 01/17] lavfi: add ff_inlink_acknowledge_status().

2016-12-29 Thread Nicolas George
Also introduce libavfilter/filters.h for all functions needed to implement filters. Signed-off-by: Nicolas George --- libavfilter/avfilter.c | 14 ++ libavfilter/filters.h | 37 + 2 files changed, 51 insertions(+) create mode 100644 libavfilter/f