<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=us-ascii">
</head>
<body>
<div style="word-wrap:break-word">Thanks for the feedback. I have attached a new draft for this ticket and incorporated your changes. Please give me any additional comments by the start of business Monday.
<div><br>
</div>
<div>Kathryn</div>
<div></div>
</div>
<div style="word-wrap:break-word">
<div><br>
<div>
<div>On Feb 14, 2014, at 7:23 AM, Dave Goodell (dgoodell) <<a href="mailto:dgoodell@cisco.com">dgoodell@cisco.com</a>> wrote:</div>
<br class="x_Apple-interchange-newline">
<blockquote type="cite">+1 to almost everything that Jeff said.  My only quibble would be to change the last sentence in his text from:<br>
<br>
The contents of returned MPI_CHAR values are only defined from index 0 through the location of the first null-terminating character.<br>
<br>
To:<br>
<br>
The contents of returned MPI_CHAR ***arrays*** are only defined from index 0 through the location of the first null-terminating character.<br>
<br>
-Dave<br>
<br>
On Feb 14, 2014, at 7:27 AM, Jeff Squyres (jsquyres) <<a href="mailto:jsquyres@cisco.com">jsquyres@cisco.com</a>> wrote:<br>
<br>
<blockquote type="cite">Here's the new text:<br>
<br>
-----<br>
The use of the datatype MPI_CHAR in the MPI tool information interface implies a null-terminated character array, i.e. a string in the C language. If a variable has type MPI_CHAR, the value of the count parameter returned by MPI_T_CVAR_HANDLE_ALLOC and MPI_T_PVAR_HANDLE_ALLOC
 will include the null-terminating character. The value returned in count must be large enough to contain any valid value for the given variable. Arrays of type MPI_CHAR are only valid to the first null character.<br>
-----<br>
<br>
There's a missing "," after "i.e." -- it should be "i.e.,"<br>
<br>
I'd make some other minor changes:<br>
<br>
The use of the datatype MPI_CHAR in the MPI tool information interface implies a null-terminated character array, i.e., a string in the C language. If a variable has type MPI_CHAR, the value of the count parameter returned by MPI_T_CVAR_HANDLE_ALLOC and MPI_T_PVAR_HANDLE_ALLOC
 ***must be large enough to include any valid value, including its null-terminating character.***  ***The contents of returned MPI_CHAR values are only defined from index 0 through the location of the first null-terminating character.***<br>
