Rockbox

  • Status Closed
  • Percent Complete
    100%
  • Task Type Bugs
  • Category User Interface → Themes
  • Assigned To No-one
  • Operating System All players
  • Severity Low
  • Priority Very Low
  • Reported Version
  • Due in Version Undecided
  • Due Date Undecided
  • Votes
  • Private
Attached to Project: Rockbox
Opened by teeky - 2007-04-10

FS#7007 - Space padding of %ce tag

The tag %ce is space padded now which didn’t use to be. This padding does not make much sense as for example

%cb %ce

may cause double space, e.g.

Apr 9

The example in the wiki documentation is also incorrect, the given code produces

Sun, Jan 01, 04:10:42

Closed by  petur
2007-04-11 21:36
Reason for closing:  Not a Bug
Additional comments about closing:  

by design

teeky commented on 2007-04-10 19:30

OK, seems the two spaces in the example were removed after adding the bug report. Suppose that period (.) is the space. Then the displayed date is

Apr..9

I corrected the wiki, but what’s the problem with the space padding ? It used to be before so what you say isn’t correct.
You say it could cause double space, but that’s precisely the point of blank padding, so I don’t see where there’s a bug.

teeky commented on 2007-04-11 21:21

Ah, you are right, I hadn’t noticed this behaviour before.

Anyway, I don’t understand the purpose of the padding. Tags like %cl and %ck don’t have this padding, why %ce has? How can I get

month space day-of-month-without-leading-zero

which always contains only one space? As I said, it is not possible with %cb %ce.

petur commented on 2007-04-11 21:35

the code of the parser has the following:

WPS_TOKEN_RTC_DAY_OF_MONTH, “cd”,
WPS_TOKEN_RTC_DAY_OF_MONTH_BLANK_PADDED, “ce”,

so it seems to me %ce = %cd + padding

I’ll close this report, feel free to request it re-opened if you feel my understanding isn’t correct

Loading...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing