This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
Re: [gold patch]Fix gold build on MacOSX Xcode 3.2.6
- From: Doug Kwan (éæå) <dougkwan at google dot com>
- To: Jing Yu <jingyu at google dot com>
- Cc: binutils at sourceware dot org, Andrew Hsieh <andrewhsieh at google dot com>
- Date: Tue, 5 Jun 2012 12:00:41 -0700
- Subject: Re: [gold patch]Fix gold build on MacOSX Xcode 3.2.6
- References: <CAJ_rGWT3KK1eXBoOhX9JUE89HR7RSPz9cVyMmt3u_vy0p6mHew@mail.gmail.com>
Please change this "do {} while(0)".
On Tue, Jun 5, 2012 at 10:59 AM, Jing Yu <jingyu@google.com> wrote:
> The gcc 4.2.1 (Apple Inc. build 5666, dot 3) comes with Xcode 3.2.6
> emits warning for "while (0);" and breaks the build (thanks to -Werror
> which treat warning as error). ?Add "{}" to fix it.
>
> Gold make check passes on x86-64.
>
> 2012-06-05 ?Jing Yu ?<jingyu@google.com>
>
> ? ? ? ?* gold.h (textdomain): Add {} to empty while(0)
> ? ? ? ? ? ? ? ? ? ? ?(bindtextdomain): Likewise
>
> Index: gold/gold.h
> ===================================================================
> RCS file: /cvs/src/src/gold/gold.h,v
> retrieving revision 1.47
> diff -u -r1.47 gold.h
> --- gold/gold.h 8 Jun 2011 04:43:28 -0000 ? ? ? 1.47
> +++ gold/gold.h 5 Jun 2012 17:46:46 -0000
> @@ -58,8 +58,8 @@
> ?# define gettext(Msgid) (Msgid)
> ?# define dgettext(Domainname, Msgid) (Msgid)
> ?# define dcgettext(Domainname, Msgid, Category) (Msgid)
> -# define textdomain(Domainname) while (0) /* nothing */
> -# define bindtextdomain(Domainname, Dirname) while (0) /* nothing */
> +# define textdomain(Domainname) while (0) {} /* nothing */
> +# define bindtextdomain(Domainname, Dirname) while (0) {} /* nothing */
> ?# define _(String) (String)
> ?# define N_(String) (String)
> ?#endif