[Mpi-forum] Ticket 125/126 PDF ready for review

[Jeff Squyres]

I think it looks ok, meaning that I checked your changes.  I did not check to see if you missed any.

That being said, I'm not a fan of the MPI_INIT_THREAD C declaration -- it seems to just be more confusing than using ***.  Specifically, it's:

int MPI_Init_thread(int *argc, char *((*argv)[]), int required, int *provided)

Personally, I think *** is cleaner/easier to understand:

int MPI_Init_thread(int *argc, char ***argv, int required, int *provided)

I would advocate changing MPI_Init_thread to *** rather than changing MPI_Init to the complicated [] syntax.



On Jan 20, 2012, at 2:17 PM, Fab Tillier wrote:

> Hi Folks,
> 
> I've implemented the changes to the standard document for tickets 125 and 126.  The PDF is attached to the ticket, and available here: https://svn.mpi-forum.org/trac/mpi-forum-web/raw-attachment/ticket/125/ticket-125.pdf
> 
> A couple notes where I deviated from the tickets:
> - I didn't distinguish between input and output parameter changes, so all the changes are marked as ticket125
> - I did not add 'const' for the input parameters, as that was already done by ticket 140
> - I only updated the C bindings
> - I did not update MPI_Type_hindexed and MPI_Type_struct in the deprecated chapter.
> - I updated MPI_Init to match the parameter definition of MPI_Init_thread.
> 
> I'd appreciate if folks could take a look and comment, letting me know if I botched anything, or missed anything that should be changed.
> 
> Thanks,
> -Fab
> 
> 
> _______________________________________________
> mpi-forum mailing list
> mpi-forum at lists.mpi-forum.org
> http://lists.mpi-forum.org/mailman/listinfo.cgi/mpi-forum


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