On Sat, Feb 17, 2024 at 05:15:43PM +0530, Gyan Doshi wrote:
> Issue:
> 
> Patch: avcodec/s302m: enable non-PCM decoding
> URL: https://patchwork.ffmpeg.org/project/ffmpeg/patch/20240127103854.9971-1-ffmpeg@gyani.pro/
> 
> The issue needing resolution is whether the patch should be added to the
> existing s302m decoder or should that decoder
> be removed and all old and new patched features inlined into the mpeg-ts
> demuxer.

[Nested decoder, bsf, demuxer, something else ...]

I think the discussion is too much looking at the fine details

What we need is
TS -> raw samples
TS -> TS    (lossless)
TS -> nonTS (lossless)
nonTS -> TS (lossless)
raw samples -> TS

I think some of the discussed options have problems with lossless non PCM
in some intermediate encapsulation in mpeg-TS

thx

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I do not agree with what you have to say, but I'll defend to the death your
right to say it. -- Voltaire