Re: [PATCH 1/2] matrix: track transform type

2013-01-29 Thread Pekka Paalanen
On Mon, 28 Jan 2013 22:40:28 +0300 Vasily Khoruzhick wrote: > Introduce several matrix transform types and track type for matrix. > Could be usefull for activating some fastpath that depends on some > transform type. > > Signed-off-by: Vasily Khoruzhick > --- > shared/matrix.c | 23 ++

Re: [PATCH 1/2] matrix: track transform type

2013-01-29 Thread Pekka Paalanen
On Mon, 28 Jan 2013 16:21:11 -0500 Kristian Høgsberg wrote: > On Mon, Jan 28, 2013 at 10:40:28PM +0300, Vasily Khoruzhick wrote: > > Introduce several matrix transform types and track type for matrix. > > Could be usefull for activating some fastpath that depends on some > > transform type. > >

Re: [PATCH 1/2] matrix: track transform type

2013-01-28 Thread Kristian Høgsberg
On Mon, Jan 28, 2013 at 03:41:09PM -0800, Bill Spitzak wrote: > This is very very useful, however the proposed set is not really correct: > > >>+enum weston_matrix_transform_type { > >>+ WESTON_MATRIX_TRANSFORM_TRANSLATE = (1 << 0), > >>+ WESTON_MATRIX_TRANSFORM_SCALE = (1 << 1

Re: [PATCH 1/2] matrix: track transform type

2013-01-28 Thread Bill Spitzak
This is very very useful, however the proposed set is not really correct: +enum weston_matrix_transform_type { + WESTON_MATRIX_TRANSFORM_TRANSLATE = (1 << 0), + WESTON_MATRIX_TRANSFORM_SCALE = (1 << 1), + WESTON_MATRIX_TRANSFORM_ROTATE = (1 << 2), +

Re: [PATCH 1/2] matrix: track transform type

2013-01-28 Thread Kristian Høgsberg
On Mon, Jan 28, 2013 at 10:40:28PM +0300, Vasily Khoruzhick wrote: > Introduce several matrix transform types and track type for matrix. > Could be usefull for activating some fastpath that depends on some > transform type. Look good, both committed. We could simplify the logic in weston_surface_

[PATCH 1/2] matrix: track transform type

2013-01-28 Thread Vasily Khoruzhick
Introduce several matrix transform types and track type for matrix. Could be usefull for activating some fastpath that depends on some transform type. Signed-off-by: Vasily Khoruzhick --- shared/matrix.c | 23 --- shared/matrix.h | 10 ++ src/compositor.c | 3 +++