Yes, this should be checked.  If the return value is true, then there should be 
an event in event_sp, but if it returns false, then you shouldn't rely on the 
contents of the event.

Jim

On Sep 30, 2013, at 9:18 AM, Carlo Kok <[email protected]> wrote:

> thread_result_t
> Process::RunPrivateStateThread ()
> {
> ...
> 
>   WaitForEventsPrivate (NULL, event_sp, control_only);
>   if (event_sp->BroadcasterIs(&m_private_state_control_broadcaster))
> 
> I'm getting a crash because the pointer in event_sp is null. Now this is 
> probably because some code isn't 100% windows ready, yet. However shouldn't 
> something be checking the return value of WaitForEventsPrivate anyway?
> 
> -- 
> Carlo Kok
> RemObjects Software
> _______________________________________________
> lldb-commits mailing list
> [email protected]
> http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits

_______________________________________________
lldb-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits

Reply via email to