This is the bug/patch tracker for Rockbox. Click here for more information.
Quick links: Bugs · Patches · Rockbox frontpage
FS#2740 - revised replay gain tags
Attached to Project:
Rockbox
Opened by Robert Kukla (roolku) - Tuesday, 08 November 2005, 18:05 GMT+2
Last edited by Nils Wallménius (nls) - Tuesday, 26 September 2006, 22:36 GMT+2
Opened by Robert Kukla (roolku) - Tuesday, 08 November 2005, 18:05 GMT+2
Last edited by Nils Wallménius (nls) - Tuesday, 26 September 2006, 22:36 GMT+2
|
DetailsThis patch replaces 1295785 and 1290401 to streamline
the way that Replaigain Information is presented in the wps. (Since 1290401 also included the crossfade patch, it is included here as well - when the crossfade settings are finalised I can make a new more comprehensive patch.) Three new tags are introduced: %rg - Replaygain type (off, track, album, track when shuffle) %rt - Replaygain value for track %ra - Replaygain value for album Suggested usage is: %?rg<RPG disabled|RPG enabled> %?rg<RPG disabled|RPG track|RPG album|RPG track when shuffle> %?rg<RPG disabled|%?rt<%rt|no track RPG>|%?ra<% ra|no album RPG>|%?ps<%?rt<%rt|no track RPG>|%? ra<%ra|no album RPG>>> You can of course replace the text with bmps where appropriate. 
wps_crossfade.patch
(1.1 KiB)
apps/gui/gwps-common.c | 13 +++++++++++++ 1 file changed, 13 insertions(+)
wps_rpg.patch
(1.7 KiB)
apps/gui/gwps-common.c | 15 +++++++++++++-- 1 file changed, 13 insertions(+), 2 deletions(-) |
This task depends upon
Closed by Martin Scarratt (mmmm-)
Wednesday, 15 November 2006, 19:41 GMT+2
Reason for closing: Fixed
Additional comments about closing: %rg now does this
Wednesday, 15 November 2006, 19:41 GMT+2
Reason for closing: Fixed
Additional comments about closing: %rg now does this
Added modified crossfade patch according to pilot000
(http://forums.rockbox.org/index.php?topic=1777.0).
Suggested usage is:
%?cf<|%?ps<%?ct<x-fade|mix>>|%?ct<x-fade|mix>>
adapted to recent CVS changes
the 12th line in the crossfade patch needs to be changed
into:
+ case 'C':
or it will conflict with the rtc_wps.patch
because of this, the syntax must contain uppercase C
instead of c. like this:
%?Cf<|%?ps<%?Ct<x-fade|mix>>|%?Ct<x-fade|mix>>
I'm very interested, as this feature makes for some nice BMP action in graphical WPS, which I'm working on.
I'm gonna bump this subject on the forums aswell.