> -----Original Message-----
> From: Peter Hutterer [mailto:[email protected]]
> Sent: Tuesday, April 22, 2014 5:43 PM
> To: Eoff, Ullysses A
> Cc: [email protected]
> Subject: Re: [PATCH weston v2 2/5] libinput-seat: allow setting libinput log
> priority in weston
>
> On Thu, Apr 17, 2014 at 07:53:23AM -0700, U. Artie Eoff wrote:
> > Look for WESTON_LIBINPUT_LOG_PRIORITY environment variable. If
> > it exists then use it to set the libinput log priority.
> > Otherwise, don't set the priority and get whatever libinput's
> > default priority is.
> >
> > Setting WESTON_LIBINPUT_LOG_PRIORITY=0 allows us to log which
> > input devices are detected at Weston startup and makes it a
> > little more consistent with Weston's original evdev input setup
> > log messages... and useful for debugging and testing.
> >
> > Signed-off-by: U. Artie Eoff <[email protected]>
>
> > ---
> > src/libinput-seat.c | 7 +++++++
> > 1 file changed, 7 insertions(+)
> >
> > diff --git a/src/libinput-seat.c b/src/libinput-seat.c
> > index 99612c4..7aca969 100644
> > --- a/src/libinput-seat.c
> > +++ b/src/libinput-seat.c
> > @@ -262,12 +262,19 @@ int
> > udev_input_init(struct udev_input *input, struct weston_compositor *c,
> > struct udev *udev,
> > const char *seat_id)
> > {
> > + const char *log_priority = NULL;
> > +
> > memset(input, 0, sizeof *input);
> >
> > input->compositor = c;
> >
> > libinput_log_set_handler(&libinput_log_func, NULL);
> >
> > + log_priority = getenv("WESTON_LIBINPUT_LOG_PRIORITY");
> > + if (log_priority) {
> > + libinput_log_set_priority(strtol(log_priority, NULL, 10));
> > + }
> > +
>
> I see this has been merged already, but I think it'd be better to have this
> with the semantic names than the numeric values, e.g.
> WESTON_LIBINPUT_LOG_PRIORITY=error
> it's merely a few added strcmps and the result is more copy/paste proof than
> pure numbers.
>
> Cheers,
> Peter
>
Yes, that would probably be better... I won't be able to get around to writing
a
new patch for a few days however. Feel free to beat me to it ;).
U. Artie
> > input->libinput = libinput_udev_create_for_seat(&libinput_interface,
> > input,
> > udev, seat_id);
> > if (!input->libinput) {
> > --
> > 1.8.5.3
_______________________________________________
wayland-devel mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/wayland-devel
