Bug #124525 for Audio-Scan: ADTS AAC's frame count is one less

Thu Feb 22 00:56:10 2018 yuki.takada [...] d2cdot.co.jp - Ticket created

Subject:	ADTS AAC's frame count is one less
Date:	Thu, 22 Feb 2018 14:55:49 +0900
To:	bug-Audio-Scan [...] rt.cpan.org
From:	高田悠揮 <yuki.takada [...] d2cdot.co.jp>

ADTS AAC info's "song_length_ms" may have short duration. When i calculated duration from aac file's frame count that calculated with another program, it was short. - Audio::Scan 0.99 - v5.26.1 - CentOS 7.3.1611 (3.10.0-514.26.2.el7.x86_64)

Thu Feb 22 10:43:50 2018 AGRUNDMA [...] cpan.org - Correspondence added

Hi, Do you have any sample file(s) I can look at? Do you have the problem with all files or are some worse than others? What tool are you using to check the correct value? I couldn't find anything that does the right thing with raw ADTS files.

Thu Feb 22 10:43:50 2018 The RT System itself - Status changed from 'new' to 'open'

Fri Feb 23 21:14:10 2018 yuki.takada [...] d2cdot.co.jp - Correspondence added

Subject:	Re: [rt.cpan.org #124525] ADTS AAC's frame count is one less
Date:	Sat, 24 Feb 2018 11:13:56 +0900
To:	bug-Audio-Scan [...] rt.cpan.org
From:	高田悠揮 <yuki.takada [...] d2cdot.co.jp>

Hi, I prepared sample audio files and scan program. https://github.com/alf-ytakada/perl-audio-scan-issue rec_3frame.aac must have 128msec(3frames), but module info shows only 85 msec. I checked only HE-AAC 48kHz 48kbps sound. so i don't know other aac will show same things. 2018-02-23 0:43 GMT+09:00 Andy Grundman via RT <bug-Audio-Scan@rt.cpan.org>: Show quoted text

> <URL: https://rt.cpan.org/Ticket/Display.html?id=124525 > > > Hi, > > Do you have any sample file(s) I can look at? Do you have the problem with all files or are some worse than others? What tool are you using to check the correct value? I couldn't find anything that does the right thing with raw ADTS files.

-- 株式会社D2C dot 　開発部　　高田　悠揮　　　E-mail : yuki.takada@d2cdot.co.jp 〒104-0061 東京都中央区銀座6‐18‐2 野村不動産銀座ビル mobile TEL : 080-9507-0032（こちらにおかけくださいませ）（会社TEL：03-6226-8930）

Mon Apr 23 11:55:17 2018 AGRUNDMA [...] cpan.org - Correspondence added

Sorry for taking so long to fix this one, please let me know if this fixes your issue: https://github.com/andygrundman/Audio-Scan/commit/a18500aba7482e8edbc353a83a3d9580eccc809a