This test is safe, but it depends on the caller setting handle to some
value other than INVALID_HANDLE_VALUE.  I think it might be safer to
have NotifyAddrChange set the handle value instead, and to mark the
test todo_wine.  I'm worried that some other app might assume that the
call to NotifyAddrChange succeeded and attempt to do a wait on an
invalid handle value, resulting in a crash.

Yes, sounds reasonable. I'll change that.

Thanks,
Janne


Reply via email to