From: Michael Niedermayer <michael@niedermayer.cc> To: FFmpeg development discussions and patches <ffmpeg-devel@ffmpeg.org> Subject: Re: [FFmpeg-devel] [PATCH] lavc: remove the QOA decoder Date: Wed, 20 Dec 2023 20:11:58 +0100 Message-ID: <20231220191158.GA6420@pb2> (raw) In-Reply-To: <584f952e5d3a56615f12426276bc1d05f81a0e49.camel@haerdin.se> [-- Attachment #1.1: Type: text/plain, Size: 3070 bytes --] 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 proof this to be false First we need to define what you mean by "maintenance burden" ? There are a few ways this could be defined A. the absolute number of hours all developers need to spend to maintain some sort of stable quality B. the number of hours on average a FFmpeg developers need to spend to maintain some sort of stable quality (A favors 9 hours over 10 hours even if the 10 hour case has 2 devlopers available but the 9 has only 1. While B favors the 10/2 over 9/1) C. the number of hours of work noone really wants to do, FFmpeg developers need to spend to maintain some sort of stable quality (again we can do this as all or per developer) (the idea of C is that we count work that people dont like to do with more weight) (and there are many more ways to define it ...) Now the sketch of a proof :) Consider all the code related to "--help" code related to printing the version and build. Code printing where to send bugreports/samples. Also all formating maybe ;) or even some random check here or there. The "maintenance burden" in all definitions will worsen as the code becomes less readable, less well documented or as things related to maintaince are removed But its more than just this i think. * If you remove code that some comnpany or major user needs, and who pays for maintaince than removial of even complex and hard to maintain code can actually be negative and similarly adding complex code can actually be positive maintaince wise. IF it also results in additional resources becoming available for maintaince. (this can be a developers time or a companies money or other) * In the same light both merging and spliting code can have an impact on maintenance burden. For example if you have a group of developers who are unable to work together, them spliting up in 2 forks and a 3rd merging their work together avoiding their inability to agree can reduce burden on both OTOH if 2 groups can join and work together the sharing of resources and such can free up time and reduce maintaince burden What iam trying to say is, the maintaince burden resulting from a change is complex In this specific case here we have a patch proposing the removial of a decoder missing a test. Its easy to say the burden is less when the decoder is removed But its author recently left the project too [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Awnsering whenever a program halts or runs forever is On a turing machine, in general impossible (turings halting problem). On any real computer, always possible as a real computer has a finite number of states N, and will either halt in less than N cycles or never halt. [-- Attachment #1.2: signature.asc --] [-- Type: application/pgp-signature, Size: 195 bytes --] [-- Attachment #2: Type: text/plain, Size: 251 bytes --] _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
next prev parent reply other threads:[~2023-12-20 19:12 UTC|newest] Thread overview: 53+ messages / expand[flat|nested] mbox.gz Atom feed top 2023-12-02 15:53 Anton Khirnov 2023-12-02 16:07 ` Paul B Mahol [not found] ` <42528A64-3630-4EBD-A2D2-A6A3E63709BB@cosmin.at> 2023-12-02 18:19 ` Cosmin Stejerean via ffmpeg-devel 2023-12-02 18:55 ` Paul B Mahol 2023-12-04 18:57 ` Jean-Baptiste Kempf 2023-12-04 18:53 ` Nicolas George 2023-12-05 4:21 ` Vittorio Giovara 2023-12-05 7:44 ` Nicolas George 2023-12-05 14:51 ` Vittorio Giovara 2023-12-05 14:59 ` Nicolas George 2023-12-05 15:42 ` Vittorio Giovara 2023-12-06 20:47 ` Nicolas George 2023-12-06 20:55 ` Vittorio Giovara 2023-12-19 13:57 ` Tomas Härdin 2023-12-19 14:02 ` Nicolas George 2023-12-20 16:57 ` Tomas Härdin 2023-12-20 18:05 ` Nicolas George 2023-12-20 19:11 ` Michael Niedermayer [this message] 2023-12-21 19:43 ` Tomas Härdin 2023-12-21 20:05 ` Paul B Mahol 2023-12-21 20:31 ` Nicolas George 2023-12-21 21:36 ` Vittorio Giovara 2023-12-21 22:21 ` Nicolas George 2023-12-21 22:37 ` Vittorio Giovara 2023-12-21 22:41 ` Nicolas George 2023-12-21 23:12 ` Vittorio Giovara 2023-12-21 23:14 ` Nicolas George 2023-12-21 23:24 ` Vittorio Giovara 2023-12-22 0:32 ` Michael Niedermayer 2023-12-24 1:17 ` Michael Niedermayer 2023-12-26 13:52 ` [FFmpeg-devel] Mailinglist conduct [was: Re: [PATCH] lavc: remove the QOA decoder] Ronald S. Bultje 2023-12-21 21:38 ` [FFmpeg-devel] [PATCH] lavc: remove the QOA decoder Vittorio Giovara 2023-12-22 6:28 ` Anton Khirnov 2023-12-22 9:28 ` Paul B Mahol 2023-12-22 13:09 ` James Almer 2023-12-24 12:10 ` Tomas Härdin 2023-12-24 12:12 ` Nicolas George 2023-12-24 13:23 ` James Almer 2023-12-24 14:33 ` Tomas Härdin 2023-12-24 14:47 ` Paul B Mahol 2023-12-24 15:11 ` Nicolas George 2023-12-24 17:21 ` Michael Niedermayer 2023-12-24 20:55 ` Tomas Härdin 2023-12-24 21:07 ` Paul B Mahol 2023-12-26 22:20 ` [FFmpeg-devel] Mailinglist conduct [was: Re: [PATCH] lavc: remove the QOA decoder] Ronald S. Bultje 2023-12-26 22:40 ` Paul B Mahol 2023-12-26 23:53 ` James Almer 2023-12-27 8:13 ` Paul B Mahol 2023-12-22 3:32 ` [FFmpeg-devel] [PATCH] lavc: remove the QOA decoder Michael Niedermayer 2023-12-25 17:22 ` Leo Izen 2023-12-26 12:58 ` Michael Niedermayer 2023-12-06 20:49 ` James Almer 2023-12-07 7:17 ` Anton Khirnov
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20231220191158.GA6420@pb2 \ --to=michael@niedermayer.cc \ --cc=ffmpeg-devel@ffmpeg.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: link
Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel This inbox may be cloned and mirrored by anyone: git clone --mirror https://master.gitmailbox.com/ffmpegdev/0 ffmpegdev/git/0.git # If you have public-inbox 1.1+ installed, you may # initialize and index your mirror using the following commands: public-inbox-init -V2 ffmpegdev ffmpegdev/ https://master.gitmailbox.com/ffmpegdev \ ffmpegdev@gitmailbox.com public-inbox-index ffmpegdev Example config snippet for mirrors. AGPL code for this site: git clone https://public-inbox.org/public-inbox.git