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

[Christophe Fergeau]

On Thu, May 12, 2011 at 11:41:34AM +0200, Christophe Fergeau wrote:
> 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?

I was too quick in sending this, I see this is already done in the next
patch series :)

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/6cffd637/attachment.pgp>