Re: [FFmpeg-devel] [PATCH 1/8] avutil/opt: add AV_OPT_FLAG_COMMAND_PARAM flag

2019-10-09 Thread Michael Niedermayer
On Wed, Oct 09, 2019 at 11:24:55AM +0200, Paul B Mahol wrote: > Because this receives no comments so fat at all, will apply asap this set. I think you should be a bit more patient, threatening to apply after only about 24h. Just causes people who have an interrest in the code but cannot

Re: [FFmpeg-devel] [PATCH 1/8] avutil/opt: add AV_OPT_FLAG_COMMAND_PARAM flag

2019-10-09 Thread Paul B Mahol
On 10/9/19, Nicolas George wrote: > Paul B Mahol (12019-10-09): >> Such interpretation is not in documentation. > >> Such line is not in MAINTAINERS. > >> This is violent attack. > >> I do not believe in your good faith regarding me ever. > > I will not argue further with you. You have been

Re: [FFmpeg-devel] [PATCH 1/8] avutil/opt: add AV_OPT_FLAG_COMMAND_PARAM flag

2019-10-09 Thread Nicolas George
Paul B Mahol (12019-10-09): > Such interpretation is not in documentation. > Such line is not in MAINTAINERS. > This is violent attack. > I do not believe in your good faith regarding me ever. I will not argue further with you. You have been warned. Regards, -- Nicolas George

Re: [FFmpeg-devel] [PATCH 1/8] avutil/opt: add AV_OPT_FLAG_COMMAND_PARAM flag

2019-10-09 Thread Paul B Mahol
On 10/9/19, Nicolas George wrote: > Paul B Mahol (12019-10-09): >> This is small patch > > Not when it touches public API. Such interpretation is not in documentation. > > Anyway, I maintain lavfi, and I tell you not to commit before I > reviewed. Such line is not in MAINTAINERS. > > Break

Re: [FFmpeg-devel] [PATCH 1/8] avutil/opt: add AV_OPT_FLAG_COMMAND_PARAM flag

2019-10-09 Thread Nicolas George
Paul B Mahol (12019-10-09): > This is small patch Not when it touches public API. Anyway, I maintain lavfi, and I tell you not to commit before I reviewed. Break the rules and I will make sure that there are consequences. By the way, I had some time this morning, and I had considered reviewing

Re: [FFmpeg-devel] [PATCH 1/8] avutil/opt: add AV_OPT_FLAG_COMMAND_PARAM flag

2019-10-09 Thread Paul B Mahol
On 10/9/19, Nicolas George wrote: > Paul B Mahol (12019-10-09): >> This only adds single flag. > > A flag that belongs in the public API, and therefore that we will have > to support for a long time. Hence the need to make sure it is properly > designed. > >> Out of question. Your reply with all

Re: [FFmpeg-devel] [PATCH 1/8] avutil/opt: add AV_OPT_FLAG_COMMAND_PARAM flag

2019-10-09 Thread Nicolas George
Paul B Mahol (12019-10-09): > This only adds single flag. A flag that belongs in the public API, and therefore that we will have to support for a long time. Hence the need to make sure it is properly designed. > Out of question. Your reply with all due respect is ignored. This is not respect,

Re: [FFmpeg-devel] [PATCH 1/8] avutil/opt: add AV_OPT_FLAG_COMMAND_PARAM flag

2019-10-09 Thread Paul B Mahol
On 10/9/19, Nicolas George wrote: > Paul B Mahol (12019-10-09): >> Because this receives no comments so fat at all, will apply asap this >> set. > > Less than 24 hours for something that changes the public API? Out of the > question. This only adds single flag. > > If in one week you had no

Re: [FFmpeg-devel] [PATCH 1/8] avutil/opt: add AV_OPT_FLAG_COMMAND_PARAM flag

2019-10-09 Thread Nicolas George
Paul B Mahol (12019-10-09): > Because this receives no comments so fat at all, will apply asap this set. Less than 24 hours for something that changes the public API? Out of the question. If in one week you had no comments, you can ask again that people look into it. Regards, -- Nicolas

Re: [FFmpeg-devel] [PATCH 1/8] avutil/opt: add AV_OPT_FLAG_COMMAND_PARAM flag

2019-10-09 Thread Paul B Mahol
Because this receives no comments so fat at all, will apply asap this set. On 10/8/19, Paul B Mahol wrote: > Signed-off-by: Paul B Mahol > --- > libavutil/opt.h | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/libavutil/opt.h b/libavutil/opt.h > index 39f4a8dda0..693a170bcc 100644 >