Re: [FFmpeg-devel] [PATCH] avfilter/buffersrc: switch to activate

2023-11-13 Thread Paul B Mahol
On Mon, Nov 13, 2023 at 10:32 PM Nicolas George wrote: > Michael Niedermayer (12023-11-13): > > dear patch iam ashamed but iam not actually blocking you. > > What was blocking this patch is that the bug had not been properly > analyzed and a review process was in progress, slowed down by Paul's

Re: [FFmpeg-devel] [PATCH] avfilter/buffersrc: switch to activate

2023-11-13 Thread Nicolas George
Michael Niedermayer (12023-11-13): > dear patch iam ashamed but iam not actually blocking you. What was blocking this patch is that the bug had not been properly analyzed and a review process was in progress, slowed down by Paul's own refusal of providing the info his has (most likely because

Re: [FFmpeg-devel] [PATCH] avfilter/buffersrc: switch to activate

2023-11-13 Thread Paul B Mahol
On Mon, Nov 13, 2023 at 6:00 PM Michael Niedermayer wrote: > On Sun, Nov 12, 2023 at 09:19:57PM +0100, Paul B Mahol wrote: > > On Sun, Nov 12, 2023 at 5:37 PM Michael Niedermayer < > mich...@niedermayer.cc> > > wrote: > > > > > On Sun, Nov 12, 2023 at 11:02:49AM +0100, Nicolas George wrote: > >

Re: [FFmpeg-devel] [PATCH] avfilter/buffersrc: switch to activate

2023-11-13 Thread Zhao Zhili
> 在 2023年11月14日,上午1:45,Sean McGovern 写道: > > On Mon, Nov 13, 2023, 11:59 Michael Niedermayer > wrote: > >>> On Sun, Nov 12, 2023 at 09:19:57PM +0100, Paul B Mahol wrote: >>> On Sun, Nov 12, 2023 at 5:37 PM Michael Niedermayer < >> mich...@niedermayer.cc> >>> wrote: >>> On Sun, Nov 12,

Re: [FFmpeg-devel] [PATCH] avfilter/buffersrc: switch to activate

2023-11-13 Thread Sean McGovern
On Mon, Nov 13, 2023, 11:59 Michael Niedermayer wrote: > On Sun, Nov 12, 2023 at 09:19:57PM +0100, Paul B Mahol wrote: > > On Sun, Nov 12, 2023 at 5:37 PM Michael Niedermayer < > mich...@niedermayer.cc> > > wrote: > > > > > On Sun, Nov 12, 2023 at 11:02:49AM +0100, Nicolas George wrote: > > > >

Re: [FFmpeg-devel] [PATCH] avfilter/buffersrc: switch to activate

2023-11-13 Thread Michael Niedermayer
On Sun, Nov 12, 2023 at 09:19:57PM +0100, Paul B Mahol wrote: > On Sun, Nov 12, 2023 at 5:37 PM Michael Niedermayer > wrote: > > > On Sun, Nov 12, 2023 at 11:02:49AM +0100, Nicolas George wrote: > > > Vittorio Giovara (12023-11-11): > > > > Nice victimization, but "i'm behaving poorly because

Re: [FFmpeg-devel] [PATCH] avfilter/buffersrc: switch to activate

2023-11-12 Thread Paul B Mahol
On Sun, Nov 12, 2023 at 5:37 PM Michael Niedermayer wrote: > On Sun, Nov 12, 2023 at 11:02:49AM +0100, Nicolas George wrote: > > Vittorio Giovara (12023-11-11): > > > Nice victimization, but "i'm behaving poorly because the other person > is a > > > meanie" is a childish argument > > > > Mock

Re: [FFmpeg-devel] [PATCH] avfilter/buffersrc: switch to activate

