- Status Closed
- Percent Complete
- Task Type Patches
- Category ID3 / meta data
- Assigned To No-one
- Operating System
- Severity Low
- Priority Very Low
- Reported Version
- Due in Version Undecided
-
Due Date
Undecided
- Votes
- Private
FS#792 - Add support for APE tags
This patch adds support for so called APE tags, which
are simple but flexible tags located at the end of a
file. I decided to add support because Foobar2000
(http://foobar2000.hydrogenaudio.org/) uses it as it’s
main tag format (ID3V2 isn’t supported).
Note that this patch hasn’t been tested much. This is
especially true for the UTF-8 reader… It works on a
couple of test files though.
ape.h
ape.zip
Closed by zagor
2003-06-04 12:25
Reason for closing: Rejected
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
2003-06-04 12:25
Reason for closing: Rejected
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
Logged In: YES
user_id=1157
Too esoteric for inclusion in mainline
Rockbox.
Loading...
Available keyboard shortcuts
- Alt + ⇧ Shift + l Login Dialog / Logout
- Alt + ⇧ Shift + a Add new task
- Alt + ⇧ Shift + m My searches
- Alt + ⇧ Shift + t focus taskid search
Tasklist
- o open selected task
- j move cursor down
- k move cursor up
Task Details
- n Next task
- p Previous task
- Alt + ⇧ Shift + e ↵ Enter Edit this task
- Alt + ⇧ Shift + w watch task
- Alt + ⇧ Shift + y Close Task
Task Editing
- Alt + ⇧ Shift + s save task
OK, here’s a new version of the APE tag patch. Changes:
* Modifies mpeg.c to make it support gapless playback of
PE-tagged files. Tested it with 20 kB tags (on –nogap files
created by Lame); worked just fine.
* Should better handle the case where the current read does
not load the entire tag, and the part loaded in the previous
read (which is likely to be in the MP3 buffer) has already
been bitswapped. I say should, because it is a bit tricky to
test this properly. ;)
This means that perhaps the code commented like this should
be removed: “/* The very rare case when the entire tag
wasn’t read in this read() call must be taken care of */”…
* Modifies app.lds. Without the last .align, I got a binary
that was an odd number of bytes long. The scrambler doesn’t
like that. (The current patch may or may not need them; in
any case, they won’t hurt.)
* Added support for Year tag. Genre isn’t supported (yet),
since it is a free-text string (which may or may not match
one of the pre-defined genres).
* Available in an unzipped version. :)
Unfortunately, I need to use the -b option (diff finds lots
of changes
otherwise), so the indentation is wrong in some places…
Too esoteric for inclusion in mainline Rockbox.