[Spice-devel] [PATCH v2 2/2] spiceqxl: Reject invalid boolean values, just like for other options.

Francois Gouget fgouget at codeweavers.com
Mon Aug 31 15:44:51 PDT 2015


On Mon, 31 Aug 2015, Frediano Ziglio wrote:
[...]
> I think etiher we should merge first patch and discard the second or you should merge
> the two patches together.

I think it would be ok to commit them separately as a way to make the 
policy change more obvious. But the idea is also that you now get to 
decide whether to comit the second patch right away, announce an 
upcoming policy change regarding the handling of booleans and comit it 6 
months or a year from, now, or drop it entirely.

And if a single patch is the preferred option, then that's available 
too:
http://lists.freedesktop.org/archives/spice-devel/2015-June/020172.html


I don't have a strong opinion on this so the ball is in your camp.

-- 
Francois Gouget <fgouget at codeweavers.com>



More information about the Spice-devel mailing list