In glib: domains = g_getenv ("G_MESSAGES_DEBUG"); if (((log_level & INFO_LEVELS) == 0) || domains == NULL || (strcmp (domains, "all") != 0 && (!log_domain || !strstr (domains, log_domain)))) return; On Thu, Sep 12, 2013 at 6:11 PM, Marc-André Lureau <marcandre.lureau@xxxxxxxxx> wrote: > On Thu, Sep 12, 2013 at 6:01 PM, Christophe Fergeau <cfergeau@xxxxxxxxxx> wrote: >> On Thu, Sep 12, 2013 at 02:09:12PM +0200, Marc-André Lureau wrote: >>> Allow to disable selectively channels, mainly used for testing, >>> ex: SPICE_DISABLE_CHANNELS=display spicy-stats -p 12345 >>> --- >>> gtk/spice-channel-priv.h | 2 ++ >>> gtk/spice-channel.c | 7 +++++++ >>> 2 files changed, 9 insertions(+) >>> >>> diff --git a/gtk/spice-channel-priv.h b/gtk/spice-channel-priv.h >>> index 92c9315..1f29c23 100644 >>> --- a/gtk/spice-channel-priv.h >>> +++ b/gtk/spice-channel-priv.h >>> @@ -134,6 +134,8 @@ struct _SpiceChannelPrivate { >>> gsize total_read_bytes; >>> uint64_t last_message_serial; >>> GSList *flushing; >>> + >>> + gboolean disable_channel_msg; >>> }; >>> >>> SpiceMsgIn *spice_msg_in_new(SpiceChannel *channel); >>> diff --git a/gtk/spice-channel.c b/gtk/spice-channel.c >>> index a0d9c15..55c8325 100644 >>> --- a/gtk/spice-channel.c >>> +++ b/gtk/spice-channel.c >>> @@ -126,6 +126,10 @@ static void spice_channel_constructed(GObject *gobject) >>> desc ? desc : "unknown", c->channel_type, c->channel_id); >>> CHANNEL_DEBUG(channel, "%s", __FUNCTION__); >>> >>> + const char *disabled = g_getenv("SPICE_DISABLE_CHANNELS"); >>> + if (disabled && strstr(disabled, desc)) >> >> Is there a reason for doing a strstr rather than iterating over the string >> split on ':' ? > > simplicity, iirc, even glib does that. > >> Patch looks good otherwise. >> >> Christophe > > > > -- > Marc-André Lureau -- Marc-André Lureau _______________________________________________ Spice-devel mailing list Spice-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/spice-devel