Re: [PATCH v3 RFC] channel: add context parameters to SpiceCoreInterfaceInternal

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

 



hey,

On Fri, Jan 08, 2016 at 11:00:39AM +0000, Frediano Ziglio wrote:
> Changes from v2:
> This patch replace basically patch 3 and 4 of previous version.
> This is more another style change (use a context not related to
> SpiceCoreInterfaceInternal pointer) than a version that wants to
> supercedes the old one.
> Changing context require to store and pass it explicitly so the
> choice between this and the old is quite an opinion.

I tend to prefer v2. If we want to get an explicit
per-SpiceCoreInterfaceInternal pointer, I'd just append a void *context
pointer to the SpiceCoreInterfaceInternal structure itself rather than
have an additional pointer passed around.

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]