Re: [FFmpeg-devel] [PATCH] lavc: remove the QOA decoder

2023-12-26 Thread Michael Niedermayer
On Mon, Dec 25, 2023 at 11:22:40AM -0600, Leo Izen wrote: > On 12/21/23 21:32, Michael Niedermayer wrote: > > > > Can you think of a way to add some lines of code to this that makes it more > > maintainable ? > > > > if yes, then i think you proofed that adding code can reduce maintaince > >

Re: [FFmpeg-devel] [PATCH] lavc: remove the QOA decoder

2023-12-25 Thread Leo Izen
On 12/21/23 21:32, Michael Niedermayer wrote: Can you think of a way to add some lines of code to this that makes it more maintainable ? if yes, then i think you proofed that adding code can reduce maintaince burden thx This is clearly not the point here. The point is that an in-house

Re: [FFmpeg-devel] [PATCH] lavc: remove the QOA decoder

2023-12-24 Thread Paul B Mahol
On Sun, Dec 24, 2023 at 6:22 PM Michael Niedermayer wrote: > On Sun, Dec 24, 2023 at 04:11:30PM +0100, Nicolas George wrote: > > Tomas Härdin (12023-12-24): > > > Nah, NIH is bad. > > > > Then you need to find a different project. FFmpeg exists because a > > genius hacker decided to NIH a MPEG

Re: [FFmpeg-devel] [PATCH] lavc: remove the QOA decoder

2023-12-24 Thread Tomas Härdin
sön 2023-12-24 klockan 18:21 +0100 skrev Michael Niedermayer: > On Sun, Dec 24, 2023 at 04:11:30PM +0100, Nicolas George wrote: > > Tomas Härdin (12023-12-24): > > > Nah, NIH is bad. > > > > Then you need to find a different project. FFmpeg exists because a > > genius hacker decided to NIH a MPEG

Re: [FFmpeg-devel] [PATCH] lavc: remove the QOA decoder

2023-12-24 Thread Michael Niedermayer
On Sun, Dec 24, 2023 at 04:11:30PM +0100, Nicolas George wrote: > Tomas Härdin (12023-12-24): > > Nah, NIH is bad. > > Then you need to find a different project. FFmpeg exists because a > genius hacker decided to NIH a MPEG software decoder. Please stop attacking Tomas! FFmpeg exists because

Re: [FFmpeg-devel] [PATCH] lavc: remove the QOA decoder

2023-12-24 Thread Nicolas George
Tomas Härdin (12023-12-24): > Nah, NIH is bad. Then you need to find a different project. FFmpeg exists because a genius hacker decided to NIH a MPEG software decoder. -- Nicolas George signature.asc Description: PGP signature ___ ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] lavc: remove the QOA decoder

2023-12-24 Thread Paul B Mahol
On Sun, Dec 24, 2023 at 3:33 PM Tomas Härdin wrote: > sön 2023-12-24 klockan 13:12 +0100 skrev Nicolas George: > > Tomas Härdin (12023-12-24): > > > But I'd prefer if it were replaced by bmxlib. > > > > Yes, we know that you have not understood that the raison d'être of > > FFmpeg is to

Re: [FFmpeg-devel] [PATCH] lavc: remove the QOA decoder

2023-12-24 Thread Tomas Härdin
sön 2023-12-24 klockan 13:12 +0100 skrev Nicolas George: > Tomas Härdin (12023-12-24): > > But I'd prefer if it were replaced by bmxlib. > > Yes, we know that you have not understood that the raison d'être of > FFmpeg is to IMPLEMENT natively all multimedia formats and protocols > and > the

Re: [FFmpeg-devel] [PATCH] lavc: remove the QOA decoder

2023-12-24 Thread James Almer
On 12/24/2023 9:12 AM, Nicolas George wrote: Tomas Härdin (12023-12-24): But I'd prefer if it were replaced by bmxlib. Yes, we know that you have not understood that the raison d'être of FFmpeg is to IMPLEMENT natively all multimedia formats and protocols and the utility code they need, not

Re: [FFmpeg-devel] [PATCH] lavc: remove the QOA decoder

2023-12-24 Thread Nicolas George
Tomas Härdin (12023-12-24): > But I'd prefer if it were replaced by bmxlib. Yes, we know that you have not understood that the raison d'être of FFmpeg is to IMPLEMENT natively all multimedia formats and protocols and the utility code they need, not be a mere collection of wrappers. -- Nicolas

Re: [FFmpeg-devel] [PATCH] lavc: remove the QOA decoder

2023-12-24 Thread Tomas Härdin
fre 2023-12-22 klockan 10:09 -0300 skrev James Almer: > On 12/21/2023 5:05 PM, Paul B Mahol wrote: > > On Thu, Dec 21, 2023 at 8:43 PM Tomas Härdin > > wrote: > > > > > ons 2023-12-20 klockan 20:11 +0100 skrev Michael Niedermayer: > > > > On Wed, Dec 20, 2023 at 05:57:40PM +0100, Tomas Härdin

Re: [FFmpeg-devel] [PATCH] lavc: remove the QOA decoder

2023-12-23 Thread Michael Niedermayer
Hi Nicolas On Thu, Dec 21, 2023 at 09:31:00PM +0100, Nicolas George wrote: > Paul B Mahol (12023-12-21): > > Say what serious feature you contributed ? - Nothing. > > I did not want to say it, but since it is now in the open: Not nothing: > negative. Just because someone attacks someone else

Re: [FFmpeg-devel] [PATCH] lavc: remove the QOA decoder

2023-12-22 Thread James Almer
On 12/21/2023 5:05 PM, Paul B Mahol wrote: On Thu, Dec 21, 2023 at 8:43 PM Tomas Härdin wrote: ons 2023-12-20 klockan 20:11 +0100 skrev Michael Niedermayer: On Wed, Dec 20, 2023 at 05:57:40PM +0100, Tomas Härdin wrote: tis 2023-12-19 klockan 15:02 +0100 skrev Nicolas George: [...] [...] ,

Re: [FFmpeg-devel] [PATCH] lavc: remove the QOA decoder

2023-12-22 Thread Paul B Mahol
On Fri, Dec 22, 2023 at 7:28 AM Anton Khirnov wrote: > Hi Paul, > Quoting Paul B Mahol (2023-12-21 21:05:18) > > Say what serious feature you contributed ? - Nothing. > > this is a personal attach. Personal attacks are not allowed on this > mailing list. Please refrain from them in the future. >

Re: [FFmpeg-devel] [PATCH] lavc: remove the QOA decoder

2023-12-21 Thread Anton Khirnov
Hi Paul, Quoting Paul B Mahol (2023-12-21 21:05:18) > Say what serious feature you contributed ? - Nothing. this is a personal attach. Personal attacks are not allowed on this mailing list. Please refrain from them in the future. -- Anton Khirnov (CC member)

Re: [FFmpeg-devel] [PATCH] lavc: remove the QOA decoder

2023-12-21 Thread Michael Niedermayer
On Thu, Dec 21, 2023 at 08:43:16PM +0100, Tomas Härdin wrote: > ons 2023-12-20 klockan 20:11 +0100 skrev Michael Niedermayer: > > On Wed, Dec 20, 2023 at 05:57:40PM +0100, Tomas Härdin wrote: > > > tis 2023-12-19 klockan 15:02 +0100 skrev Nicolas George: > > [...] > > > [...] , but every line of

Re: [FFmpeg-devel] [PATCH] lavc: remove the QOA decoder

2023-12-21 Thread Michael Niedermayer
On Thu, Dec 21, 2023 at 05:37:58PM -0500, Vittorio Giovara wrote: > On Thu, Dec 21, 2023 at 5:22 PM Nicolas George wrote: > > > Vittorio Giovara (12023-12-21): > > > Oh so you know how it feels! > > > > I am not surprised to see your animosity against me is stronger than > > your care about the

Re: [FFmpeg-devel] [PATCH] lavc: remove the QOA decoder