<br>
<br>
<br>
<br>
<br>
On Feb 13, 2014, at 3:24 PM, Kathryn Mohror <<a href="mailto:kathryn@llnl.gov">kathryn@llnl.gov</a>> wrote:<br>
<br>
<blockquote type="cite">Hi all,<br>
<br>
Please have a look at this since we  have to decide if it's ready for a reading, and submit it to the forum for review on Monday if we do.<br>
<br>
Kathryn<br>
<br>
On Feb 7, 2014, at 1:05 PM, "Mohror, Kathryn" <<a href="mailto:mohror1@llnl.gov">mohror1@llnl.gov</a>> wrote:<br>
<br>
<blockquote type="cite">Hi all,<br>
<br>
As we discussed in our call, I have drafted new text for ticket 405 which clarifies our intent for the MPI_CHAR datatype to represent strings in the tools interface. Please take a look at the red text for ticket 405 in the attached document and send your comments.<br>
<br>
Kathryn<br>
<br>
<br>
_________________________________________________________________<br>
Kathryn Mohror, <a href="mailto:kathryn@llnl.gov">kathryn@llnl.gov</a>, <a href="http://scalability.llnl.gov/">
http://scalability.llnl.gov/</a><br>
Scalability Team @ Lawrence Livermore National Laboratory, Livermore, CA, USA<br>
<br>
<br>
<br>
<br>
<br>
<br>
<br>
<br>
<tools-3.pdf>_______________________________________________<br>
mpiwg-tools mailing list<br>
<a href="mailto:mpiwg-tools@lists.mpi-forum.org">mpiwg-tools@lists.mpi-forum.org</a><br>
http://lists.mpi-forum.org/mailman/listinfo.cgi/mpiwg-tools<br>
</blockquote>
<br>
_________________________________________________________________<br>
Kathryn Mohror, <a href="mailto:kathryn@llnl.gov">kathryn@llnl.gov</a>, <a href="http://scalability.llnl.gov/">
http://scalability.llnl.gov/</a><br>
Scalability Team @ Lawrence Livermore National Laboratory, Livermore, CA, USA<br>
<br>
<br>
<br>
<br>
<br>
<br>
<br>
<br>
_______________________________________________<br>
mpiwg-tools mailing list<br>
<a href="mailto:mpiwg-tools@lists.mpi-forum.org">mpiwg-tools@lists.mpi-forum.org</a><br>
http://lists.mpi-forum.org/mailman/listinfo.cgi/mpiwg-tools<br>
</blockquote>
<br>
<br>
-- <br>
Jeff Squyres<br>
<a href="mailto:jsquyres@cisco.com">jsquyres@cisco.com</a><br>
For corporate legal information go to: http://www.cisco.com/web/about/doing_business/legal/cri/<br>
<br>
_______________________________________________<br>
mpiwg-tools mailing list<br>
mpiwg-tools@lists.mpi-forum.org<br>
http://lists.mpi-forum.org/mailman/listinfo.cgi/mpiwg-tools<br>
</blockquote>
<br>
_______________________________________________<br>
mpiwg-tools mailing list<br>
<a href="mailto:mpiwg-tools@lists.mpi-forum.org">mpiwg-tools@lists.mpi-forum.org</a><br>
http://lists.mpi-forum.org/mailman/listinfo.cgi/mpiwg-tools<br>
</blockquote>
</div>
<br>
<div>
<div style="color:rgb(0,0,0); font-family:Helvetica; font-size:medium; font-style:normal; font-variant:normal; font-weight:normal; letter-spacing:normal; line-height:normal; orphans:2; text-indent:0px; text-transform:none; white-space:normal; widows:2; word-spacing:0px; word-wrap:break-word">
<div style="color:rgb(0,0,0); font-family:Helvetica; font-size:medium; font-style:normal; font-variant:normal; font-weight:normal; letter-spacing:normal; line-height:normal; orphans:2; text-indent:0px; text-transform:none; white-space:normal; widows:2; word-spacing:0px; word-wrap:break-word">
<div style="color:rgb(0,0,0); font-family:Helvetica; font-size:medium; font-style:normal; font-variant:normal; font-weight:normal; letter-spacing:normal; line-height:normal; orphans:2; text-indent:0px; text-transform:none; white-space:normal; widows:2; word-spacing:0px; word-wrap:break-word">
<div style="color:rgb(0,0,0); font-family:Helvetica; font-size:medium; font-style:normal; font-variant:normal; font-weight:normal; letter-spacing:normal; line-height:normal; orphans:2; text-indent:0px; text-transform:none; white-space:normal; widows:2; word-spacing:0px; word-wrap:break-word">
<div style="color:rgb(0,0,0); font-family:Helvetica; font-size:medium; font-style:normal; font-variant:normal; font-weight:normal; letter-spacing:normal; line-height:normal; orphans:2; text-indent:0px; text-transform:none; white-space:normal; widows:2; word-spacing:0px; word-wrap:break-word">
<div>_________________________________________________________________</div>
<div>Kathryn Mohror, <a href="mailto:kathryn@llnl.gov">kathryn@llnl.gov</a>, <a href="http://scalability.llnl.gov/">http://scalability.llnl.gov/</a><br>
Scalability Team @ Lawrence Livermore National Laboratory, Livermore, CA, USA<br>
</div>
<div><br>
</div>
</div>
<br class="x_Apple-interchange-newline">
</div>
<br class="x_Apple-interchange-newline">
</div>
<br class="x_Apple-interchange-newline">
</div>
<br class="x_Apple-interchange-newline">
</div>
<br class="x_Apple-interchange-newline" style="color:rgb(0,0,0); font-family:Helvetica; font-size:medium; font-style:normal; font-variant:normal; font-weight:normal; letter-spacing:normal; line-height:normal; orphans:2; text-indent:0px; text-transform:none; white-space:normal; widows:2; word-spacing:0px">
<br class="x_Apple-interchange-newline">
</div>
<br>
</div>
</div>
</body>
</html>