From: Zhao Zhili <quinkblack@foxmail.com> To: FFmpeg development discussions and patches <ffmpeg-devel@ffmpeg.org> Subject: Re: [FFmpeg-devel] Sovereign Tech Fund Date: Mon, 5 Feb 2024 21:10:44 +0800 Message-ID: <tencent_71219F8CAB724E2C70672D979A99407B9A08@qq.com> (raw) In-Reply-To: <20240205102130.GQ6420@pb2> > On Feb 5, 2024, at 18:21, Michael Niedermayer <michael@niedermayer.cc> wrote: > > On Wed, Jan 31, 2024 at 09:55:00PM +0000, Kieran Kunhya wrote: >> On Wed, 31 Jan 2024 at 21:45, Derek Buitenhuis <derek.buitenhuis@gmail.com> >> wrote: >> >>> On 1/30/2024 1:48 AM, Michael Niedermayer wrote: >>>> https://trac.ffmpeg.org/wiki/SponsoringPrograms/STF/2024 >>> >>> Not to derail this fine thread, but what forks does the Merge Forks >>> project refer to? >>> >>> - Derek >>> >> >> I also added a note that 70 USD for coverity is way too much. I picked a >> random issue 1503073 and within a minute saw that it was a false positive. >> I don't deserve 70USD for that. > > I fixed 2 coverity issues yesterday and it took me over 3 hours > I cant do this for 70USD per issue > (you can see the ML for the 2 patches) > > In the first, the issue depended on fbw_channels to be 0. > If you look at its initialization that is possible if you have a > mono LFE channel but is that possible and can the code be reached > in that case. > For someone who hasnt worked at that specific code it takes some time > to build an argument that this should not be possible > > The second issue, its obvious a bug but how do we even reach that code? > No fate tests .... > luckily there are examples in the docs but it took me several tries to > get the code to execute with similar testcases. > now looking at it, i suspect the patch i posted probably should be split > so we need a 2nd iteration > and looking at the clock when i posted this and when i started yesterday > fact is it was 3-4h work for these 2 issues I think work on two to three issues in spare time per day is a rough but reasonable number, with one to two being easy and one from medium to hard. 210$ isn’t that much, especially for overtime pay. Many people have working on open source for free (as in beer) for many years, but it doesn’t mean that their work not worth like 70 $. > > did i pick these randomly? no, i started frm the top and skiped a few > i really did not want to work on like the flac parser. > > Some coverity isssues are dead easy and need seconds to categorize > or even fix. But for others its difficult > > Also to categorize coverity issues one needs to understand the affectd > code. coverity telling you that after 355 conditions theres a out of > array access, you need to know if the 355 conditions are inconsistant > and contradicting. If they contradict its a false positive otherwise > its a bug. > similar when you check the return code of a function most of the time > coverity will create an issue for cases where its not checked. Thats > trivial to fix IF you know the code. But IF you do not know the code > that can some decent time too. And i think noone knows all code. > > Either way, iam interrested in helping with coverity work while > at the same time this environment where peole finger point and say > "is way too much" is something i dont feel comfortable to work in. > > maybe doing it per hour instead of per issue is a safer way > > thx > > [...] > > -- > Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB > > Republics decline into democracies and democracies degenerate into > despotisms. -- Aristotle > _______________________________________________ > 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". _______________________________________________ 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:[~2024-02-05 13:11 UTC|newest] Thread overview: 123+ messages / expand[flat|nested] mbox.gz Atom feed top 2024-01-28 3:25 Michael Niedermayer 2024-01-28 15:54 ` Kieran Kunhya 2024-01-28 17:12 ` Michael Niedermayer 2024-01-28 18:59 ` Kieran Kunhya 2024-01-28 19:20 ` Jonatas L. Nogueira via ffmpeg-devel 2024-01-28 19:30 ` Jonatas L. Nogueira via ffmpeg-devel 2024-01-28 19:34 ` Kieran Kunhya 2024-01-28 21:18 ` Jonatas L. Nogueira via ffmpeg-devel 2024-01-28 21:33 ` Kieran Kunhya 2024-01-29 21:27 ` Anton Khirnov 2024-01-28 20:06 ` Michael Niedermayer 2024-01-28 20:32 ` Kieran Kunhya 2024-01-28 20:34 ` Kieran Kunhya 2024-01-28 20:37 ` Kieran Kunhya 2024-01-28 20:42 ` Kieran Kunhya 2024-01-28 21:47 ` Michael Niedermayer 2024-01-29 18:31 ` Kieran Kunhya 2024-01-29 18:46 ` Jonatas L. Nogueira via ffmpeg-devel 2024-01-29 18:54 ` Michael Niedermayer 2024-01-29 19:02 ` Kieran Kunhya 2024-01-29 20:04 ` Michael Niedermayer 2024-01-29 22:54 ` Kieran Kunhya 2024-01-30 9:20 ` Tomas Härdin 2024-01-28 21:34 ` Michael Niedermayer 2024-01-28 21:39 ` Kieran Kunhya 2024-01-29 2:26 ` Jonatas L. Nogueira via ffmpeg-devel 2024-01-29 14:52 ` Derek Buitenhuis 2024-01-29 15:02 ` Kieran Kunhya 2024-01-29 15:05 ` Derek Buitenhuis 2024-01-29 16:40 ` Jonatas L. Nogueira via ffmpeg-devel 2024-01-29 17:05 ` Kieran Kunhya 2024-01-29 17:27 ` Michael Niedermayer 2024-01-29 17:36 ` Kieran Kunhya 2024-01-29 17:43 ` Rémi Denis-Courmont 2024-01-29 18:11 ` Michael Niedermayer 2024-01-29 21:01 ` Rémi Denis-Courmont 2024-01-29 22:43 ` Michael Niedermayer 2024-01-30 6:30 ` Rémi Denis-Courmont 2024-01-30 17:15 ` Michael Niedermayer 2024-01-30 18:00 ` Michael Niedermayer [not found] ` <A40E9FF7-EC74-458A-A195-26EE8062992E@cosmin.at> 2024-01-29 22:23 ` Cosmin Stejerean via ffmpeg-devel 2024-01-29 22:31 ` Kieran Kunhya 2024-01-30 10:12 ` Nicolas George 2024-01-30 10:19 ` Kieran Kunhya 2024-01-30 10:31 ` Nicolas George 2024-01-30 10:44 ` Kieran Kunhya 2024-01-30 10:46 ` Nicolas George 2024-01-30 10:53 ` Kieran Kunhya 2024-01-30 11:47 ` Anton Khirnov 2024-01-28 19:17 ` Rémi Denis-Courmont 2024-01-28 20:33 ` Michael Niedermayer 2024-01-29 14:38 ` Derek Buitenhuis 2024-01-29 18:25 ` Michael Niedermayer 2024-01-29 18:37 ` Derek Buitenhuis 2024-01-29 19:21 ` Michael Niedermayer 2024-01-29 20:09 ` Vittorio Giovara 2024-01-29 20:15 ` Derek Buitenhuis 2024-01-30 6:48 ` Rémi Denis-Courmont 2024-01-29 20:19 ` Anton Khirnov 2024-01-29 20:20 ` Derek Buitenhuis 2024-01-29 20:36 ` Vittorio Giovara 2024-01-29 21:27 ` Michael Niedermayer 2024-01-31 11:19 ` Anton Khirnov 2024-01-29 20:41 ` Diederick C. Niehorster 2024-01-29 21:19 ` Anton Khirnov 2024-01-29 21:11 ` Anton Khirnov 2024-01-29 23:41 ` Jonatas L. Nogueira via ffmpeg-devel 2024-01-29 23:53 ` Stefano Sabatini 2024-01-31 12:30 ` Anton Khirnov 2024-01-31 21:26 ` Stefano Sabatini 2024-01-30 0:15 ` Michael Niedermayer 2024-01-30 0:19 ` Michael Niedermayer 2024-01-31 12:59 ` Anton Khirnov 2024-01-31 14:10 ` Jonatas L. Nogueira via ffmpeg-devel 2024-01-31 15:17 ` Anton Khirnov 2024-01-31 15:17 ` Kieran Kunhya 2024-01-31 16:00 ` Jonatas L. Nogueira via ffmpeg-devel 2024-01-31 16:03 ` Jonatas L. Nogueira via ffmpeg-devel 2024-01-31 16:10 ` Rémi Denis-Courmont 2024-01-31 17:04 ` Jonatas L. Nogueira via ffmpeg-devel 2024-01-31 18:03 ` Michael Niedermayer 2024-01-31 18:22 ` Kieran Kunhya 2024-01-31 18:40 ` Jonatas L. Nogueira via ffmpeg-devel 2024-01-31 18:48 ` Kieran Kunhya 2024-01-31 19:07 ` Michael Niedermayer [not found] ` <A7F30D96-F8DB-45EA-9CDB-3545E3ECE0C9@cosmin.at> 2024-01-31 19:16 ` Cosmin Stejerean via ffmpeg-devel 2024-01-31 20:19 ` Kieran Kunhya 2024-01-31 21:43 ` Michael Niedermayer 2024-01-31 21:54 ` Kieran Kunhya 2024-01-31 22:40 ` Michael Niedermayer 2024-01-31 22:45 ` Kieran Kunhya 2024-02-02 13:52 ` Michael Niedermayer 2024-02-02 13:58 ` Kieran Kunhya 2024-01-31 19:20 ` Jonatas L. Nogueira via ffmpeg-devel 2024-01-31 17:58 ` Michael Niedermayer 2024-01-31 23:15 ` Stefano Sabatini 2024-02-01 0:16 ` Stefano Sabatini 2024-02-06 0:00 ` Jonatas L. Nogueira via ffmpeg-devel 2024-01-30 1:48 ` Michael Niedermayer 2024-01-30 9:32 ` Vittorio Giovara 2024-01-30 10:07 ` Nicolas George 2024-01-30 10:13 ` Vittorio Giovara 2024-01-30 10:15 ` Nicolas George 2024-01-30 10:56 ` Vittorio Giovara 2024-01-31 1:07 ` Michael Niedermayer 2024-01-31 21:44 ` Derek Buitenhuis 2024-01-31 21:55 ` Kieran Kunhya 2024-01-31 23:07 ` Michael Niedermayer 2024-02-01 17:59 ` Anton Khirnov 2024-02-01 18:14 ` Rémi Denis-Courmont 2024-02-01 22:55 ` Michael Niedermayer 2024-02-05 10:21 ` Michael Niedermayer 2024-02-05 11:53 ` Kieran Kunhya 2024-02-05 13:10 ` Zhao Zhili [this message] 2024-02-01 19:22 ` Derek Buitenhuis 2024-02-04 9:49 ` Rémi Denis-Courmont 2024-02-04 10:02 ` J. Dekker 2024-02-04 10:09 ` Paul B Mahol 2024-02-04 13:41 ` Michael Niedermayer 2024-02-04 14:38 ` Rémi Denis-Courmont 2024-02-04 19:28 ` Michael Niedermayer 2024-02-04 21:21 ` Rémi Denis-Courmont 2024-04-12 23:43 ` Thilo Borgmann via ffmpeg-devel
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=tencent_71219F8CAB724E2C70672D979A99407B9A08@qq.com \ --to=quinkblack@foxmail.com \ --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