Rockbox.org home
release
dev builds
extras
themes manual
wiki
device status forums
mailing lists
IRC bugs
patches
dev guide



Rockbox mail archive

Subject: Re: Announce: id3v2 patch 706111 updated
From: Henrik Backe (backe_at_swipnet.se)
Date: 2003-05-23


"TP Diffenbach" wrote in message > I've updated patch 706111 with diffs to
cvs 2003-05-22, and corrected the diffs to not include unchanged portions of
the files diffed.
>
> Patch 706111 adds freeform genre, freeform year, freeform track number,
and composer to the Id3v2 information.
>

I'm only missing one thing, the ID3 browser does not use your freeform tags.
Feel free to add the following to your patch.

../Henrik

--- orig/apps/wps.c 2003-05-23 13:25:55.000000000 +0200
+++ dev/apps/wps.c 2003-05-23 14:35:16.000000000 +0200
@@ -186,19 +186,17 @@

             case 3:
                 lcd_puts(0, 0, str(LANG_ID3_TRACKNUM));
-
- if (id3->tracknum) {
- snprintf(scroll_text,sizeof(scroll_text), "%d",
- id3->tracknum);
- lcd_puts_scroll(0, 1, scroll_text);
- }
- else
- lcd_puts_scroll(0, 1, str(LANG_ID3_NO_TRACKNUM));
+ lcd_puts_scroll(0, 1,
+ id3->track_string ?
+ id3->track_string :
+ (char*)str(LANG_ID3_NO_TRACKNUM));
                 break;

             case 4:
                 lcd_puts(0, 0, str(LANG_ID3_GENRE));
                 lcd_puts_scroll(0, 1,
+ id3->genre_string ?
+ id3->genre_string :
                                 wps_get_genre(id3->genre) ?
                                 wps_get_genre(id3->genre) :
                                 (char*)str(LANG_ID3_NO_INFO));
@@ -206,13 +204,10 @@

             case 5:
                 lcd_puts(0, 0, str(LANG_ID3_YEAR));
- if (id3->year) {
- snprintf(scroll_text,sizeof(scroll_text), "%d",
- id3->year);
- lcd_puts_scroll(0, 1, scroll_text);
- }
- else
- lcd_puts_scroll(0, 1, str(LANG_ID3_NO_INFO));
+ lcd_puts_scroll(0, 1,
+ id3->year_string ?
+ id3->year_string :
+ (char*)str(LANG_ID3_NO_INFO));
                 break;

             case 6:



Page was last modified "Jan 10 2012" The Rockbox Crew
aaa