Merge lp:~larsu/indicator-datetime/dont-escape-unicode-chars into lp:indicator-datetime/13.10
Proposed by
Lars Karlitski
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | Charles Kerr | ||||
Approved revision: | 373 | ||||
Merged at revision: | 374 | ||||
Proposed branch: | lp:~larsu/indicator-datetime/dont-escape-unicode-chars | ||||
Merge into: | lp:indicator-datetime/13.10 | ||||
Diff against target: |
83 lines (+26/-20) 1 file modified
src/utils.c (+26/-20) |
||||
To merge this branch: | bzr merge lp:~larsu/indicator-datetime/dont-escape-unicode-chars | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
PS Jenkins bot (community) | continuous-integration | Approve | |
Charles Kerr (community) | Approve | ||
Sebastien Bacher | Approve | ||
Review via email: mp+233726@code.launchpad.net |
Commit message
Use unicode characters directly instead of escaping them.
Description of the change
Use unicode characters directly instead of escaping them
Launchpad's xgettext seems too old to properly transform the escaped characters. It passes literal characters right through.
To post a comment you must log in.
Thanks Lars. As discussed on IRC, that's probably the best we can do at the moment. Ideally we should fix launchpad to handle the \u<nnnn> chars properly, but that's not likely to be the case before rtm so we need a workaround. The solution listed there works, +1 from me