Rockbox

Tasklist

FS#2740 - revised replay gain tags

Attached to Project: Rockbox
Opened by Robert Kukla (roolku) - Tuesday, 08 November 2005, 17:05 GMT
Last edited by Nils Wallménius (nls) - Tuesday, 26 September 2006, 20:36 GMT
Task Type Patches
Category Themes
Status Closed
Assigned To No-one
Operating System
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

This 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.
This task depends upon

Closed by  Martin Scarratt (mmmm-)
Wednesday, 15 November 2006, 18:41 GMT
Reason for closing:  Fixed
Additional comments about closing:  %rg now does this
Comment by Robert Kukla (roolku) - Friday, 11 November 2005, 12:11 GMT

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>>
Comment by Robert Kukla (roolku) - Friday, 18 November 2005, 01:19 GMT

adapted to recent CVS changes
Comment by Anonymous Submitter - Friday, 13 January 2006, 12:41 GMT

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>>
Comment by Flid (Flid) - Wednesday, 12 April 2006, 17:55 GMT
I've just realised I've commented in an older version of the crossfade patch. I would love this to be applied to CVS. Does it still work with recent CVS due to the addition of RCT?

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.
Comment by Nils Wallménius (nls) - Tuesday, 26 September 2006, 20:41 GMT
I reopened this patch because parts of it wasn't committed namely the %rt and %ra tags.

Loading...