Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
 help / color / mirror / Atom feed
From: Stefano Sabatini <stefasab@gmail.com>
To: FFmpeg development discussions and patches <ffmpeg-devel@ffmpeg.org>
Subject: Re: [FFmpeg-devel] [PATCH 0/2] Remove SDL2 output devices
Date: Mon, 5 Feb 2024 01:02:20 +0100
Message-ID: <ZcAljPSUbRb2pChF@mariano> (raw)
In-Reply-To: <20240204090233.1157950-1-jdek@itanimul.li>

On date Sunday 2024-02-04 10:02:31 +0100, J. Dekker wrote:
> With the addition of threading in ffmpeg.c, the SDL2 devices no longer have the
> 'main' thread. This means that both the SDL2 and OpenGL output device are broken
> in master. Rather than attempting to fix it, they should be removed instead as
> there are better alternatives for debugging or viewing streams.
> 
> The 'pipe:' output can be used with a real video player such as mpv, vlc, or
> even ffplay. For cases where the user was an application using the API they
> should supply their own renderer.
> 
> J. Dekker (2):
>   avdevice: remove sdl2 outdev
>   avdevice: remove OpenGL device

I am against this.

1. If at all, the devices should be deprecated and removed only after a
given timeframe, so users can switch to something else (you don't know
what they are using).

2. > With the addition of threading in ffmpeg.c, the SDL2 devices no longer have the
   > 'main' thread. 

This implies a misunderstanding of what these components are. If
they are broken with ffmpeg.c this is not a good reason to remove
them (ffmpeg.c is not the only user).

Also, it was already suggested some way to fix it, it's not like they
are "broken by design", it is just that the assumptions done
previuosly apply no more. Probably ffmpeg.c should not use the main
thread or make this selectable. If this cannot be supported in a given
OS, the feature should be disabled only for that OS.
_______________________________________________
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:[~2024-02-05  0:02 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
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 [this message]
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=ZcAljPSUbRb2pChF@mariano \
    --to=stefasab@gmail.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