2023-11-12 Thread Michael Niedermayer
On Sun, Nov 12, 2023 at 11:02:49AM +0100, Nicolas George wrote: > Vittorio Giovara (12023-11-11): > > Nice victimization, but "i'm behaving poorly because the other person is a > > meanie" is a childish argument > > Mock all your want, the fact that you attacked me even though I was just >

Re: [FFmpeg-devel] [PATCH] avfilter/buffersrc: switch to activate

2023-11-12 Thread Nicolas George
Vittorio Giovara (12023-11-11): > Nice victimization, but "i'm behaving poorly because the other person is a > meanie" is a childish argument Mock all your want, the fact that you attacked me even though I was just replying and not even told Paul to stop his childish games proves you are biassed

Re: [FFmpeg-devel] [PATCH] avfilter/buffersrc: switch to activate

2023-11-12 Thread Nicolas George
Paul B Mahol (12023-11-12): > So all this time you play arrogant moves and expect something out of > nothing but just posting cryptic text. “Out of nothing”? What does that even mean? It is a collaboration: you do not share because I want it but because it helps your own goal. > The case can be

Re: [FFmpeg-devel] [PATCH] avfilter/buffersrc: switch to activate

2023-11-11 Thread Paul B Mahol
On Sat, Nov 11, 2023 at 7:27 PM Nicolas George wrote: > Paul B Mahol (12023-11-11): > > I waited enough, applied, now for 100% true statement. > > Your teacher called, you forgot your pacifier in class yesterday. > > I am still expecting you to share the test cases you have so that the > bug can

Re: [FFmpeg-devel] [PATCH] avfilter/buffersrc: switch to activate

2023-11-11 Thread Vittorio Giovara
On Sat, Nov 11, 2023 at 2:04 PM Nicolas George wrote: > Vittorio Giovara (12023-11-11): > > I said don't reply to me! > > Fortunately you have no authority over me. > I said that to avoid ridiculing yourself any more > > I'm not biased, > > You are. In this message

Re: [FFmpeg-devel] [PATCH] avfilter/buffersrc: switch to activate

2023-11-11 Thread Nicolas George
Vittorio Giovara (12023-11-11): > I said don't reply to me! Fortunately you have no authority over me. > I'm not biased, You are. In this message again. > reread what you wrote and think > if "pacifier" belongs on a technical

Re: [FFmpeg-devel] [PATCH] avfilter/buffersrc: switch to activate

2023-11-11 Thread Vittorio Giovara
On Sat, Nov 11, 2023 at 1:42 PM Nicolas George wrote: > Vittorio Giovara (12023-11-11): > > No, bad Nicolas, stop this language, it's unacceptable and unbecoming. > You > > can ask for more time in a kinder manner. > > No, don't reply to me with insults or excuses, just take a walk outside > and

Re: [FFmpeg-devel] [PATCH] avfilter/buffersrc: switch to activate

2023-11-11 Thread Nicolas George
Vittorio Giovara (12023-11-11): > No, bad Nicolas, stop this language, it's unacceptable and unbecoming. You > can ask for more time in a kinder manner. > No, don't reply to me with insults or excuses, just take a walk outside and > reflect on your mistakes. Address your reproaches to the person

Re: [FFmpeg-devel] [PATCH] avfilter/buffersrc: switch to activate

2023-11-11 Thread Vittorio Giovara
On Sat, Nov 11, 2023 at 1:27 PM Nicolas George wrote: > Paul B Mahol (12023-11-11): > > I waited enough, applied, now for 100% true statement. > > Your teacher called, you forgot your pacifier in class yesterday > No, bad Nicolas, stop this language, it's unacceptable and unbecoming. You can

Re: [FFmpeg-devel] [PATCH] avfilter/buffersrc: switch to activate

2023-11-11 Thread Nicolas George
Paul B Mahol (12023-11-11): > I waited enough, applied, now for 100% true statement. Your teacher called, you forgot your pacifier in class yesterday. I am still expecting you to share the test cases you have so that the bug can be properly fixed. -- Nicolas George signature.asc

