----- Original Message -----
> Hi
>
> ----- Original Message -----
> > usbredir on debug level is highly verbose. That does not help much when
> > we are trying to debug the spice-gtk side of usbredir.
> >
> > This patch introduces SPICE_DEBUG_USBREDIR env var to set the log level
> > for usbredir side. If this is not set, usbredir will behave the same way
> > as it was by checking SPICE_DEBUG env var.
> >
> > WARNING level is the default when neither env var are set.
>
> Why not filtering out the strings you don't want with -v ? In general, with
> the spice-gtk strings, you can easily strip down the log to what you need,
> and this is a lot more flexible than having various environment variables.
I mean filtering with "grep -v"
>
> > ---
> > src/channel-usbredir.c | 12 +++++++++++-
> > 1 file changed, 11 insertions(+), 1 deletion(-)
> >
> > diff --git a/src/channel-usbredir.c b/src/channel-usbredir.c
> > index 0be72ba..84cabe0 100644
> > --- a/src/channel-usbredir.c
> > +++ b/src/channel-usbredir.c
> > @@ -202,6 +202,16 @@ static void channel_set_handlers(SpiceChannelClass
> > *klass)
> >
> > /* ------------------------------------------------------------------ */
> > /* private api */
> > +static gint
> > +usbredir_get_log_level(void)
> > +{
> > + const gchar *debug_env = g_getenv("SPICE_DEBUG_USBREDIR");
> > + if (debug_env != NULL) {
> > + return CLAMP(atoi(debug_env), usbredirparser_none,
> > usbredirparser_debug_data);
> > + }
> > +
> > + return spice_util_get_debug() ? usbredirparser_debug :
> > usbredirparser_warning;
> > +}
> >
> > G_GNUC_INTERNAL
> > void spice_usbredir_channel_set_context(SpiceUsbredirChannel *channel,
> > @@ -223,7 +233,7 @@ void
> > spice_usbredir_channel_set_context(SpiceUsbredirChannel *channel,
> > usbredir_unlock_lock,
> > usbredir_free_lock,
> > channel, PACKAGE_STRING,
> > - spice_util_get_debug() ?
> > usbredirparser_debug : usbredirparser_warning,
> > + usbredir_get_log_level(),
> > usbredirhost_fl_write_cb_owns_buffer);
> > if (!priv->host)
> > g_error("Out of memory allocating usbredirhost");
> > --
> > 2.5.0
> >
> > _______________________________________________
> > Spice-devel mailing list
> > Spice-devel@xxxxxxxxxxxxxxxxxxxxx
> > http://lists.freedesktop.org/mailman/listinfo/spice-devel
> >
>
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/spice-devel