Rockbox

  • Status Closed
  • Percent Complete
    100%
  • Task Type Patches
  • Category Codecs
  • Assigned To No-one
  • Operating System All players
  • Severity Low
  • Priority Very Low
  • Reported Version Daily build (which?)
  • Due in Version Undecided
  • Due Date Undecided
  • Votes
  • Private
Attached to Project: Rockbox
Opened by Yusaku_Inui - 2011-07-09
Last edited by learman - 2011-07-24

FS#12185 - Fix gapless playback for Nero AAC

This patch is fix to work gapless playback for AAC encoded by Nero AAC (Ver.1.3.3.0) and FAAC.

   aac.patch (1.2 KiB)
Closed by  learman
2011-07-24 13:06
Reason for closing:  Fixed
Additional comments about closing:   Warning: Undefined array key "typography" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 371 Warning: Undefined array key "camelcase" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 407

Fixed in r30149.

Yusaku_Inui commented on 2011-07-09 11:53

Fix compile error.

   aac.patch (1 KiB)
learman commented on 2011-07-16 08:02

Could you explain the patch a bit, please? To me it looks like it re-uses the lead_trim at the end of the file if no tail_trim has been specified, and that doesn’t look right (the MP4 container already allows the last frame to be properly trimmed, so it shouldn’t be necessary for FAAC/Nero at least).

Yusaku_Inui commented on 2011-07-16 14:27

I’m not sure.
But after r29727, unable gapless playback.
So I undid the point of deleted in r29727.

I have tested this file.
This file was encoded by FAAC and Nero 1.3.3.0.

   test1_faac.m4a (418.3 KiB)
   test2_faac.m4a (488.9 KiB)
   test1_nero.m4a (430.9 KiB)
   test2_nero.m4a (473.9 KiB)
learman commented on 2011-07-16 16:27

Hm, yes, in the decode of test1_nero (using test_codec), there is a small piece of silence. Not much, 0.01s or so, and it is inaudible to me (the test track I used also get some silence, but only 0.003s). Foobar2000 does a good job, so there’s room for improvement. There’s got to be a better way though…

learman commented on 2011-07-17 09:44

I see what you mean now. r29727 removed the last frame trimming used by FAAC/Nero files. I’ll commit a fix soon.

Yusaku_Inui commented on 2011-07-17 15:58

r30149 is very nice.
Sounds fine some test files.
Thanks.

Loading...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing