[Spice-devel] [PATCH 7/9] common: add ring_get_length() for debugging purposes

Christophe Fergeau cfergeau at redhat.com
Wed Apr 27 08:53:34 PDT 2011


On Wed, Apr 27, 2011 at 10:24:01AM -0400, Marc-André Lureau wrote:
> > Do you mean it should be #ifdef'ed? or that it's better to add the
> > equivalent of G_GNUC_UNUSED to the function definition to avoid the
> > compiler
> > warning?
> 
> Is there such warning? I was expecting unused inline functions not to produce warning.
> The ifdef shouldn't be necessary. It's just one more functions, similar to the other provided in ring.h

Not sure we get such a warning with the default spice cflags, but it's easy
to get gcc to warn about such "dead" code (unused static functions)

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/20110427/8b379c06/attachment.pgp>


More information about the Spice-devel mailing list