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