Re: [FFmpeg-devel] [PATCH] avformat/ffrtmpcrypt: Fix int-conversion warning

2024-01-04 Thread Martin Storsjö
On Fri, 22 Dec 2023, Frank Plowman wrote: Hi Martin, Thanks for the review. On 22/12/2023 12:15, Martin Storsjö wrote: The change LGTM, but the wording here is slightly confusing IMO. The problem isn't with using per se AVERROR, that's just a macro for generating suitable integers, the issue

Re: [FFmpeg-devel] [PATCH] avformat/ffrtmpcrypt: Fix int-conversion warning

2023-12-22 Thread Frank Plowman
Hi Martin, Thanks for the review. On 22/12/2023 12:15, Martin Storsjö wrote: The change LGTM, but the wording here is slightly confusing IMO. The problem isn't with using per se AVERROR, that's just a macro for generating suitable integers, the issue is more about the fact that we're returnin

Re: [FFmpeg-devel] [PATCH] avformat/ffrtmpcrypt: Fix int-conversion warning

2023-12-22 Thread Martin Storsjö
Hi Frank, On Fri, 22 Dec 2023, Frank Plowman wrote: The gcrypt definition of `bn_new` used to use `AVERROR`, however it is called in `dh_generate_key` and `ff_dh_init` which return pointers. As a result, compiling with gcrypt and the ffrtmpcrypt protocol resulted in an int-conversion warning. G