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
pgp2wrUqhLyFs.pgp
Description: PGP signature
_______________________________________________ Spice-devel mailing list Spice-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/spice-devel