<div dir="ltr"><div>Submitted:</div><div><br></div><div><font face="courier new, monospace">> Transmitting file data .</font></div><div><font face="courier new, monospace">> Committed revision 198742.</font></div></div>
<div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Jan 7, 2014 at 6:55 PM, Steve Pucci <span dir="ltr"><<a href="mailto:spucci@google.com" target="_blank">spucci@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr">Please commit the attached patch.  Thanks!<div><div class="h5"><br><br><div class="gmail_quote">---------- Forwarded message ----------<br>From: <b class="gmail_sendername">Greg Clayton</b> <span dir="ltr"><<a href="mailto:gclayton@apple.com" target="_blank">gclayton@apple.com</a>></span><br>

Date: Tue, Jan 7, 2014 at 4:37 PM<br>Subject: Re: [PATCH] Python register definition script not being executed<br>To: Steve Pucci <<a href="mailto:spucci@google.com" target="_blank">spucci@google.com</a>><br>Cc: Todd Fiala <<a href="mailto:tfiala@google.com" target="_blank">tfiala@google.com</a>><br>

<br><br>Looks good.<br>
<div><div><br>
On Jan 7, 2014, at 3:58 PM, Steve Pucci <<a href="mailto:spucci@google.com" target="_blank">spucci@google.com</a>> wrote:<br>
<br>
> Hi Greg,<br>
><br>
> (Thanks for your previous mail.  Still working through it to get good answers).<br>
><br>
> The attached patch fixes a problem where our register definition script was not being invoked.  I believe that as a result of this change<br>
><br>
> <a href="http://lists.cs.uiuc.edu/pipermail/lldb-commits/Week-of-Mon-20131202/010096.html" target="_blank">http://lists.cs.uiuc.edu/pipermail/lldb-commits/Week-of-Mon-20131202/010096.html</a><br>
><br>
> , the call asking for the first register definition "succeeds" even though the response from qRegisterInfo0 is $00#, and this success disturbs the logic in the rest of the method, which was assuming failure from the packet call (specifically, reg_num was now 1 rather than 0 when exiting the for loop, which caused the reading of the Python script to be skipped).<br>


><br>
> One thing I'm not sure of:  there is another dependency on "reg_num == 0" later on in the same method, which is used in the call to m_register_info.HardcodeARMRegisters(from_scratch).  It seems clear that the intent is that reg_num be 0 here as well, though, which this patch will also accomplish.<br>


><br>
> Thanks,<br>
>   Steve<br>
><br>
> diff --git a/source/Plugins/Process/gdb-remote/ProcessGDBRemote.cpp b/source/Plugins/Process/gdb-remote/ProcessGDBRemote.cpp<br>
> index 0cf3284..a9cf080 100644<br>
> --- a/source/Plugins/Process/gdb-remote/ProcessGDBRemote.cpp<br>
> +++ b/source/Plugins/Process/gdb-remote/ProcessGDBRemote.cpp<br>
> @@ -509,6 +509,10 @@ ProcessGDBRemote::BuildDynamicRegisterInfo (bool force)<br>
><br>
>                  m_register_info.AddRegister(reg_info, reg_name, alt_name, set_name);<br>
>              }<br>
> +            else<br>
> +            {<br>
> +                break;  // ensure exit before reg_num is incremented<br>
> +            }<br>
>          }<br>
>          else<br>
>          {<br>
><br>
</div></div>> <patch.txt><br>
<br>
</div><br></div></div></div>
<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>
<br></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>