Hi,

On 06/19/2014 10:34 AM, Peter Hutterer wrote:
> On Thu, Jun 19, 2014 at 10:02:30AM +0200, Hans de Goede wrote:
>> Hi,
>>
>> On 06/19/2014 06:57 AM, Peter Hutterer wrote:
>>> Rather than a single global logging function, make the logging dependent on
>>> the individual context. This way we won't stomp on each other's feet in the
>>> (admittedly unusual) case of having multiple libinput contexts.
>>>
>>> The userdata argument to the log handler was dropped. The caller has a ref 
>>> to
>>> the libinput context now, any userdata can be attached to that context
>>> instead.
>>>
>>> Signed-off-by: Peter Hutterer <[email protected]>
>>
>> Hmm, you've removed the user_data argument to the log-handler how is
>> a custom log-handler supposed to get to any specific (dynamic) data it
>> may need ?
> 
> The libinput context is now available to the logging function, and that 
> context can be passed to libinput_get_user_data()*. tbh, I didn't see quite
> the need to have a separate userdata for logging.

Ah I did not know the user could store user_data in the libinput context,
now that I do, this patch is:

Reviewed-by: Hans de Goede <[email protected]>

Regards,

Hans

_______________________________________________
wayland-devel mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/wayland-devel

Reply via email to