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 727320 - docs: Remove escaping '\' from literals
docs: Remove escaping '\' from literals
Status: RESOLVED FIXED
Product: glib
Classification: Platform
Component: docs
2.40.x
Other All
: Normal normal
: ---
Assigned To: gtkdev
gtkdev
Depends on:
Blocks:
 
 
Reported: 2014-03-30 00:16 UTC by Volker Sobek (weld)
Modified: 2014-03-30 12:39 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
docs: Remove escaping '\' from literals (3.10 KB, patch)
2014-03-30 00:16 UTC, Volker Sobek (weld)
none Details | Review
docs: Remove escaping '\' from literals (3.61 KB, patch)
2014-03-30 00:26 UTC, Volker Sobek (weld)
committed Details | Review

Description Volker Sobek (weld) 2014-03-30 00:16:56 UTC
commit 35066ed6c6b51317f49069f2564c547aa309f9f1 replaced entities, but
escaped the replacement text also inside literals, which resulted in the
escaping '\' to also appear in the documentation.

Command used to find possibly wrong escapes in literals:
find . -name '*.c' -exec grep -n '\*.*`.*\\[^[:alnum:]].*`' '{}' + | \
grep --color '\\[^[:alnum:]]'
Comment 1 Volker Sobek (weld) 2014-03-30 00:16:59 UTC
Created attachment 273264 [details] [review]
docs: Remove escaping '\' from literals
Comment 2 Volker Sobek (weld) 2014-03-30 00:26:51 UTC
Created attachment 273265 [details] [review]
docs: Remove escaping '\' from literals

commit 35066ed6c6b51317f49069f2564c547aa309f9f1 replaced entities, but
escaped the replacement text also inside literals, which resulted in the
escaping '\' to also appear in the documentation.
Comment 3 Colin Walters 2014-03-30 12:36:55 UTC
Review of attachment 273265 [details] [review]:

Makes sense to me, thanks for the patch!
Comment 4 Colin Walters 2014-03-30 12:39:44 UTC
Attachment 273265 [details] pushed as a0b932a - docs: Remove escaping '\' from literals