Re: [PATCH spice-server v3 4/5] red-channel-capabilities: Enhance

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

 



On Thu, Mar 02, 2017 at 07:03:55AM -0500, Frediano Ziglio wrote:
> I used TYPE_RED_CHANNEL_CAPABILITIES define, seems more consistent with
> other code and keep the "RedChannel" together.

I disagree, RedChannel is for example RED_TYPE_CHANNEL, not
TYPE_RED_CHANNEL.
The way I see it is that 'red' (and sometimes 'reds' or 'spice') is our
namespace, just like 'g' is glib's namespace. So just like we have
G_TYPE_OBJECT, we should use RED_TYPE_XXX

We happen to have various objects with no namespace (sndchannel, ...),
in this case the types are indeed TYPE_XXX. I would not encourage
types without the red namespace, but TYPE_RED is definitely not correct.

Christophe

Attachment: signature.asc
Description: PGP signature

_______________________________________________
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]