Re: [hwloc-devel] Attribute unsed not regognized

2010-03-26 Thread Samuel Thibault
Bert Wesarg, le Fri 26 Mar 2010 19:39:51 +0100, a écrit : > On Fri, Mar 26, 2010 at 17:01, Samuel Thibault > wrote: > > Thanks for the idea. > > > > Bert Wesarg, le Fri 26 Mar 2010 12:33:00 +0100, a écrit : > >> +#define HWLOC_HAVE(what) (defined(HWLOC_HAVE_##what) &&

Re: [hwloc-devel] Attribute unsed not regognized

2010-03-26 Thread Samuel Thibault
Bert Wesarg, le Fri 26 Mar 2010 12:43:33 +0100, a écrit : > FYI: I don't know if this is hwloc or autotools specific, but there is > no build dependency on > include/hwloc/config.h.in. It'd tend to say that it is autotools-specific. For instance, we do not put anything special for

Re: [hwloc-devel] Attribute unsed not regognized

2010-03-26 Thread Samuel Thibault
Samuel Thibault, le Fri 26 Mar 2010 15:49:36 +0100, a écrit : > All these don't have any problem with the above. I mean, with putting the attributes after the variable name. Samuel

Re: [hwloc-devel] Attribute unsed not regognized

2010-03-26 Thread Samuel Thibault
Bert Wesarg, le Fri 26 Mar 2010 11:09:05 +0100, a écrit : > AFAIK the correct usage would be: > > int square(int __attribute__ ((__unused__)) arg1, int arg2) { > return arg2; } > > I.e. the attribute is between type and name. Do you have a reference on this? For variables, the gcc

Re: [hwloc-devel] Attribute unsed not regognized

2010-03-26 Thread Bert Wesarg
On Fri, Mar 26, 2010 at 11:57, Brice Goglin wrote: > Bert Wesarg wrote: >> There is also a problem, that these __hwloc_attributes defines don't >> get through after install: >> > > Are you using the embedding stuff ? Or only including our headers ? > > There's no guarantee

Re: [hwloc-devel] Attribute unsed not regognized

2010-03-26 Thread Bert Wesarg
There is also a problem, that these __hwloc_attributes defines don't get through after install: $ gcc -DHAVE_CONFIG_H -I. -DSYSCONFDIR=\"/home/wesarg/opt/htop-dev/etc\" -I/home/wesarg/opt/hwloc-dev/include -W -Wunused-parameter -Wall -std=gnu99 -D_XOPEN_SOURCE_EXTENDED -g -O2 -MT