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

[Devin Heitmueller]

On Tue, Jan 23, 2024 at 4:05 AM Kieran Kunhya <kierank at obe.tv> wrote:
> Yes, and the other uncertainty is that the PTS must be cotimed with the
> video so you have to guess whether it's the previous PTS or the next one.

Isn't the correct behavior to determine the offset within the raw PCM
payload, and use that to determine the actual PTS of the embedded
non-PCM packet?  Unless I"m misreading the code, it seems like it's
simply setting the PTS of the non-pcm packet to be the PTS of the
original S302 packet, which will result in incorrect A/V sync.

Devin

-- 
Devin Heitmueller, Senior Software Engineer
LTN Global Communications
o: +1 (301) 363-1001
w: https://ltnglobal.com  e: devin.heitmueller at ltnglobal.com