+1.

I submitted a patch like this a while ago, and it met violent resistance.  :-)  
Although no one on the call today remembers exactly who raised the 
resistance... 



On Jul 2, 2013, at 10:40 AM, "Barrett, Brian W" <bwba...@sandia.gov> wrote:

> On 7/2/13 8:22 AM, "George Bosilca" <bosi...@icl.utk.edu> wrote:
> 
>> Our macros for the OMPI-level free list had one extra argument, a possible 
>> return value to signal that the operation of retrieving the element from the 
>> free list failed. However in this case the returned pointer was set to NULL 
>> as well, so the error code was redundant. Moreover, this was a continuous 
>> source of warnings when the picky mode was on.
>> 
>> The attached parch remove the rc argument from the OMPI_FREE_LIST_GET and 
>> OMPI_FREE_LIST_WAIT macros, and change to check if the item is NULL instead 
>> of using the return code.
>> 
>> Deadline: July 4th
> 
> Works for me.
> 
> Brian
> 
> --
>   Brian W. Barrett
>   Scalable System Software Group
>   Sandia National Laboratories
> _______________________________________________
> devel mailing list
> de...@open-mpi.org
> http://www.open-mpi.org/mailman/listinfo.cgi/devel


-- 
Jeff Squyres
jsquy...@cisco.com
For corporate legal information go to: 
http://www.cisco.com/web/about/doing_business/legal/cri/


Reply via email to