<div dir="ltr">I've wasted a bit of time in the past on this.  Typically turning on more log channels cleared it up relatively quickly.<div><br></div><div>I'd personally rather not see all the missing requirements.</div><div><br></div><div>I'm in favor of showing the biggest thing missing if we know that will always make sense.  Are there cases, though, when working with say multiple processors on a board that might invalidate this approach?  If that's not a concern, then I like it as you last stated it, Jason (show biggest concept missing).  Sounds like it's almost a sanity-check call.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Sep 18, 2014 at 10:44 PM, Jason Molenda <span dir="ltr"><<a href="mailto:jason@molenda.com" target="_blank">jason@molenda.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Ah, there were a couple of emails that I don't see on lldb-commits, not sure if they're delayed, or if I forgot to cc the list while at work.<br>
<br>
I thought about this a bit more.  Right now if you start lldb and try to reg read pc, you get an error about there being no frame.  This is the "smallest" thing that you need to fulfill the request.  But of course you also don't have the Thread (next bigger thing), or a Process (next bigger) or even a Target (biggest).<br>
<br>
Maybe we should change it so the behavior is<br>
<br>
% lldb<br>
(lldb) reg read pc<br>
<span class="">error: invalid target, create a target using the 'target create' command<br>
<br>
<br>
</span>instead of listing all of the things that the command needs (frame, thread, process, target)?<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
<br>
On Sep 18, 2014, at 7:49 PM, Jason Molenda <<a href="mailto:jmolenda@apple.com">jmolenda@apple.com</a>> wrote:<br>
<br>
> I applied the patch and was going to commit it, then I realized I don't clearly understand what behavior we're fixing.  e.g. these commands all report a reasonable error msg when a thread/process/frame is missing before the patch is applied.<br>
><br>
> % lldb<br>
> (lldb) f<br>
> error: invalid thread<br>
> (lldb) bt<br>
> error: invalid process<br>
> (lldb) reg read pc<br>
> error: invalid frame<br>
> (lldb) c<br>
> error: invalid process<br>
> (lldb)<br>
><br>
><br>
><br>
>> On Sep 18, 2014, at 9:09 AM, Todd Fiala <<a href="mailto:tfiala@google.com">tfiala@google.com</a>> wrote:<br>
>><br>
>> Jason, do you want to submit this or do you want me to run it through over<br>
>> here?<br>
>><br>
>> <a href="http://reviews.llvm.org/D5374" target="_blank">http://reviews.llvm.org/D5374</a><br>
>><br>
>><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>
<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>