Re: [Spice-devel] [PATCH] Limit maximum "n-surfaces" via param spec

2016-10-31 Thread Frediano Ziglio
> > On Fri, 2016-10-28 at 06:46 -0400, Frediano Ziglio wrote: > > > > > > > > > In commit beec1b41, we manually limited this property value in > > > _set_property(). But there's a simpler way to do it: via the param > > > spec > > > for the property. > > > > > > This also means that we can remo

Re: [Spice-devel] [PATCH] Limit maximum "n-surfaces" via param spec

2016-10-28 Thread Jonathon Jongsma
On Fri, 2016-10-28 at 06:46 -0400, Frediano Ziglio wrote: > > > > > > In commit beec1b41, we manually limited this property value in > > _set_property(). But there's a simpler way to do it: via the param > > spec > > for the property. > > > > This also means that we can remove the warning log in

Re: [Spice-devel] [PATCH] Limit maximum "n-surfaces" via param spec

2016-10-28 Thread Frediano Ziglio
> > In commit beec1b41, we manually limited this property value in > _set_property(). But there's a simpler way to do it: via the param spec > for the property. > > This also means that we can remove the warning log in red_worker_new() > since GObject will automatically warn if a property is assi

[Spice-devel] [PATCH] Limit maximum "n-surfaces" via param spec

2016-10-27 Thread Jonathon Jongsma
In commit beec1b41, we manually limited this property value in _set_property(). But there's a simpler way to do it: via the param spec for the property. This also means that we can remove the warning log in red_worker_new() since GObject will automatically warn if a property is assigned a value ou