Re: [PATCH weston v7 1/3] x11: port the x11 backend to the new init api

2016-05-03 Thread Bryce Harrington
On Sat, Apr 30, 2016 at 10:32:59AM +0200, Benoit Gschwind wrote: > Hello Bryce, > > It's look the you use space within Makefile.am instead of tabulation. > > Le 30/04/2016 10:30, Benoit Gschwind a écrit : > > Hello Bryce, > > > > It look good after trivial rebase to master. > > > > Reviewed-by:

Re: [PATCH weston v7 1/3] x11: port the x11 backend to the new init api

2016-04-30 Thread Benoit Gschwind
Hello Bryce, It's look the you use space within Makefile.am instead of tabulation. Le 30/04/2016 10:30, Benoit Gschwind a écrit : > Hello Bryce, > > It look good after trivial rebase to master. > > Reviewed-by: Benoit Gschwind > Tested-by: Benoit Gschwind > > Le 30/04/2016 00:40, Bryce Harri

Re: [PATCH weston v7 1/3] x11: port the x11 backend to the new init api

2016-04-30 Thread Benoit Gschwind
Hello Bryce, It look good after trivial rebase to master. Reviewed-by: Benoit Gschwind Tested-by: Benoit Gschwind Le 30/04/2016 00:40, Bryce Harrington a écrit : > From: Benoit Gschwind > > Use a "well" defined structure to configure x11-backend and move configuration > file parsing inside t

[PATCH weston v7 1/3] x11: port the x11 backend to the new init api

2016-04-29 Thread Bryce Harrington
From: Benoit Gschwind Use a "well" defined structure to configure x11-backend and move configuration file parsing inside the weston compositor code. Enforce destruction of all backend config objects after initialization. Since the backend config struct versioning implies that there we expect pot