[Spice-devel] [RFCv5 01/47] server/main_channel.c: set NET_TEST_STAGE_INVALID=0 explicitly (TODO - the rest)

Christophe Fergeau cfergeau at redhat.com
Tue May 10 01:38:52 PDT 2011


On Sun, May 08, 2011 at 04:10:57PM +0300, Alon Levy wrote:
> should change all the enums that don't explicitly set their first value to 0.

I'm not really convinced this is useful, what does that bring us?

There are probably some enums where the actual int values of the enum members
don't matter, so changing these ones would give the false impression that
it's important that the first member is 0 while it's not. For the ones
where the exact int value is important, why not, but I don't know if there
are many of these?

Christophe
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/spice-devel/attachments/20110510/ea090f1f/attachment.pgp>


More information about the Spice-devel mailing list