[Spice-devel] [PATCH spice-protocol] macros: define INLINE

[Christophe Fergeau]

Hi,

On Thu, May 12, 2011 at 11:49:38AM +0300, Arnon Gilboa wrote:
> needed for spice/common files used by the client, server & qxl driver.
> in windows _inline works for both c/c++, while inline is c++ only.
> compiling the client with mixed c/c++ code required this define.

If this change is done, it's a good opportunity to look at removing the
similar #defines from spice/:

$ git grep "define.*INLINE"

common/lz_config.h:#define INLINE inline
common/lz_config.h:#define INLINE _inline
common/lz_config.h:#define INLINE inline
common/quic_config.h:#define INLINE inline
common/quic_config.h:#define INLINE _inline
common/quic_config.h:#define INLINE inline
server/glz_encoder_config.h:#define INLINE inline

I assume there's a followup patch changing some inline to INLINE in
spice/common? Can you send it as well?

Thanks,

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/20110512/65293bab/attachment.pgp>