Re: [PATCH v4 0/4] MR908: uiautomationcore: Add more nested node features.
27 Sep
2022
27 Sep
'22
3:16 p.m.
Esme Povirk (@madewokherd) commented about dlls/uiautomationcore/uia_private.h:
HWND hwnd; BOOL nested_node; + struct list *prov_thread_list_entry;
I really don't think this needs to be a pointer. If there's going to be at most 1 entry per uia_node, and it doesn't need to outlive the uia_node, you can put the entry in there directly. You can use list_init() when creating uia_node and list_empty() to check if the entry is being used. (Though list_remove will leave the entry in an invalid state, and you'll have to init it again if you want that check to work afterwards.) -- https://gitlab.winehq.org/wine/wine/-/merge_requests/908#note_9455
1179
Age (days ago)
1179
Last active (days ago)
0 comments
1 participants
participants (1)
-
Esme Povirk (@madewokherd)