2023-12-21 Thread Vittorio Giovara
On Thu, Dec 21, 2023 at 6:14 PM Nicolas George wrote: > Vittorio Giovara (12023-12-21): > > I agree, I hope you can work on your lack of self-awareness. > > Oh, the “Ah ?… Et moi, Cyrano-Savinien-Hercule De Bergerac.” comeback. > You're really scrapping the bottom of the barrel, aren't your. >

Re: [FFmpeg-devel] [PATCH] lavc: remove the QOA decoder

2023-12-21 Thread Nicolas George
Vittorio Giovara (12023-12-21): > I agree, I hope you can work on your lack of self-awareness. Oh, the “Ah ?… Et moi, Cyrano-Savinien-Hercule De Bergerac.” comeback. You're really scrapping the bottom of the barrel, aren't your. -- Nicolas George

Re: [FFmpeg-devel] [PATCH] lavc: remove the QOA decoder

2023-12-21 Thread Vittorio Giovara
On Thu, Dec 21, 2023 at 5:41 PM Nicolas George wrote: > Vittorio Giovara (12023-12-21): > > Not sure what gave you the idea > > Mails like this one for example. The lack of self-awareness is > hilarious. > I agree, I hope you can work on your lack of self-awareness. -- Vittorio

Re: [FFmpeg-devel] [PATCH] lavc: remove the QOA decoder

2023-12-21 Thread Nicolas George
Vittorio Giovara (12023-12-21): > Not sure what gave you the idea Mails like this one for example. The lack of self-awareness is hilarious. -- Nicolas George ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

Re: [FFmpeg-devel] [PATCH] lavc: remove the QOA decoder

2023-12-21 Thread Vittorio Giovara
On Thu, Dec 21, 2023 at 5:22 PM Nicolas George wrote: > Vittorio Giovara (12023-12-21): > > Oh so you know how it feels! > > I am not surprised to see your animosity against me is stronger than > your care about the project. > Not sure what gave you the idea, but I have no animosity towards you

Re: [FFmpeg-devel] [PATCH] lavc: remove the QOA decoder

2023-12-21 Thread Nicolas George
Vittorio Giovara (12023-12-21): > Oh so you know how it feels! I am not surprised to see your animosity against me is stronger than your care about the project. -- Nicolas George signature.asc Description: PGP signature ___ ffmpeg-devel mailing

Re: [FFmpeg-devel] [PATCH] lavc: remove the QOA decoder

2023-12-21 Thread Vittorio Giovara
On Thu, Dec 21, 2023 at 3:05 PM Paul B Mahol wrote: > On Thu, Dec 21, 2023 at 8:43 PM Tomas Härdin wrote: > > > ons 2023-12-20 klockan 20:11 +0100 skrev Michael Niedermayer: > > > On Wed, Dec 20, 2023 at 05:57:40PM +0100, Tomas Härdin wrote: > > > > tis 2023-12-19 klockan 15:02 +0100 skrev

Re: [FFmpeg-devel] [PATCH] lavc: remove the QOA decoder

2023-12-21 Thread Vittorio Giovara
On Thu, Dec 21, 2023 at 3:31 PM Nicolas George wrote: > Paul B Mahol (12023-12-21): > > Say what serious feature you contributed ? - Nothing. > > I did not want to say it, but since it is now in the open: Not nothing: > negative. His naysaying discouraged me from working further on the >

Re: [FFmpeg-devel] [PATCH] lavc: remove the QOA decoder

2023-12-21 Thread Nicolas George
Paul B Mahol (12023-12-21): > Say what serious feature you contributed ? - Nothing. I did not want to say it, but since it is now in the open: Not nothing: negative. His naysaying discouraged me from working further on the built-in documentation system. -- Nicolas George signature.asc

Re: [FFmpeg-devel] [PATCH] lavc: remove the QOA decoder

2023-12-21 Thread Paul B Mahol
On Thu, Dec 21, 2023 at 8:43 PM Tomas Härdin wrote: > ons 2023-12-20 klockan 20:11 +0100 skrev Michael Niedermayer: > > On Wed, Dec 20, 2023 at 05:57:40PM +0100, Tomas Härdin wrote: > > > tis 2023-12-19 klockan 15:02 +0100 skrev Nicolas George: > > [...] > > > [...] , but every line of code > >

