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

Attachment: pgp2wrUqhLyFs.pgp
Description: PGP signature

_______________________________________________
Spice-devel mailing list
Spice-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/spice-devel

Reply via email to