[FFmpeg-devel] [PATCH v2] avformat/mpegts: set data broadcast streams as such

[Marton Balint]

On Thu, 14 Apr 2022, Jan Ekström wrote:

> On Mon, Apr 11, 2022 at 1:50 PM Jan Ekström <jeebjp at gmail.com> wrote:
>>
>> From: Jan Ekström <jan.ekstrom at 24i.com>
>>
>> Additionally, they should not be probed, as this is essentially
>> various types of binary data.
>>
>> Signed-off-by: Jan Ekström <jan.ekstrom at 24i.com>
>> ---
>
> Ping.
>
> Basically this checks if we have an unknown stream with a private
> stream type still at the end of the per-stream loop in PMT parsing,
> and then cancels the stop of parsing that usually occurs as a PMT is
> hit. Instead the logic will continue parsing further. When an SDT is
> then found and a PMT for that program has already been received, it
> will then stop header reading at that point.

But why does it matter when the initial parsing is stopped? I mean it 
stops at the first PMT right now, nobody expects it to find all the 
programs and all the streams or all the stream codecs/parameters.

I am saying, that ideally, the ts->stop_parse magic should not be needed 
to be changed to fix your issue and when an SDT is detected with the 
broadcast descriptor that should stop any existing data stream parsing. Do 
you know why it does not work like that?

Thanks,
Marton