Re: [FFmpeg-devel] [PATCH] lavc: remove the QOA decoder

2023-12-21 Thread Tomas Härdin
ons 2023-12-20 klockan 20:11 +0100 skrev Michael Niedermayer: > On Wed, Dec 20, 2023 at 05:57:40PM +0100, Tomas Härdin wrote: > > tis 2023-12-19 klockan 15:02 +0100 skrev Nicolas George: > [...] > > [...] , but every line of code > > carries with it a non-zero maintenance burden > > Assuming you

Re: [FFmpeg-devel] [PATCH] lavc: remove the QOA decoder

2023-12-20 Thread Michael Niedermayer
On Wed, Dec 20, 2023 at 05:57:40PM +0100, Tomas Härdin wrote: > tis 2023-12-19 klockan 15:02 +0100 skrev Nicolas George: [...] > [...] , but every line of code > carries with it a non-zero maintenance burden Assuming you mean with "non-zero" a "larger than zero" maintenance burden then we can

Re: [FFmpeg-devel] [PATCH] lavc: remove the QOA decoder

2023-12-20 Thread Nicolas George
Tomas Härdin (12023-12-20): > I'm not even sure what this is trying to say, but every line of code > carries with it a non-zero maintenance burden Sure, if you want. But for some parts of FFmpeg, like the one we are discussing, the maintenance burden is so small that we wasted ten time as much

Re: [FFmpeg-devel] [PATCH] lavc: remove the QOA decoder

2023-12-20 Thread Tomas Härdin
tis 2023-12-19 klockan 15:02 +0100 skrev Nicolas George: > Tomas Härdin (12023-12-19): > > Technical debt concerns everyone > > There is no technical debt. There is never a technical debt from > something that can be removed at an instant notice and that some are > considering removing now I'm

Re: [FFmpeg-devel] [PATCH] lavc: remove the QOA decoder

2023-12-19 Thread Nicolas George
Tomas Härdin (12023-12-19): > Technical debt concerns everyone There is no technical debt. There is never a technical debt from something that can be removed at an instant notice and that some are considering removing now. Learn from the Dread Pirate Roberts: always put off stupid things until

Re: [FFmpeg-devel] [PATCH] lavc: remove the QOA decoder

2023-12-19 Thread Tomas Härdin
tis 2023-12-05 klockan 08:44 +0100 skrev Nicolas George: > Vittorio Giovara (12023-12-04): > > It's almost 2024, when will you be able to drop it? > > Never. When will YOU be able to drop bickering about parts of the > code > that do no harm and do not bother you? Technical debt concerns

Re: [FFmpeg-devel] [PATCH] lavc: remove the QOA decoder

2023-12-06 Thread Anton Khirnov
Quoting James Almer (2023-12-06 21:49:45) > On 12/2/2023 12:53 PM, Anton Khirnov wrote: > > Its author not only failed to add any tests, as is required by the > > development rules, but continues to actively refuse to do so. > > > > Untested decoders are worse than useless, so remove it. > > ---

Re: [FFmpeg-devel] [PATCH] lavc: remove the QOA decoder

2023-12-06 Thread Vittorio Giovara
I ain't reading all of that Happy for you Or sorry that happened On Wed, Dec 6, 2023 at 3:47 PM Nicolas George wrote: > Vittorio Giovara (12023-12-05): > > Your attitude for "omgfeatures" is also pretty toxic, there are heaps of > > literature about feature creep and how important it is to

Re: [FFmpeg-devel] [PATCH] lavc: remove the QOA decoder

2023-12-06 Thread James Almer
On 12/2/2023 12:53 PM, Anton Khirnov wrote: Its author not only failed to add any tests, as is required by the development rules, but continues to actively refuse to do so. Untested decoders are worse than useless, so remove it. --- Changelog | 2 +- libavcodec/Makefile|

Re: [FFmpeg-devel] [PATCH] lavc: remove the QOA decoder

2023-12-06 Thread Nicolas George
Vittorio Giovara (12023-12-05): > Your attitude for "omgfeatures" is also pretty toxic, there are heaps of > literature about feature creep and how important it is to remove dead code. You should read said literature before quoting it, you would learn the difference between more features and

