gmacros: Ensure GUINT32/64_SWAP_LE_BE macros parenthesize arguments
authorStephan Bergmann <sbergman@redhat.com>
Fri, 15 Mar 2013 16:30:24 +0000 (17:30 +0100)
committerColin Walters <walters@verbum.org>
Fri, 15 Mar 2013 16:51:49 +0000 (12:51 -0400)
commit255c65f83c1515a7e3fc4609b36e72acc08a79e4
tree90f061dfaf7d1463f8a7cce12ad81a5afe49aea7
parent55a235d377f8da416834156b89c7f5b98b02583b
gmacros: Ensure GUINT32/64_SWAP_LE_BE macros parenthesize arguments

Like all macros, we need to parenthesize arguments to ensure the order
of operations is correct.

See the mail thread starting at
<http://lists.fedoraproject.org/pipermail/devel/2013-March/180302.html>
"GCC produced wrong code in gvfs-1.14.2-3.fc18.x86_64" for how this
caused trouble with GVFS (which in turn caused trouble with
LibreOffice, where running "soffice sftp://.../.../test.odt" to access
an .odt file via GVFS failed to properly type-detect that file as a
Writer document and produced bogus error messages about the file being
broken).

https://bugzilla.gnome.org/show_bug.cgi?id=695925
glib/gtypes.h