<div dir="ltr"><div>A tiny typo:</div><div>  p566, line 30:  </div><div>    IN index    number of the value  ...</div><div> should be</div><div>    IN index    index of the value ...<br></div></div><div class="gmail_extra">
<br clear="all"><div><div dir="ltr">--Junchao Zhang</div></div>
<br><br><div class="gmail_quote">On Fri, Nov 22, 2013 at 3:35 PM, Kathryn Mohror <span dir="ltr"><<a href="mailto:kathryn@llnl.gov" target="_blank">kathryn@llnl.gov</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div style="word-wrap:break-word">Hi Junchao,<div><br></div><div>Sorry for the delay. I've been at SC.<br><div><br><div><div class="im"><blockquote type="cite"><div dir="ltr">In the MPI_T return code Table (P589), there are <div>
<br></div><div>Return Codes for Datatype Functions: MPI_T_ENUM_*</div><div><div>MPI_T_ERR_INVALID_INDEX    The enumeration index is invalid or has been deleted.</div>
<div>MPI_T_ERR_INVALID_ITEM      The item index queried is out of range (for MPI_T_ENUM_GET_ITEM only)</div><div><br></div><div>Why we used "<span style="color:rgb(255,0,0)">enumeration index</span>" here? Since in the standard, it is called a handle. </div>
</div></div></blockquote><div><br></div></div>I think the text could better read "enumeration item index" since the index is for an item in the enumeration, not the enumeration itself.</div><div><div class="im">
<br><blockquote type="cite"><div dir="ltr"><div><div>And why designed MPI_T_ERR_INVALID_ITEM specially for MPI_T_ENUM_GET_ITEM?</div></div></div></blockquote><div><br></div></div>I believe the intent was to differentiate between an index that is out of range and one that may have been deleted or invalidated for some reason, which would give more information than just one error code.</div>
<div><br></div><div>Do others feel this needs changing? If so, what do you think needs to happen? Possibly this:</div><div><div class="im"><br><blockquote type="cite"><div dir="ltr"><div>
<div>In my view, there should be MPI_T_ERR_INVALID_ENUM, MPI_T_ERR_INVALID_INDEX respectively.</div></div></div></blockquote><div><br></div></div>Or another suggestion:</div><div>return MPI_T_ERR_INVALID_INDEX for indices that are invalid or deleted and MPI_T_ERR_INVALID in the case that the index is out of range. If it's out of range, it's a misuse of the interface, so it fits with the intent of the new return code.<br>
<div><br></div>Kathryn</div><div><br><blockquote type="cite"><div dir="ltr"><div><div><br clear="all"><div><div dir="ltr">--Junchao Zhang</div></div>
</div></div></div>
_______________________________________________<br>mpiwg-tools mailing list<br><a href="mailto:mpiwg-tools@lists.mpi-forum.org" target="_blank">mpiwg-tools@lists.mpi-forum.org</a><br><a href="http://lists.mpi-forum.org/mailman/listinfo.cgi/mpiwg-tools" target="_blank">http://lists.mpi-forum.org/mailman/listinfo.cgi/mpiwg-tools</a></blockquote>
</div><br><div>
<div style="text-indent:0px;letter-spacing:normal;font-variant:normal;text-align:-webkit-auto;font-style:normal;font-weight:normal;line-height:normal;text-transform:none;font-size:medium;white-space:normal;font-family:Helvetica;word-wrap:break-word;word-spacing:0px">
<div>______________________________________________________________<br>Kathryn Mohror, <a href="mailto:kathryn@llnl.gov" target="_blank">kathryn@llnl.gov</a>, <a href="http://people.llnl.gov/mohror1" target="_blank">http://people.llnl.gov/mohror1</a><br>
CASC @ Lawrence Livermore National Laboratory, Livermore, CA, USA</div><div><br></div></div><br><br>
</div>
<br></div></div></div><br>_______________________________________________<br>
mpiwg-tools mailing list<br>
<a href="mailto:mpiwg-tools@lists.mpi-forum.org">mpiwg-tools@lists.mpi-forum.org</a><br>
<a href="http://lists.mpi-forum.org/mailman/listinfo.cgi/mpiwg-tools" target="_blank">http://lists.mpi-forum.org/mailman/listinfo.cgi/mpiwg-tools</a><br></blockquote></div><br></div>