----- Original Message ----- > On Tue, Jan 25, 2011 at 02:33:34PM -0500, Marc-André Lureau wrote: > > Hi > > > > > > +static inline unsigned int ring_get_length(Ring *ring) > > > > > > This won't compile if no one uses it - so you probably want to > > > make > > > it non static (or do a hack by doing (void)ring_get_length > > > somewhere). > > > > Correct, if you intend to do (gdb) call ring_get_length(ring). But I > > was using it in some temporary SPICE_DEBUG code. > > 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 regards -- Marc-André Lureau _______________________________________________ Spice-devel mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/spice-devel
