Jacek Caban (@jacek) commented about dlls/winex11.drv/xdnd.c:
x11drv_dnd_position_event
- Handle an XdndPosition event.
- Should return the DROPEFFECT_xxxx value instead of the NTSTATUS one.
Please drop this comment. We actually want callbacks to return NTSTATUS eventually, but for this particular case, it would be even better to move the callback out of winex11.
Anyway, it's unrelated to the rest of this patch, looks good otherwise, thanks.