Bug 521552 - Malformed string in pulseaudio
Summary: Malformed string in pulseaudio
Keywords:
Status: CLOSED UPSTREAM
Alias: None
Product: Fedora
Classification: Fedora
Component: pulseaudio
Version: rawhide
Hardware: All
OS: Linux
low
medium
Target Milestone: ---
Assignee: Lennart Poettering
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2009-09-06 22:27 UTC by Miloš Komarčević
Modified: 2009-09-07 21:35 UTC (History)
3 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2009-09-07 21:35:30 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Miloš Komarčević 2009-09-06 22:27:06 UTC
gettext tools report the following warning:

internationalized messages should not contain the `\r' escape sequence

The string in question is:

#: ../src/utils/pacat.c:577
#, c-format
msgid "Time: %0.3f sec; Latency: %0.0f usec.  \r"

Comment 1 Lennart Poettering 2009-09-07 14:26:01 UTC
But the \r is actually intended to be there (to move the cursor back to the beginning of the line). I really don't see what the point would be to drop that.

Comment 2 Miloš Komarčević 2009-09-07 16:37:04 UTC
By all means leave that control sequence in the code if that is the intended behaviour, but you could move it out from the translatable string, i.e. break those up. That way you can guarantee the intended behaviour and not risk some translator removing control sequences because gettext complains - separating real language content from functionality, makes sense, no?

Comment 3 Lennart Poettering 2009-09-07 21:35:30 UTC
Ok. Convinced.

Fixed upstream. Will hit rawhide shortly.


Note You need to log in before you can comment on or make changes to this bug.