./orte/mca/errmgr/errmgr.h(135): warning #1286: invalid attribute for
"orte_errmgr_base_module_abort_fn_t"

typedef void (*orte_errmgr_base_module_abort_fn_t)(int error_code, char
*fmt, ...) __opal_attribute_format__(__printf__, 2, 3);

I think the issue is that you can't apply attributes to the type def for the
function pointer, but only to the particular instance of that function. At
one time, we also had an attribute indicating that a function did not
return. This would also apply to this function, but IIRC you cannot have
multiple attributes (or at least, we used to run into problems with it).

So I figured I would just let this ride for now and re-address it later.

On 3/3/08 10:21 AM, "Rainer Keller" <kel...@hlrs.de> wrote:

> Ralph,
> On Monday 03 March 2008 17:06, r...@osl.iu.edu wrote:
>> Author: rhc
>> Date: 2008-03-03 11:06:47 EST (Mon, 03 Mar 2008)
>> New Revision: 17681
>> URL: https://svn.open-mpi.org/trac/ompi/changeset/17681
>> 
>> Log:
>> Cleanup an attribute warning - not sure which one to set or where it should
>> go, so I'll leave that to someone more familiar with "attributes".
>> 
>> Ensure some debugging is only enabled when have_debug is set.
> 
>> Modified: trunk/orte/mca/errmgr/errmgr.h
>> ===========================================================================
>> === --- trunk/orte/mca/errmgr/errmgr.h (original)
>> +++ trunk/orte/mca/errmgr/errmgr.h 2008-03-03 11:06:47 EST (Mon, 03 Mar
>> 2008) @@ -132,7 +132,7 @@
>>   * itself, and then exit - it takes no other actions. The intent here is
>> to provide * a last-ditch exit procedure that attempts to clean up a
>> little. */
>> -typedef void (*orte_errmgr_base_module_abort_fn_t)(int error_code, char
>> *fmt, ...) __opal_attribute_format__(__printf__, 2, 3);
>> +typedef void (*orte_errmgr_base_module_abort_fn_t)(int error_code, char
>> *fmt, ...); 
> 
> What was the warning requiring You to get rid of the attribute?
> 
> The attribute should help find errors in the callers to
> orte_errmgr_base_abort...
> 
> Maybe the help on
>   https://svn.open-mpi.org/trac/ompi/wiki/CompilerAttributes
> could be improved?
> 
> 
> Thanks,
> Rainer


Reply via email to