[FFmpeg-devel] [PATCH v3 3/5] avformat/hls: relay programs to child muxers

[Gyan Doshi]

On 2023-02-18 03:31 am, Marton Balint wrote:
>
>
> On Fri, 17 Feb 2023, Gyan Doshi wrote:
>
>> ---
>> libavformat/hlsenc.c | 8 ++++++++
>> 1 file changed, 8 insertions(+)
>>
>> diff --git a/libavformat/hlsenc.c b/libavformat/hlsenc.c
>> index e1f96feda3..cf2b36da01 100644
>> --- a/libavformat/hlsenc.c
>> +++ b/libavformat/hlsenc.c
>> @@ -911,6 +911,14 @@ FF_ENABLE_DEPRECATION_WARNINGS
>>         st->id = vs->streams[i]->id;
>>     }
>>
>> +    for (i = 0; i < s->nb_programs; i++) {
>> +        ret = av_program_copy(oc, (const AVFormatContext *)s, 
>> s->programs[i]->id, 0);
>> +        if (ret < 0) {
>> +            av_log(s, AV_LOG_WARNING, "unable to transfer program %d 
>> to child muxer\n", s->programs[i]->id);
>
> Aren't these errors supposed to be propagated back as hard failures, 
> instead of simply ignoring them? The next 2 patches also has this 
> problem.


Updated in v4.

Regards,
Gyan