This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: Fix typo on cygming.h
- From: Kai Tietz <ktietz70 at googlemail dot com>
- To: Nick Clifton <nickc at redhat dot com>
- Cc: gcc-patches at gcc dot gnu dot org
- Date: Tue, 22 Mar 2011 10:07:40 +0100
- Subject: Re: Fix typo on cygming.h
- References: <m3tyevlec0.fsf@redhat.com>
2011/3/22 Nick Clifton <nickc@redhat.com>:
> Hi Guys,
>
> ?I am applying the patch below as an obvious fix. ?The definition of
> ?SUBTARGET_OVERRIDE_OPTIONS in the i386/cygming.h header file has a
> ?trailing backslash at the end of the macro, leading it to include the
> ?line that follows. ?Fortunately that line is blank, but could change
> ?one day, so it is best to clean up the macro definition.
>
> ?Tested by building an i686-pc-cygwin toolchain.
>
> Cheers
> ?Nick
>
> gcc/ChangeLog
> 2011-03-22 ?Nick Clifton ?<nickc@redhat.com>
>
> ? ? ? ?* config/i386/cygming.h (SUBTARGET_OVERRIDE_OPTIONS): Remove
> ? ? ? ?trailing backslash from the end of the macro definition.
>
> Index: gcc/config/i386/cygming.h
> ===================================================================
> --- gcc/config/i386/cygming.h ? (revision 171279)
> +++ gcc/config/i386/cygming.h ? (working copy)
> @@ -207,7 +207,7 @@
> ? ? ? ? ? ? ? (flag_pic > 1) ? "PIC" : "pic"); ? ? ? ? ? ? ? ? ? ? ? ? \
> ? ? ? flag_pic = 0; ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?\
> ? ? } ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?\
> -} while (0) ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?\
> +} while (0)
>
> ?/* Define this macro if references to a symbol must be treated
> ? ?differently depending on something about the variable or
Thanks for catching this.
Cheers,
Kai