[FFmpeg-devel] [PATCH 1/2] avcodec/s302m: enable non-PCM decoding

[Anton Khirnov]

Hi,
sorry for the delay, I've been busy fixing things for the release
Quoting Gyan Doshi via ffmpeg-devel (2024-01-29 05:00:33)
> On 2024-01-28 04:24 pm, Anton Khirnov wrote:
> >> a) it would mean essentially inlining this decoder in the demuxer.
> > Why is that a problem? This decoder seems like it shouldn't be a
> > decoder.
> >
> > I agree with Andreas that this seems like it's a demuxer pretending to
> > be a decoder.
> 
> This module transforms the entire raw payload data to generate its 
> output, even if the syntax is simple which
> essentially makes it a de-coder. The de-multiplexer aspect of multiple 
> streams is an academic possibility allowed
> by the standard but not seen in any sample which makes me suspect it's 
> used for carriage between broadcast
> facilities rather than something ever sent to an OTT provider, let alone 
> an end user.

If it dynamically generates nested decoders, then it's not a proper
codec in our model. It should be either a part of the demuxer, or a
bitstream filter (possibly inserted automatically by the demuxer).

-- 
Anton Khirnov