Re: [FFmpeg-devel] [PATCH] avfilter/buffersrc: switch to activate

2023-11-11 Thread Paul B Mahol
On Wed, Nov 8, 2023 at 7:22 PM Nicolas George wrote: > Paul B Mahol (12023-11-08): > > I did. I simplified it right from start. But I kept it for myself as It > is > > not really relevant. > > It is very relevant for reproducing the problem and reviewing the patch. > Please show what you have,

Re: [FFmpeg-devel] [PATCH] avfilter/buffersrc: switch to activate

2023-11-08 Thread Nicolas George
Paul B Mahol (12023-11-08): > I did. I simplified it right from start. But I kept it for myself as It is > not really relevant. It is very relevant for reproducing the problem and reviewing the patch. Please show what you have, it will save time. > With this pace of your debugging it can be

Re: [FFmpeg-devel] [PATCH] avfilter/buffersrc: switch to activate

2023-11-08 Thread Paul B Mahol
On Wed, Nov 8, 2023 at 5:53 PM Nicolas George wrote: > I have now been able to set up a reasonable test case with limit > memoryuse 600M. Next step is to simplify the test case. > > (I cannot understand how you can pretend you analyzed the bug when your > test case contains unrelated filters

Re: [FFmpeg-devel] [PATCH] avfilter/buffersrc: switch to activate

2023-11-08 Thread Nicolas George
I have now been able to set up a reasonable test case with limit memoryuse 600M. Next step is to simplify the test case. (I cannot understand how you can pretend you analyzed the bug when your test case contains unrelated filters like hqdn3d or setsar.) -- Nicolas George

Re: [FFmpeg-devel] [PATCH] avfilter/buffersrc: switch to activate

2023-11-07 Thread Nicolas George
Sean McGovern (12023-11-07): > This is really unfair to Paul to let this drag on for so long. What does fair have to do with this? If there is a bug, it has been there for years, it can be there for a few more months without any drawback. Plus, Paul has the patch that makes the bug disappear in

Re: [FFmpeg-devel] [PATCH] avfilter/buffersrc: switch to activate

2023-11-07 Thread Sean McGovern
Hi, On Tue, Nov 7, 2023, 16:50 Nicolas George wrote: > Paul B Mahol (12023-11-07): > > Will apply in next 48h. > > So, you are again threatening to violate the project's policy. > > > Libavfilter code is not so complex to need so big time interval for > review > > from experienced C developer.

Re: [FFmpeg-devel] [PATCH] avfilter/buffersrc: switch to activate

2023-11-07 Thread Nicolas George
Paul B Mahol (12023-11-07): > Will apply in next 48h. So, you are again threatening to violate the project's policy. > Libavfilter code is not so complex to need so big time interval for review > from experienced C developer. First, the issue is not only analyzing the code but first finding

Re: [FFmpeg-devel] [PATCH] avfilter/buffersrc: switch to activate

2023-11-07 Thread Paul B Mahol
Will apply in next 48h. Libavfilter code is not so complex to need so big time interval for review from experienced C developer. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe,

Re: [FFmpeg-devel] [PATCH] avfilter/buffersrc: switch to activate

2023-11-07 Thread Nicolas George
Vittorio Giovara (12023-11-07): > To better explain the case at hand, there is no developer policy that lets > you request an indefinite amount of review time with a single "No, you are > not" message. There is a policy that maintainers review patches and patches are not pushed when somebody

Re: [FFmpeg-devel] [PATCH] avfilter/buffersrc: switch to activate

2023-11-07 Thread Vittorio Giovara
On Tue, Nov 7, 2023 at 2:44 AM Nicolas George wrote: > Vittorio Giovara (12023-11-06): > > Correct, it's you trying to win arguments with aggressive communication > > No. You came into an argument without knowing the first thing about it > and you decided to side with the guy who wants to

Re: [FFmpeg-devel] [PATCH] avfilter/buffersrc: switch to activate

2023-11-06 Thread Nicolas George
Vittorio Giovara (12023-11-06): > Correct, it's you trying to win arguments with aggressive communication No. You came into an argument without knowing the first thing about it and you decided to side with the guy who wants to violate the project's policy about review. Your behavior in this

Re: [FFmpeg-devel] [PATCH] avfilter/buffersrc: switch to activate

2023-11-06 Thread Vittorio Giovara
On Mon, Nov 6, 2023 at 6:05 PM Nicolas George wrote: > Vittorio Giovara (12023-11-06): > > Stop twisting people's words and implying things they never said :) > > This is not you presenting excuses. > Correct, it's you trying to win arguments with aggressive communication > And thank you for

Re: [FFmpeg-devel] [PATCH] avfilter/buffersrc: switch to activate

2023-11-06 Thread Nicolas George
Vittorio Giovara (12023-11-06): > Stop twisting people's words and implying things they never said :) This is not you presenting excuses. > And thank you for ignoring my request to stay on topic There is no topic to stay on: I will review this bug when time permits, and you do not know anything

Re: [FFmpeg-devel] [PATCH] avfilter/buffersrc: switch to activate

2023-11-06 Thread Vittorio Giovara
On Mon, Nov 6, 2023 at 5:49 PM Nicolas George wrote: > Vittorio Giovara (12023-11-06): > > I'm not misrepresenting anybody's words, > > Me: “I never doubted that Paul's patches do make the bug go away in your > test case.” > > You: “you insisted it wasn't there” > > This is precisely

Re: [FFmpeg-devel] [PATCH] avfilter/buffersrc: switch to activate

2023-11-06 Thread Nicolas George
Vittorio Giovara (12023-11-06): > I'm not misrepresenting anybody's words, Me: “I never doubted that Paul's patches do make the bug go away in your test case.” You: “you insisted it wasn't there” This is precisely misrepresenting my words. So stop lying and present excuses. -- Nicolas

Re: [FFmpeg-devel] [PATCH] avfilter/buffersrc: switch to activate

2023-11-06 Thread Vittorio Giovara
On Mon, Nov 6, 2023 at 4:46 PM Nicolas George wrote: > Vittorio Giovara (12023-11-06): > > Nice try but pointing out your logical fallacies is not being rude, but > > rather is just showing you how easily your message can be misinterpreted, > > You have done nothing of the sort. You have just

Re: [FFmpeg-devel] [PATCH] avfilter/buffersrc: switch to activate

2023-11-06 Thread Nicolas George
Vittorio Giovara (12023-11-06): > Nice try but pointing out your logical fallacies is not being rude, but > rather is just showing you how easily your message can be misinterpreted, You have done nothing of the sort. You have just misrepresented my words, showing that you decided to barge into a

Re: [FFmpeg-devel] [PATCH] avfilter/buffersrc: switch to activate

2023-11-06 Thread Vittorio Giovara
On Mon, Nov 6, 2023 at 3:50 PM Nicolas George wrote: > Vittorio Giovara (12023-11-06): > > world, please make a minimum of effort to keep communication polite > > Please realize that politeness resides not only in which words we use, > like “bullshit”, but also in what we say, even if it is with

Re: [FFmpeg-devel] [PATCH] avfilter/buffersrc: switch to activate

2023-11-06 Thread Nicolas George
Vittorio Giovara (12023-11-06): > world, please make a minimum of effort to keep communication polite Please realize that politeness resides not only in which words we use, like “bullshit”, but also in what we say, even if it is with very mild words. For example: > while you insisted it wasn't

Re: [FFmpeg-devel] [PATCH] avfilter/buffersrc: switch to activate

2023-11-06 Thread Vittorio Giovara
On Mon, Nov 6, 2023 at 3:28 PM Nicolas George wrote: > Vittorio Giovara (12023-11-06): > > You gave no indication you were willing to, and I thought he gave you > time > > Bullshit. Please do not intervene in a thread when you do not know what > is going on at all. Paul has been repeatedly rude

Re: [FFmpeg-devel] [PATCH] avfilter/buffersrc: switch to activate

2023-11-06 Thread Nicolas George
Vittorio Giovara (12023-11-06): > You gave no indication you were willing to, and I thought he gave you time Bullshit. Please do not intervene in a thread when you do not know what is going on at all. Paul has been repeatedly rude and threatened to push before review, yet it is towards me you

Re: [FFmpeg-devel] [PATCH] avfilter/buffersrc: switch to activate

2023-11-06 Thread Vittorio Giovara
On Mon, Nov 6, 2023 at 3:07 PM Nicolas George wrote: > Vittorio Giovara (12023-11-06): > > Nicolas, have you thought of providing a patch to the bug Paul is > > describing? > > Why do you not ask Paul if he have thought of giving me time to have a > chance to do so? > You gave no indication you

Re: [FFmpeg-devel] [PATCH] avfilter/buffersrc: switch to activate

2023-11-06 Thread Nicolas George
Vittorio Giovara (12023-11-06): > Nicolas, have you thought of providing a patch to the bug Paul is > describing? Why do you not ask Paul if he have thought of giving me time to have a chance to do so? -- Nicolas George signature.asc Description: PGP signature

Re: [FFmpeg-devel] [PATCH] avfilter/buffersrc: switch to activate

2023-11-06 Thread Vittorio Giovara
On Mon, Nov 6, 2023 at 1:50 PM Nicolas George wrote: > Paul B Mahol (12023-11-06): > > Gonna apply soon. > > No you are not. > While I appreciate the brevity of this email, I think this kind of communication is non constructive. If the mailing list is the only and universal voice of the

Re: [FFmpeg-devel] [PATCH] avfilter/buffersrc: switch to activate

2023-11-06 Thread Nicolas George
Paul B Mahol (12023-11-06): > Gonna apply soon. No you are not. -- Nicolas George signature.asc Description: PGP signature ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe,

Re: [FFmpeg-devel] [PATCH] avfilter/buffersrc: switch to activate

2023-11-06 Thread Paul B Mahol
Gonna apply soon. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-devel] [PATCH] avfilter/buffersrc: switch to activate

