Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
 help / color / mirror / Atom feed
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".

  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