After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 700746 - Use 'dumb quotes' rather than `really dumb quotes'
Use 'dumb quotes' rather than `really dumb quotes'
Status: RESOLVED FIXED
Product: glib
Classification: Platform
Component: general
unspecified
Other All
: Normal normal
: ---
Assigned To: gtkdev
gtkdev
Depends on:
Blocks:
 
 
Reported: 2013-05-20 21:20 UTC by Dan Winship
Modified: 2013-05-21 14:40 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Use 'dumb quotes' rather than `really dumb quotes' (191.21 KB, patch)
2013-05-20 21:20 UTC, Dan Winship
committed Details | Review

Description Dan Winship 2013-05-20 21:20:24 UTC
This always annoys me. While adding some g_test_expect_message()s to
NM I decided to fix it.

This will almost certainly break some other peoples' use of
g_test_expect_message(), but fortunately there aren't that many people
using it yet, so now is the time to do it, and it's easy to fix in a
way that works with or without this commit (by just skipping over the
quotes in the pattern to be matched, eg "*assertion*foo > 0*failed*")

Oh, I should probably fix the .po files before committing too.

(Done with perl, so if everyone else hates this idea, it's not like I
spent a ton of time on it :-)
Comment 1 Dan Winship 2013-05-20 21:20:33 UTC
Created attachment 244863 [details] [review]
Use 'dumb quotes' rather than `really dumb quotes'

Back in the far-off twentieth century, it was normal on unix
workstations for U+0060 GRAVE ACCENT to be drawn as "‛" and for U+0027
APOSTROPHE to be drawn as "’". This led to the convention of using
them as poor-man's ‛smart quotes’ in ASCII-only text.

However, "'" is now universally drawn as a vertical line, and "`" at a
45-degree angle, making them an `odd couple' when used together.

Unfortunately, there are lots of very old strings in glib, and also
lots of new strings in which people have kept up the old tradition,
perhaps entirely unaware that it used to not look stupid.

Fix this by just using 'dumb quotes' everywhere.
Comment 2 Colin Walters 2013-05-20 21:28:34 UTC
Review of attachment 244863 [details] [review]:

I am in favor of this, but something like this should probably get at least one other reviewer to +1.
Comment 3 Matthias Clasen 2013-05-20 21:47:37 UTC
yes, please. I hate the odd couple very much
Comment 4 Dan Winship 2013-05-21 14:40:11 UTC
Attachment 244863 [details] pushed as 4b94c08 - Use 'dumb quotes' rather than `really dumb quotes'