Skip to content

MPI-5 RC1 review Aurelien #945

Open
Open
@devreal

Description

@devreal
  • "P7L7: missing space ""Standard[1]""
  • P13L39: A'_i is talked about but never defined, reference to p_i!=p appears on P14L13 this could be more clear
    • I don't think this needs fixing
  • P16L35: there is a ""2.4.3 MPI Datatypes"" section, just before the ""2.5 Datatypes"" section (P17L20), 2.4.3 should be consolidated as 2.5.1 (prepend before current 2.5.1)
  • P22L15: missing space ""_c""suffixed
  • P24L29: ""/ !(_c) /"" are these supposed to be ""/* */""?
  • P25L3: ""To support the profiling interface, programs must not declare functions with names ... PMPI_"" seems to imply that programs that do not wish to support profiling can declare PMPI_ functions, presumably intended meaning is ""programs must not declare functions with names ... PMPI_, as these are used by the profiling interface.""
    • Not sure this help disambiguate here. I wouldn't touch it.
  • P28L35: missing space ""World Model(see Section""
  • P32L26: use of ""strcpy"" in example, should we use ""strncpy""?"
    • Not worth fixing since the string is fixed legth.

Metadata

Metadata

Assignees

Labels

had readingCompleted the formal proposal readingmpi-5.0For inclusion in the MPI 5.0 standardscheduled first-and-only voteFirst-and-only vote is scheduled for the next meeting

Type

No type

Projects

Status

Had Reading

Relationships

None yet

Development

No branches or pull requests

Issue actions