[vlc-devel] [PATCH v2 01/11] core: move all actions handling in actions.c

[Rémi Denis-Courmont]

On vendredi 11 août 2017 15:58:27 EEST Hugo Beauzée-Luyssen wrote:
> Either we want to be future proof and in this case, I don't see why
> there couldn't be 2 different entities being provided as context,

There can't be two contexts because input focus simply does not work that way 
on any existing system.

But that's not even the problem. To use variadic arguments, or even to use a 
single generic void pointer, there needs to be a discriminant among the fixed 
arguments. And there is not. So that can't work.

Furthermore, adding a vout_window_ReportKey() event seems far more consistent 
with the existing window abstraction in general, and mouse events in 
particular.

-- 
Rémi Denis-Courmont