Rockbox

Tasklist

FS#2556 - wps alignment patch

Attached to Project: Rockbox
Opened by Per Holmäng (perholmang) - Friday, 01 July 2005, 22:32 GMT
Last edited by Christi Scarborough (christi-s) - Tuesday, 05 July 2005, 22:29 GMT
Task Type Patches
Category Themes
Status Closed
Assigned To Christi Scarborough (christi-s)
Operating System
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 0%
Votes 0
Private No

Details

Patch to enable alignment settings in wps files.

Syntax:

%al - align left (defaults)
%ar - align right
%ac - align center

Put this before the item you wish to align. For example:

%al%ia - Left aligned Artist

If you want to use scrolling, use %s before the alignment:

%s%ac%ia - Center aligned scrolling Artist

This task depends upon

Closed by  Christi Scarborough (christi-s)
Tuesday, 05 July 2005, 22:29 GMT
Reason for closing:  Accepted
Additional comments about closing:  Logged In: YES
user_id=1077278

Commited to CVS. Thank you. When submitting future
patches, please ensure that they do not contain tab characters.
Comment by Christi Scarborough (christi-s) - Friday, 01 July 2005, 23:38 GMT

You don't seem to have attached the patch.
Comment by Per Holmäng (perholmang) - Friday, 01 July 2005, 23:48 GMT

Fixed now.
Comment by Christi Scarborough (christi-s) - Tuesday, 05 July 2005, 01:32 GMT

switch (*fmt)
@@ -795,12 +798,30 @@
case 0:
The following addition to wps-display.c appears to be
redundant. Is there a reason why you included this?

<pre> *buf++ = '%';
break;
-
+ case '(':
+ ++fmt;
+ break;
case 's':
*flags |= WPS_REFRESH_SCROLL;
</pre>
Comment by Per Holmäng (perholmang) - Tuesday, 05 July 2005, 10:14 GMT

No, I just forgot to clean that part up, it's been removed
now, hopefully. I've never worked with diff/patch before...
Comment by Christi Scarborough (christi-s) - Tuesday, 05 July 2005, 22:29 GMT

Commited to CVS. Thank you. When submitting future
patches, please ensure that they do not contain tab characters.

Loading...