2023-11-04 Thread Paul B Mahol
On Sat, Nov 4, 2023 at 8:07 PM Nicolas George wrote: > Paul B Mahol (12023-11-04): > > From 31a6a78ebc3a3f8785ec7c8e5ffd4257c7eadec3 Mon Sep 17 00:00:00 2001 > > From: Paul B Mahol > > Date: Fri, 27 Oct 2023 14:26:50 +0200 > > Subject: [PATCH] avfilter/buffersrc: switch to activate > > > >

Re: [FFmpeg-devel] [PATCH] avfilter/buffersrc: switch to activate

2023-11-04 Thread Nicolas George
Paul B Mahol (12023-11-04): > From 31a6a78ebc3a3f8785ec7c8e5ffd4257c7eadec3 Mon Sep 17 00:00:00 2001 > From: Paul B Mahol > Date: Fri, 27 Oct 2023 14:26:50 +0200 > Subject: [PATCH] avfilter/buffersrc: switch to activate > > Fixes error when caller keeps adding frames into filtergraph > that

[FFmpeg-devel] [PATCH] avfilter/buffersrc: switch to activate

2023-11-04 Thread Paul B Mahol
Attached. From 31a6a78ebc3a3f8785ec7c8e5ffd4257c7eadec3 Mon Sep 17 00:00:00 2001 From: Paul B Mahol Date: Fri, 27 Oct 2023 14:26:50 +0200 Subject: [PATCH] avfilter/buffersrc: switch to activate Fixes error when caller keeps adding frames into filtergraph that reached EOF by other means, for