[FFmpeg-devel] [PATCH v5 3/3] avformat/whip: fix typos

[Jack Lau]

Hi Timo,
> On Jun 29, 2025, at 06:49, Timo Rothenpieler <timo at rothenpieler.org> wrote:
> 
> I've actually cleaned this up a bit while trying to implement DTLS via schannel, and effectively removed the whole section:
> 
>> https://github.com/BtbN/FFmpeg/commit/b6794f1373fb07b791cfacd2189c7efc4d6bdbcc
> 
> There's also a bunch of other necessary UDP related fixes in tls.c.
> 
> Don't try to use an http proxy for UDP. It doesn't work:
>> https://github.com/BtbN/FFmpeg/commit/709ce9e5c48e3a27a400cf5af35038d3f0602c8a
I totally agree with the above two patches, it’s very reasonable
> 
> Properly forward the various hosts and ports to udp.c so it actually works when using a non-external UDP connection:
>> https://github.com/BtbN/FFmpeg/commit/46375adf7d9cc61f709ab14dd2ea017995f735db
But I think this patch need a bit modify, I think the c->listen stands for FFmpeg if is server. When it is true, FFmpeg as server. 
So maybe:
        if (c->listen) {
            av_dict_set_int(options, "localport", port, 0);
            av_dict_set(options, "localaddr", c->underlying_host, 0);Add commentMore actions
        } else {
            av_dict_set_int(options, "connect", 1, 0);
        }
What do you think?
If you don’t mind, I’m glad to fix this and submit it to mail list.

BTW, do you have other comments for this patchset https://ffmpeg.org/pipermail/ffmpeg-devel/2025-June/345948.html  <https://ffmpeg.org/pipermail/ffmpeg-devel/2025-June/345948.html?> ?
I think we can merge that firstly because this patchset is simple and I want to submit WHIP NACK patch that depends these patch.

Best regards
Jack
> 
> 
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> 
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".