On May 14, 2009, at 9:57 AM, Rainer Keller wrote:

> This generated a bunch of warnings - the "z" length modifier is not a
> generally supported option, which is why we do not use it.
I see You compile with -pedantic?


Ya, configure adds that automatically if you --enable-picky.

> btl_tcp_component.c: In function ‘mca_btl_tcp_component_create_listen’:
> btl_tcp_component.c:682: warning: too many arguments for format
This is the nice part of the attribute-fix: showing mistakes like this...
Fixed...


Yep; it is good to expose all of these. Annoying to fix, but they should be fixed. :-)

> Could you please fix this? If you want to deal with the size_t 32/64
> bit issues, there is another little macro thingy that we created for
> just that purpose (someone else here undoubtedly remembers it).
Well, does anyone now the "little macro thingy" ;-)
I failed to find it...

However, in other parts of the code base, we just use "%lu" and cast to
unsigned long...


Unfortunately, I think that's the best we ever came up with:

    https://svn.open-mpi.org/trac/ompi/wiki/PrintfCodes

--
Jeff Squyres
Cisco Systems


Reply via email to