* Re: [FFmpeg-devel] [PATCH] avcodec/wmadec: fix WMA gapless playback
[not found] <20210911194900.17480-1-onemda@gmail.com>
@ 2022-04-28 10:58 ` Mika Fischer
0 siblings, 0 replies; only message in thread
From: Mika Fischer @ 2022-04-28 10:58 UTC (permalink / raw)
To: ffmpeg-devel
Hi,
I think this patch, which was applied on 2021-09-12 in
19802d170a304f5853d92e01d0513b9e06897d61 and is included in n5.0 and
n5.0.1 causes a regression.
This is the (shortened) diff for the test decoding in the patch:
----------------
diff --git a/tests/ref/fate/flcl1905 b/tests/ref/fate/flcl1905
index 5f5245ebcf..d702139db8 100644
--- a/tests/ref/fate/flcl1905
+++ b/tests/ref/fate/flcl1905
@@ -1,6 +1,4 @@
packet|[...]|pts=0|pts_time=0.000000|[...]
-frame|[...]|pts=0|pts_time=0.000000|[...]
-frame|[...]|pts=N/A|pts_time=N/A|[...]
frame|[...]|pts=N/A|pts_time=N/A|[...]
----------------
As can be seen, the patch causes the first two frames to be dropped,
which might be OK to better conform to the MS decoder.
The problem is that only the first frame had the timestamp info from the
packet. Since that got dropped, now the first frame decoded from the
packet has no timestamp info.
This causes us issues since we assumed that the first audio frame
decoded after seeking always has timestamp info.
Best regards,
Mika Fischer
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
^ permalink raw reply [flat|nested] only message in thread