On Fri, Jul 20, 2018 at 06:03:03AM -0400, Frediano Ziglio wrote:
> >
> > Rather than using
> > #ifdef COMPRESS_DEBUG
> > spice_info(...);
> > #endif
> >
> > we can #define COMPRESS_DEBUG to spice_debug() or to do nothing for a
> > slight readability improvement. This opportunity is used to replace
> > these spice_info() calls with g_debug(). The "do nothing" macro is a bit
>
> Current master is using spice_debug, not spice_info (2 time in the
> commit message). I think this confused Uri whom was suggesting the
> use of g_info.
Ah, I initially also got g_message VS g_debug wrong, which I fixed in
this v2, and I missed the misleading spice_info! Will amend before
pushing.
Christophe
>
> > convoluted to ensure that we will have a compile-time check for our
> > g_debug args.
> >
> > Signed-off-by: Christophe Fergeau <cfergeau@xxxxxxxxxx>
>
> Otherwise,
> Acked-by: Frediano Ziglio <fziglio@xxxxxxxxxx>
>
> > ---
> > Changes since v1:
> > - adjust/fix commit log
> > - use slighly more sophisticated COMPRESS_DEBUG macros for improved
> > static checks
> >
> > server/image-encoders.c | 28 +++++++++++++---------------
> > 1 file changed, 13 insertions(+), 15 deletions(-)
> >
> > diff --git a/server/image-encoders.c b/server/image-encoders.c
> > index 880972e70..88073a3ee 100644
> > --- a/server/image-encoders.c
> > +++ b/server/image-encoders.c
> > @@ -32,6 +32,14 @@
> >
> > #define MAX_GLZ_DRAWABLE_INSTANCES 2
> >
> > +#if 0
> > +#define COMPRESS_DEBUG(...) g_debug(__VA_ARGS__)
> > +#else
> > +#define COMPRESS_DEBUG(...) G_STMT_START { \
> > + if (0) g_debug(__VA_ARGS__); \
> > + } G_STMT_END
> > +#endif
> > +
> > typedef struct RedGlzDrawable RedGlzDrawable;
> > typedef struct GlzDrawableInstanceItem GlzDrawableInstanceItem;
> >
> > @@ -837,9 +845,7 @@ bool image_encoders_compress_quic(ImageEncoders *enc,
> > SpiceImage *dest,
> > stat_start_time_t start_time;
> > stat_start_time_init(&start_time, &enc->shared_data->quic_stat);
> >
> > -#ifdef COMPRESS_DEBUG
> > - spice_debug("QUIC compress");
> > -#endif
> > + COMPRESS_DEBUG("QUIC compress");
> >
> > switch (src->format) {
> > case SPICE_BITMAP_FMT_32BIT:
> > @@ -926,9 +932,7 @@ bool image_encoders_compress_lz(ImageEncoders *enc,
> > stat_start_time_t start_time;
> > stat_start_time_init(&start_time, &enc->shared_data->lz_stat);
> >
> > -#ifdef COMPRESS_DEBUG
> > - spice_debug("LZ LOCAL compress");
> > -#endif
> > + COMPRESS_DEBUG("LZ LOCAL compress");
> >
> > encoder_data_init(&lz_data->data);
> >
> > @@ -997,9 +1001,7 @@ bool image_encoders_compress_jpeg(ImageEncoders *enc,
> > SpiceImage *dest,
> > stat_start_time_t start_time;
> > stat_start_time_init(&start_time, &enc->shared_data->jpeg_alpha_stat);
> >
> > -#ifdef COMPRESS_DEBUG
> > - spice_debug("JPEG compress");
> > -#endif
> > + COMPRESS_DEBUG("JPEG compress");
> >
> > switch (src->format) {
> > case SPICE_BITMAP_FMT_16BIT:
> > @@ -1114,9 +1116,7 @@ bool image_encoders_compress_lz4(ImageEncoders *enc,
> > SpiceImage *dest,
> > stat_start_time_t start_time;
> > stat_start_time_init(&start_time, &enc->shared_data->lz4_stat);
> >
> > -#ifdef COMPRESS_DEBUG
> > - spice_debug("LZ4 compress");
> > -#endif
> > + COMPRESS_DEBUG("LZ4 compress");
> >
> > encoder_data_init(&lz4_data->data);
> >
> > @@ -1226,9 +1226,7 @@ bool image_encoders_compress_glz(ImageEncoders *enc,
> > int glz_size;
> > int zlib_size;
> >
> > -#ifdef COMPRESS_DEBUG
> > - spice_debug("LZ global compress fmt=%d", src->format);
> > -#endif
> > + COMPRESS_DEBUG("LZ global compress fmt=%d", src->format);
> >
> > if ((src->x * src->y) >=
> > glz_enc_dictionary_get_size(enc->glz_dict->dict)) {
> > return FALSE;
>
> Frediano
Attachment:
signature.asc
Description: PGP signature
_______________________________________________ Spice-devel mailing list Spice-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/spice-devel