[MPI3 Fortran] MPI draft 2011-04-21

[Rolf Rabenseifner]

Hubert,

thank you very much for your in-deep reading.

It is hard to keep such a new interface consistent with
the rest of MPI and the Fortran standard and if you did not find more
problems, then I'm happy!

I'll do nearly all of your advices, therefore only a few comments:

> (*) Page 545, Lines 36-40:
> Therefore, ... and ",..., s(96)" in Line 40.
This part is keppt,because the descriptor describes all these elements.

> (*) Page 546, Line 17 and Line 21:
> The font of _NOT_ in MPI_SUBARRAYS_NOT_SUPPORTED is too large.
I used this larger font to highlight the "NOT" which should not be overread.

> (*) Section Fortran Support Through the mpi_f08 Module and
> Section Fortran Support Through the mpi Module
> "Provide Interfaces for all MPI routines, ..."
> Does all include the deprecated MPI routines ?

I make it more clear: in mpi_f08 the deprecated routines must be included
at least in the quality as in the mpi module.

> What should item "Declare each argument with an INTENT=IN, OUT, or
> INOUT as appropriate" mean ?
Yes, I change to "as defined in this standard".

Best regards
Rolf


----- Original Message -----
> From: "Hubert Ritzdorf" <hritzdorf at hpce.nec.com>
> To: "MPI-3 Fortran working group" <mpi3-fortran at lists.mpi-forum.org>
> Sent: Friday, April 29, 2011 7:38:10 PM
> Subject: Re: [MPI3 Fortran] MPI draft 2011-04-21
> Hi,
> 
> I reviewed the draft and I propose only some small corrections to the
> draft
> 
> https://svn.mpi-forum.org/trac/mpi-forum-web/attachment/ticket/229/mpi-report-F2008-2011-04-21-changeonlyplustickets.pdf
> 
> (*) Page 15 Line 1:
> This is done by having two bindings where one has to optional argument
> and one does not.
> 
> This sentence is not really correct since at least functions
> MPI_Dist_graph_create_adjacent and MPI_Dist_graph_neighbors have
> 3 bindings since MPI_UNWEIGHTED can be used for 2 arguments.
> 
> I would propose to simply remove the sentence. This is an
> implementation
> detail.
> 
> (*) Page 546, Line 17 and Line 21:
> The font of _NOT_ in MPI_SUBARRAYS_NOT_SUPPORTED is too large.
> 
> (*) Page 552, Line 23:
> 
> Remove "The" after "the" in "In Example, 16.12, the The compiler"
> 
> (*) Page 553, Line 17:
> applictions -> applications
> 
> (*) Page 562, Line 9:
> gerneric -> generic
> 
> (*) Page 545, Lines 36-40:
> MPI will work operate only on the elements s(1), s(6) s(11).
> Therefore, remove the "etc.," in Line 36 and ",..., s(96)" in Line 40.
> 
> (*) Page 558, Line 34:
> autamatic -> automatic
> 
> (*) Page 559, Line 7
> Add a newline before subsubsection "Fortran Support Though the mpif.h
> Include File"
> 
> (*) Page 564, Line 38:
> interfces -> interfaces
> 
> In addition, I have 2 questions to
> 
> (*) Section Fortran Support Through the mpi_f08 Module and
> Section Fortran Support Through the mpi Module
> "Provide Interfaces for all MPI routines, ..."
> 
> Does all include the deprecated MPI routines ?
> 
> When looking for aim "compile-time checking", they should be included.
> Since no Fortran 2008 interface is defined, it is not required that
> they are included. What is right ?
> 
> (*) Section 16.2.15 "Fortran Support Through the mpi_f08 Module"
> 
> What should item "Declare each argument with an INTENT=IN, OUT, or
> INOUT as appropriate" mean ?
> 
> The MPI 3.0 draft defines all Intents. I would propose to remove this
> item entirely or to change it into
> "Declare each argument with an INTENT as defined in this standard" and
> remove the rational.
> 
> The following 4 items are also defined by each individual MPI
> interface and could also be removed.
> 
> Best regards
> 
> Hubert
> 
> 
> 
> 
> 
> 
> 
> 
> _______________________________________________
> mpi3-fortran mailing list
> mpi3-fortran at lists.mpi-forum.org
> http://lists.mpi-forum.org/mailman/listinfo.cgi/mpi3-fortran

-- 
Dr. Rolf Rabenseifner . . . . . . . . . .. email rabenseifner at hlrs.de
High Performance Computing Center (HLRS) . phone ++49(0)711/685-65530
University of Stuttgart . . . . . . . . .. fax ++49(0)711 / 685-65832
Head of Dpmt Parallel Computing . . . www.hlrs.de/people/rabenseifner
Nobelstr. 19, D-70550 Stuttgart, Germany . (Office: Allmandring 30)