[FFmpeg-devel] [PATCH] avformat/libsrt: remove url_get_file_handle implementation

Marton Balint cus at passwd.hu
Sat Sep 18 23:12:19 EEST 2021



On Mon, 9 Aug 2021, Zhao Zhili wrote:

> SRTSOCKET is an abstraction designed by libsrt, it's not guaranteed
> to be a real file descriptor. Even if it is, it should not be
> operated directly outside of libsrt.
> ---
> libavformat/libsrt.c | 7 -------
> 1 file changed, 7 deletions(-)
>
> diff --git a/libavformat/libsrt.c b/libavformat/libsrt.c
> index e5701625b8..9aeaa8eb40 100644
> --- a/libavformat/libsrt.c
> +++ b/libavformat/libsrt.c
> @@ -706,12 +706,6 @@ static int libsrt_close(URLContext *h)
>     return 0;
> }
>
> -static int libsrt_get_file_handle(URLContext *h)
> -{
> -    SRTContext *s = h->priv_data;
> -    return s->fd;
> -}
> -
> static const AVClass libsrt_class = {
>     .class_name = "libsrt",
>     .item_name  = av_default_item_name,
> @@ -725,7 +719,6 @@ const URLProtocol ff_libsrt_protocol = {
>     .url_read            = libsrt_read,
>     .url_write           = libsrt_write,
>     .url_close           = libsrt_close,
> -    .url_get_file_handle = libsrt_get_file_handle,
>     .priv_data_size      = sizeof(SRTContext),
>     .flags               = URL_PROTOCOL_FLAG_NETWORK,
>     .priv_data_class     = &libsrt_class,

Thanks, applied.

Regards,
Marton


More information about the ffmpeg-devel mailing list