Re: [PATCH spice 12/18] Declare AsyncCommand in red-common

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

 



On Wed, Dec 16, 2015 at 07:54:07AM -0500, Marc-André Lureau wrote:
> Hi
> 
> ----- Original Message -----
> > 
> > > > > This is the simplest way I could find to have a AsyncCommand in
> > > > > QXLState in following patch.
> > > > > 
> > > > 
> > > > The following patch just update spice-common repository.
> > > > This patch is referring the "following"?
> > > > 
> > > > The reasoning looks a bit weak to me.
> > > 
> > > Why? I just wanted to store the current gl-draw AsyncCommand in QXLState.
> > > 
> > 
> > You can just use
> > 
> > struct AsyncCommand *aync_command;
> > 
> > like for dispatcher
> 
> Correct, that seems to be a common practice in Spice server code. I'll
> change the series to use that instead then. However, it would be nice
> to consider having the common typedefs in common imho. It doesn't make
> sense to me that in some header you can use "Foo" type directly and
> some you don't and have to use "struct Foo".

Yup, agreed.

Christophe

Attachment: signature.asc
Description: PGP signature

_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
http://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]