From: Marton Balint <cus@passwd.hu> To: FFmpeg development discussions and patches <ffmpeg-devel@ffmpeg.org> Subject: Re: [FFmpeg-devel] [PATCH 0/2] Remove SDL2 output devices Date: Sun, 4 Feb 2024 15:55:41 +0100 (CET) Message-ID: <fd1d3ee7-fe18-8c90-6730-18ae490a2bc9@passwd.hu> (raw) In-Reply-To: <6FC895D9-500D-437B-BFEE-48E44C994499@remlab.net> On Sun, 4 Feb 2024, Rémi Denis-Courmont wrote: > > > Le 4 février 2024 11:11:12 GMT+01:00, Marton Balint <cus@passwd.hu> a écrit : >> Actually they work here on a linux box with OpenSuse 15.5. So even if they >> are broken on some setups, they are not broken everywhere, or not more broken than they used to be. > > No. They were always broken in terms of the design, and they are more technically broken than before because the threading rework exposed the design bugs from within fftools. > > No sane application would use this. If it doesn't even work in fftools, it should be removed. As I wrote earlier, it works for me. > >> Also, poper deprecation is needed here, since not only the CLI tools might use these. Especially since there is no drop-in replacement. > > First it's not what would be considered an API. The removal shouldn't break source compatibility, so deprecation won't get us anything here. Where would you even put the deprecation guards? See what Anton did with the BKTR device. > > And then deprecation only makes sense if it can be fixed. Nobody has come forward with a practical solution to make it work, probably because there is not one, at least on MacOS. See Michael's suggestions with the thread safety flag. Regards, Marton _______________________________________________ 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-04 14:55 UTC|newest] Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top 2024-02-04 9:02 J. Dekker 2024-02-04 9:02 ` [FFmpeg-devel] [PATCH 1/2] avdevice: remove sdl2 outdev J. Dekker 2024-02-04 9:02 ` [FFmpeg-devel] [PATCH 2/2] avdevice: remove OpenGL device J. Dekker 2024-02-04 9:19 ` [FFmpeg-devel] [PATCH 0/2] Remove SDL2 output devices Zhao Zhili 2024-02-04 9:31 ` Anton Khirnov 2024-02-04 10:08 ` Paul B Mahol 2024-02-04 9:26 ` Anton Khirnov 2024-02-04 10:11 ` Marton Balint 2024-02-04 11:36 ` Rémi Denis-Courmont 2024-02-04 14:55 ` Marton Balint [this message] 2024-02-07 10:31 ` Anton Khirnov 2024-02-07 12:13 ` Nicolas George 2024-02-04 11:37 ` Rémi Denis-Courmont 2024-02-04 13:15 ` Michael Niedermayer 2024-02-05 0:02 ` Stefano Sabatini 2024-02-07 17:09 ` Anton Khirnov 2024-02-07 17:37 ` Nicolas George 2024-02-07 17:40 ` Vittorio Giovara 2024-02-07 22:27 ` Paul B Mahol 2024-02-06 8:08 ` Michael Koch 2024-02-06 11:45 ` Zhao Zhili 2024-02-06 12:40 ` Nicolas George 2024-02-06 20:51 ` Michael Koch 2024-02-07 9:35 ` Zhao Zhili 2024-02-07 10:31 ` Michael Koch 2024-02-06 15:02 ` Vittorio Giovara
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=fd1d3ee7-fe18-8c90-6730-18ae490a2bc9@passwd.hu \ --to=cus@passwd.hu \ --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