<div dir="ltr">Ah right yep. I'll address that too.</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, May 15, 2014 at 9:31 AM, Greg Clayton <span dir="ltr"><<a href="mailto:gclayton@apple.com" target="_blank">gclayton@apple.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Looks fine. Just be sure that all code that looks for 'W' also handles 'X'. I see some code in ProcessGDBRemote which handles 'W' but not 'X' (in ProcessGDBRemote::SetThreadStopInfo (StringExtractor& stop_packet)).<br>
<div class="HOEnZb"><div class="h5"><br>
> On May 14, 2014, at 4:38 PM, Todd Fiala <<a href="mailto:todd.fiala@gmail.com">todd.fiala@gmail.com</a>> wrote:<br>
><br>
> Hey guys,<br>
><br>
> Debugserver is responding to a $k kill process command with W09 rather than X09 (i.e. indicating a normal exit with return code 9 rather than a signal-based exit from signal 9). Any issue with me changing that? I've tested it locally and it doesn't appear to cause any harm. lldb still picks up the exit signal.<br>
><br>
> Thanks!<br>
><br>
> --- tools/debugserver/source/RNBRemote.cpp (revision 208785)<br>
> +++ tools/debugserver/source/RNBRemote.cpp (working copy)<br>
> @@ -3770,7 +3770,7 @@<br>
> // No response to should be sent to the kill packet<br>
> if (m_ctx.HasValidProcessID())<br>
> DNBProcessKill (m_ctx.ProcessID());<br>
> - SendPacket ("W09");<br>
> + SendPacket ("X09");<br>
> return rnb_success;<br>
> }<br>
><br>
> --<br>
> -Todd<br>
</div></div><div class="HOEnZb"><div class="h5">> _______________________________________________<br>
> lldb-commits mailing list<br>
> <a href="mailto:lldb-commits@cs.uiuc.edu">lldb-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits</a><br>
<br>
_______________________________________________<br>
lldb-commits mailing list<br>
<a href="mailto:lldb-commits@cs.uiuc.edu">lldb-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits</a><br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div dir="ltr"><table cellspacing="0" cellpadding="0" style="color:rgb(136,136,136);font-family:'Times New Roman'"><tbody><tr style="color:rgb(85,85,85);font-family:sans-serif;font-size:small">
<td nowrap style="border-top-style:solid;border-top-color:rgb(213,15,37);border-top-width:2px">Todd Fiala |</td><td nowrap style="border-top-style:solid;border-top-color:rgb(51,105,232);border-top-width:2px"> Software Engineer |</td>
<td nowrap style="border-top-style:solid;border-top-color:rgb(0,153,57);border-top-width:2px"> <a href="mailto:tfiala@google.com" style="color:rgb(17,85,204)" target="_blank"><span style="background-color:rgb(255,255,204);color:rgb(34,34,34);background-repeat:initial initial">tfiala@google.com</span></a> |</td>
<td nowrap style="border-top-style:solid;border-top-color:rgb(238,178,17);border-top-width:2px"><font color="#1155cc"> <a>650-943-3180</a></font></td></tr></tbody></table><br></div>
</div>