Re: [spice-server 05/10] qxl: Rename 'qxl' to 'qxl_cmd' in red_get_cursor_cmd()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> 
> On Mon, Apr 16, 2018 at 06:58:18AM -0400, Frediano Ziglio wrote:
> > Don't like this, lot of structures in this file use "qxl", for coherence
> > I would change all or nothing but changing all would mean a lot of changes
> > with not much value
> 
> Imo 'red' and 'qxl' are not very good names, I'd prefer to have them
> named red_cmd and qxl_cmd. I agree it's quite a few changes though. I
> can send an initial patch changing all of these if you prefer.
> 
> Christophe
> 

I would personally keep red and qxl and add a qxl_instance.

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




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]