Hi!

2016-09-24 14:58 GMT+02:00 Ronald S. Bultje <rsbul...@gmail.com>:
>
> On Fri, Sep 23, 2016 at 6:00 PM, Carl Eugen Hoyos <ceho...@ag.or.at> wrote:
>>
>> I use this patch locally for some time. It fixes
>> many misdetections, a similar patch was applied to
>> loas detection three years ago.
>> An alternative would be to return "1" for two frames.
>
>
> Is there any point in detecting two AAC frames? I mean, we're
> talking about a split second audio...

True, but I believe if it is the beginning of the file, a positive
score should be returned, so new patch attached.

Thank you, Carl Eugen
From ab91c039600b4518dc34a61d42176b1d68b814a1 Mon Sep 17 00:00:00 2001
From: Carl Eugen Hoyos <ceho...@ag.or.at>
Date: Sat, 24 Sep 2016 15:09:27 +0200
Subject: [PATCH] lavf/aacdec: Do not autodetect a single frame inside the
 file.

---
 libavformat/aacdec.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/libavformat/aacdec.c b/libavformat/aacdec.c
index aa23756..fecb1e3 100644
--- a/libavformat/aacdec.c
+++ b/libavformat/aacdec.c
@@ -70,7 +70,7 @@ static int adts_aac_probe(AVProbeData *p)
         return AVPROBE_SCORE_EXTENSION;
     else if (max_frames >= 3)
         return AVPROBE_SCORE_EXTENSION / 2;
-    else if (max_frames >= 1)
+    else if (first_frames >= 1)
         return 1;
     else
         return 0;
-- 
1.7.10.4

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to