On Wed Jun 21 13:09:21 2023 +0000, Connor McAdams wrote:
I thought about this a bit yesterday before pushing this branch, it would be nice to have a corresponding node AddRef inside of `uia_event_queue_post_event()`, but that'd add a bit of ugliness here IMO. We'd have to check the event type, get the node out of the variant, and then addref it. I guess if we had separate functions/structures for posting serverside vs clientside events to the queue it would be cleaner to do.
I kinda assumed when you added a UiaEventArgs field that it would eventually be a union.
AFAICT you're only using the VARIANT to store a ULONG_PTR. It doesn't carry any information about what kind of value you're actually storing in it, nor does it make it easier to free the value.