[Spice-devel] [PATCH 3/3] make allocation fixed instead of variable

Christophe Fergeau cfergeau at redhat.com
Wed Mar 2 14:35:16 UTC 2016


On Tue, Feb 23, 2016 at 01:57:59PM -0600, Jonathon Jongsma wrote:
> 
> I'm a bit ambivalent about this change. Although it may generate better code, I
> think that it makes the code slightly more difficult to read. I'd almost prefer
> a #define to the enum solution. But I guess I don't care too much.

I definitely prefer a #define to the enum solution. But do we care at
all about making these array sizes known at compile-time? Do we get any
benefit from it?

Christophe
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <https://lists.freedesktop.org/archives/spice-devel/attachments/20160302/00112303/attachment.sig>


More information about the Spice-devel mailing list