Re: [FFmpeg-devel] [PATCH] mlp_parser: fetch a new timestamp when major sync is found

2022-03-07 Thread Jan Ekström
On Mon, Mar 7, 2022 at 7:10 PM Michael Niedermayer wrote: > > On Mon, Mar 07, 2022 at 12:32:59AM +0200, Jan Ekström wrote: > > From: Hendrik Leppkes > > > > Decoding can only start on a major sync > > > and the parser discards > > any data received until that point. > > Thats a bug and its

Re: [FFmpeg-devel] [PATCH] mlp_parser: fetch a new timestamp when major sync is found

2022-03-07 Thread Hendrik Leppkes
On Mon, Mar 7, 2022 at 6:10 PM Michael Niedermayer wrote: > So why is this considered to be "ok" in audio codecs? > Because we dont visually see that data is lost ? This patch is not trying to argue that its "ok" to drop data, nor does it introduce this behavior. Someone else did that years ago.

Re: [FFmpeg-devel] [PATCH] mlp_parser: fetch a new timestamp when major sync is found

2022-03-07 Thread Michael Niedermayer
On Mon, Mar 07, 2022 at 12:32:59AM +0200, Jan Ekström wrote: > From: Hendrik Leppkes > > Decoding can only start on a major sync > and the parser discards > any data received until that point. Thats a bug and its inconsistant For example video codec parsers do not drop frames until a keyframe

[FFmpeg-devel] [PATCH] mlp_parser: fetch a new timestamp when major sync is found

2022-03-06 Thread Jan Ekström
From: Hendrik Leppkes Decoding can only start on a major sync and the parser discards any data received until that point. Therefore, previously received timestamps are no longer relevant to the data being returned from the parser. Additionally, adds a test which remuxes an existing TrueHD