Re: [OMPI devel] [OMPI svn-full] svn:open-mpi r26118

2012-03-09 Thread Josh Hursey
Fixed in r26122. I tested locally with the ibm test suite, and it looks good. MTT should highlight if there are any other issues - but I doubt there will be. -- Josh On Thu, Mar 8, 2012 at 5:16 PM, Josh Hursey wrote: > Good point (I did not even look at ompi_comm_compare, I was using this for >

Re: [OMPI devel] [OMPI svn-full] svn:open-mpi r26118

2012-03-08 Thread Josh Hursey
Good point (I did not even look at ompi_comm_compare, I was using this for something else). I'll take a pass at converting the ompi_comm_compare to use the ompi_group_compare functionality - it is good code reuse. Thanks, Josh On Thu, Mar 8, 2012 at 4:08 PM, George Bosilca wrote: > Josh, > > Op

Re: [OMPI devel] [OMPI svn-full] svn:open-mpi r26118

2012-03-08 Thread George Bosilca
Josh, Open MPI already have a similar function in the communicator part, function that is not exposed to the upper layer. I think that using the code in ompi_comm_compare (the second part that compare groups) is sound proof. Moreover, if now we have an ompi_group_compare function you should use