> 
> On Fri, Jul 21, 2017 at 02:51:38PM +0100, Frediano Ziglio wrote:
> > +    const int palette_colors = out_bits > 8 ? 0 : 1 << (bits == 2 ? 2 :
> > out_bits);
> 
> This really deserves to be split in multiple statements to make this
> easier to parse :)
> 
> Looks good otherwise.
> 
> Christophe
> 

Maybe just

    const int palette_colors = out_bits > 8 ? 0 : // no palette
                               bits == 2 ? 4 :    // 2 bit PNG converted to 4 
bit BMP
                               1 << out_bits;     // palette

(with the comments too)

Frediano
_______________________________________________
Spice-devel mailing list
Spice-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/spice-devel

Reply via email to