OK, thanks. I think I may have some more patches soon.

Regards,

  Filipe

On Thu, Aug 25, 2011 at 11:28, Johnny Chen <[email protected]> wrote:

> Hi Filipe,
>
> Thanks for the patches.  The ifdef and uninitialized patches have been
> applied.
> As to the get-first-thread patch, we're considering to modify the
> GetSelectedThread() method impl
> so that it returns the 0th thread if there is no selected thread.
>
> Thanks again!
> Johnny
>
> On Aug 24, 2011, at 6:23 PM, Filipe Cabecinhas wrote:
>
> Hi,
>
> I sent some small fixes to the list, but I don't know if anyone saw them,
> since they got mixed with some questions I had.
>
> One fixes debug code that usually gets ifdeffed out of compilation.
> Another one initializes a uint64_t variable that would get the highest
> 32-bits zeroed out when executing a 32-bit target, resulting in possibly
> wrong addresses when disassembling code.
> The "get-first-thread-if-no-selected-thread" patch
> makes CommandObjectThreadStepWithTypeAndScope::Execute behave like what was
> told to be the desired behaviour: If there is no selected thread, the first
> thread is selected (
> http://lists.cs.uiuc.edu/pipermail/lldb-dev/2011-July/000522.html ).
>
> Regards,
>
>   Filipe
>
> <fix-ifdeffed-code-stackframelist.patch>
> <get-first-thread-if-no-selected-thread.patch><uninitialized-uint64.patch>
> _______________________________________________
> lldb-dev mailing list
> [email protected]
> http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev
>
>
>
_______________________________________________
lldb-dev mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev

Reply via email to