[Spice-devel] [PATCH v3 RFC] channel: add context parameters to SpiceCoreInterfaceInternal

Christophe Fergeau cfergeau at redhat.com
Fri Jan 8 05:58:10 PST 2016


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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/spice-devel/attachments/20160108/b079e5ae/attachment-0001.sig>


More information about the Spice-devel mailing list