Re: [FFmpeg-devel] [PATCH] lavc: remove the QOA decoder

2023-12-05 Thread Vittorio Giovara
On Tue, Dec 5, 2023 at 9:59 AM Nicolas George wrote: > Vittorio Giovara (12023-12-05): > > I'm not bickering, > > That is exactly what you do. > And you are not? The view must be great from that glass house > actively include testing as part of the bare minimum process to accept the > >

Re: [FFmpeg-devel] [PATCH] lavc: remove the QOA decoder

2023-12-05 Thread Nicolas George
Vittorio Giovara (12023-12-05): > I'm not bickering, That is exactly what you do. > actively include testing as part of the bare minimum process to accept the > patch/review is harming the whole generation of developers. I agree that Paul's attitude is utterly toxic to the project, but Anton's

Re: [FFmpeg-devel] [PATCH] lavc: remove the QOA decoder

2023-12-05 Thread Vittorio Giovara
On Tue, Dec 5, 2023 at 2:45 AM Nicolas George wrote: > Vittorio Giovara (12023-12-04): > > It's almost 2024, when will you be able to drop it? > > Never. When will YOU be able to drop bickering about parts of the code > that do no harm and do not bother you? > I'm not bickering, we're having a

Re: [FFmpeg-devel] [PATCH] lavc: remove the QOA decoder

2023-12-04 Thread Nicolas George
Vittorio Giovara (12023-12-04): > It's almost 2024, when will you be able to drop it? Never. When will YOU be able to drop bickering about parts of the code that do no harm and do not bother you? -- Nicolas George ___ ffmpeg-devel mailing list

Re: [FFmpeg-devel] [PATCH] lavc: remove the QOA decoder

2023-12-04 Thread Vittorio Giovara
On Mon, Dec 4, 2023 at 1:53 PM Nicolas George wrote: > Anton Khirnov (12023-12-02): > > Its author not only failed to add any tests, as is required by the > > development rules, but continues to actively refuse to do so. > > > > Untested decoders are worse than useless, so remove it. > > Only

Re: [FFmpeg-devel] [PATCH] lavc: remove the QOA decoder

2023-12-04 Thread Jean-Baptiste Kempf
Hi, On Sat, 2 Dec 2023, at 19:55, Paul B Mahol wrote: > Its not hard to add/write test, and even its easy to upload test sample to > server, but last time I did it people complained of too big samples. What is too big a sample? What's the problem with large samples? What is the documented

Re: [FFmpeg-devel] [PATCH] lavc: remove the QOA decoder

2023-12-04 Thread Nicolas George
Anton Khirnov (12023-12-02): > Its author not only failed to add any tests, as is required by the > development rules, but continues to actively refuse to do so. > > Untested decoders are worse than useless, so remove it. Only tested manually ≠ untested. A code that works is better than no code

Re: [FFmpeg-devel] [PATCH] lavc: remove the QOA decoder

2023-12-02 Thread Paul B Mahol
On Sat, Dec 2, 2023 at 7:20 PM Cosmin Stejerean via ffmpeg-devel < ffmpeg-devel@ffmpeg.org> wrote: > > > > On Dec 2, 2023, at 7:53 AM, Anton Khirnov wrote: > > > > Its author not only failed to add any tests, as is required by the > > development rules, but continues to actively refuse to do so.

Re: [FFmpeg-devel] [PATCH] lavc: remove the QOA decoder

2023-12-02 Thread Cosmin Stejerean via ffmpeg-devel
> On Dec 2, 2023, at 7:53 AM, Anton Khirnov wrote: > > Its author not only failed to add any tests, as is required by the > development rules, but continues to actively refuse to do so. > > Untested decoders are worse than useless, so remove it. While I agree that decoders (among other

Re: [FFmpeg-devel] [PATCH] lavc: remove the QOA decoder

2023-12-02 Thread Paul B Mahol
What a nice narrative. Yes, remove this decoder. And keep two sonic decoders at same time of questionable code quality, functionality and what not. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel