Re: [PATCH 4/5] compositor: Version the backend configuration structures

2016-04-14 Thread Pekka Paalanen
On Wed, 13 Apr 2016 11:38:37 -0700 Bryce Harrington wrote: > On Wed, Apr 13, 2016 at 02:47:06PM +0300, Pekka Paalanen wrote: > > On Tue, 12 Apr 2016 21:44:10 -0700 > > Bryce Harrington wrote: > > > > > On Wed, Apr 06, 2016 at 11:43:54AM +0300, Pekka Paalanen wrote: > > > > On Mon, 21 Mar 20

Re: [PATCH 4/5] compositor: Version the backend configuration structures

2016-04-13 Thread Bryce Harrington
On Wed, Apr 13, 2016 at 02:47:06PM +0300, Pekka Paalanen wrote: > On Tue, 12 Apr 2016 21:44:10 -0700 > Bryce Harrington wrote: > > > On Wed, Apr 06, 2016 at 11:43:54AM +0300, Pekka Paalanen wrote: > > > On Mon, 21 Mar 2016 23:11:29 +0100 > > > Benoit Gschwind wrote: > > > > > > > Hello, > > >

Re: [PATCH 4/5] compositor: Version the backend configuration structures

2016-04-13 Thread Pekka Paalanen
On Tue, 12 Apr 2016 21:44:10 -0700 Bryce Harrington wrote: > On Wed, Apr 06, 2016 at 11:43:54AM +0300, Pekka Paalanen wrote: > > On Mon, 21 Mar 2016 23:11:29 +0100 > > Benoit Gschwind wrote: > > > > > Hello, > > > > > > I think that struct_version and struct_size should not be belong > > > c

Re: [PATCH 4/5] compositor: Version the backend configuration structures

2016-04-12 Thread Bryce Harrington
On Tue, Apr 12, 2016 at 09:44:10PM -0700, Bryce Harrington wrote: > On Wed, Apr 06, 2016 at 11:43:54AM +0300, Pekka Paalanen wrote: > > On Mon, 21 Mar 2016 23:11:29 +0100 > > Benoit Gschwind wrote: > > > > > Hello, > > > > > > I think that struct_version and struct_size should not be belong > >

Re: [PATCH 4/5] compositor: Version the backend configuration structures

2016-04-12 Thread Bryce Harrington
On Wed, Apr 06, 2016 at 11:43:54AM +0300, Pekka Paalanen wrote: > On Mon, 21 Mar 2016 23:11:29 +0100 > Benoit Gschwind wrote: > > > Hello, > > > > I think that struct_version and struct_size should not be belong > > compositor.h. I think those versioning should be back-end detail, and > > each b

Re: [PATCH 4/5] compositor: Version the backend configuration structures

2016-04-06 Thread Pekka Paalanen
On Mon, 21 Mar 2016 23:11:29 +0100 Benoit Gschwind wrote: > Hello, > > I think that struct_version and struct_size should not be belong > compositor.h. I think those versioning should be back-end detail, and > each back-end should provide a major version number through #define in > the backend h

Re: [PATCH 4/5] compositor: Version the backend configuration structures

2016-03-21 Thread Benoit Gschwind
Hello, I think that struct_version and struct_size should not be belong compositor.h. I think those versioning should be back-end detail, and each back-end should provide a major version number through #define in the backend header. Otherwise the patch look good Best regards. Reviewed-by: Benoi

[PATCH 4/5] compositor: Version the backend configuration structures

2016-03-09 Thread Bryce Harrington
With this struct versioning, it is possible to add new options without breaking the ABI, as long as all additions are made to the end of a struct and nothing existing is modified or removed. When things are added, the structure's size will increase, and we'll use this size as our minor version num