Re: [PATCH proto] macros: remove INLINE

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Oct 07, 2013 at 09:16:30AM -0400, Marc-André Lureau wrote:
> 
> 
> > Won't that make VC++ unhappy?
> 
> It's not being used in the protocol headers.
> 
> For the usage, it's used partially in spice-common (mixed with regular
> inline), in which we require c99 anyway, as said in commit message.

This does not answer the question. Iirc vc++ does not support c99, so
INLINE may be needed by vc++. The spice-common headers may already be c99
or broken with vc++, but 'it's already broken' is not a very convincing reason
to make things worse. glib seems to have some portability magic for
'inline'.
With that said, this would only impact win builds of the old spicec client,
so we can probably decide it's not a big issue.

Christophe

Attachment: pgpoMlqTC75M9.pgp
Description: PGP signature

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

[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]