Re: [PATCH v2 1/6] Bypass GCC attributes on NonStop platform where used.

From: Ramsay Jones <ramsay@ramsayjones.plus.com>
Date: Jan 19, 2018, 20:28 UTC
Message-ID: <86c7345f-814d-8d81-bb55-e91fe630e118@ramsayjones.plus.com>
In-Reply-To: <20180119173406.13324-3-randall.s.becker@rogers.com>

On 19/01/18 17:34, randall.s.becker@rogers.com wrote:

Show 22 quoted lines

> From: "Randall S. Becker" <rsbecker@nexbridge.com>
> 
> * remote.c: force ignoring of GCC __attribute construct not supported
>   by c99 by defining it as an empty CPP macro.
> 
> Signed-off-by: Randall S. Becker <rsbecker@nexbridge.com>
> ---
>  remote.c | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/remote.c b/remote.c
> index 4e93753e1..c18f9de7f 100644
> --- a/remote.c
> +++ b/remote.c
> @@ -11,6 +11,10 @@
>  #include "mergesort.h"
>  #include "argv-array.h"
>  
> +#if defined (__TANDEM)
> +#define __attribute(a)
> +#endif
> +

Hmm, the only use of __attribute() I can find is in compat/regex/. In particular, there is no use of __attribute() in regex.c. [__attribute__() is used in regex.c]

Is this an old patch which is no longer required?

puzzled.

ATB, Ramsay Jones

>  enum map_direction { FROM_SRC, FROM_DST };
>  
>  static struct refspec s_tag_refspec = {
>