Rockbox

This is the bug/patch tracker for Rockbox. Click here for more information.

Quick links: Bugs · Patches · Rockbox frontpage

Tasklist

FS#12149 - Read Lyricist/Text writer from mp3 id3 tag for display in wps

Attached to Project: Rockbox
Opened by Kaname Madoka (Lyrical Madoka) - Sunday, 05 June 2011, 10:45 GMT+2
Task Type Patches
Category Database
Status Unconfirmed
Assigned To No-one
Player Type All players
Severity Low
Priority Normal
Reported Version Daily build (which?)
Due in Version Undecided
Due Date Undecided
Percent Complete 0%
Private No

Details

This patch allows for the display id3 (v2.2-v2.4) tag
of Lyricist/Text writer in the wps, database, track info.

To display in wps, use "%iw" tag.

Example:
-----
# Disable Status Bar
%wd
# Title
%s%al%?it<%it|%fn>
# Artist
%s%al%?ia<%ia|%?d(2)<%d(2)|%(root%)>>
# Album
%s%al%?id<%id|%?d(1)<%d(1)|%(root%)>> %?iy<%(%iy%)|>
# Songwriter
%s%al%?iw<writer %iw|>
# Composer
%s%al%?ic<composer %ic|>
# Comment
%s%al%?iC<%iC|>
# Time PlayNumber
%al[%pc/%pt] %ar%(%pp/%pe%)
#%s%ac%Sx(Next:) %?It<%It|%Fn>
-----

Tested on sansa c200 v1.
Target source code revision is #29961.

The following files were changed:
apps/screens.c
apps/metadata/id3tags.c
apps/metadata/metadata_common.c
apps/lang/japanese.lang
apps/lang/english.lang
apps/gui/skin_engine/wps_debug.c
apps/gui/skin_engine/skin_tokens.c
apps/metadata.c
apps/metadata.h
apps/tagcache.c
apps/tagcache.h
apps/tagnavi.config
apps/tagtree.c
lib/skin_parser/tag_table.c
lib/skin_parser/tag_table.h

Currently, only English and Japanese languages are supported.

I am not good at English.
Please point it out if there are a problem.
   songwriter.29961.patch (17.1 KiB)
 apps/screens.c                     |    8 +-
 apps/metadata/id3tags.c            |    8 +-
 apps/metadata/metadata_common.c    |    4 +
 apps/lang/japanese.lang            |  117 ++++++++++++++++++++++++++++++++++++-
 apps/lang/english.lang             |   14 ++++
 apps/gui/skin_engine/wps_debug.c   |    5 +
 apps/gui/skin_engine/skin_tokens.c |    2 
 apps/metadata.c                    |    1 
 apps/metadata.h                    |    3 
 apps/tagcache.c                    |   27 ++++----
 apps/tagcache.h                    |   10 +--
 apps/tagnavi.config                |    4 -
 apps/tagtree.c                     |    2 
 lib/skin_parser/tag_table.c        |    8 +-
 lib/skin_parser/tag_table.h        |    1 
 15 files changed, 187 insertions(+), 27 deletions(-)

This task depends upon

Comment by PurlingNayuki (yzflcyq) - Saturday, 25 June 2011, 17:01 GMT+2
I don't think it's useful. Not all the MP3 file (Most of the MP3 files don't) contains those info.
Comment by Kaname Madoka (Lyrical Madoka) - Sunday, 03 July 2011, 18:45 GMT+2
I don't think so.
Songwriter, composer, and arranger are the basic information for pop songs.
If not contains those information in mp3 files, you can put it in.

Loading...