On 19 September 2012 07:09, Pekka Paalanen <[email protected]> wrote:
> On Tue, 18 Sep 2012 16:44:57 +0100

Hi Pekka,

> this seems to have quite much of code duplication. Would it not be any
> easier to do the swap only at notify_*() calling time instead?
> Or do you really need to limit it to these very specific evdev events?

This device only generates absolute events.

In the absolute case it takes into consideration the width and height
of the screen and it also adds this value onto the x and y of the
output. So doing it  later would require us to store the relative
position e.g. (e->value - device->abs.min_x) / (device->abs.max_x -
device->abs.min_x) and then apply the scale up to to width and output
position of x at the notify point.

I'll cook up a version of the patch that applies this at the point in
the code where it does the notify..

Cheerio,

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

Reply via email to