Add Notify::Window enum member #133
Conversation
evmar
left a comment
evmar
left a comment
There was a problem hiding this comment.
This is great, I am impressed you figured this out!
| pt_x: 0, | ||
| pt_y: 0, | ||
| }; | ||
| dispatch_message(sys, &msg).await; |
There was a problem hiding this comment.
This executes the message handler in the current x86 thread, which (from the name of the function you're in) you can see is the multimedia thread. That is, all the winmm functionality is supposed to happen on this winmm thread and the different callbacks it makes (with the exception of the function callback, I think?) are supposed to go onto the caller's thread.
Concretely, I think that means this should put this message in the target's message queue, rather than dispatching it. Take a look at PostMessage's impl
retrowin32/win32/dll/user32/src/message.rs
Line 443 in 8ff30f8
I think maybe the way I'd do it is to change that PostMessage into a wrapper around a post_message() function that accepts a MSG and then call it from here.
2 participants
Unfortunately, this did not fix my problem (#126 (comment)), but I still believe that this implementation should be correct...