[PATCH libevdev 1/4] Shut up compiler warning

Peter Hutterer peter.hutterer at who-t.net
Tue Mar 18 15:59:27 PDT 2014


On Tue, Mar 18, 2014 at 10:16:44AM -0400, Benjamin Tissoires wrote:
> On Tue, Mar 18, 2014 at 12:20 AM, Peter Hutterer
> <peter.hutterer at who-t.net> wrote:
> > Triggered with -O2
> 
> Weird, on my box, I manage to trigger it with -O or -Og, but not -O2 nor -O0...

I'll fix up the message. could've been with -O, I messed around with the
flags yesterday, not sure which one exactly triggered this one.

Cheers,
   Peter

> This series is reviewed-by: Benjamin Tissoires <benjamin.tissoires at gmail.com>
> 
> Cheers,
> Benjamin
> 
> >
> > ../libevdev/libevdev.c: In function ‘libevdev_has_event_code’:
> > ../libevdev/libevdev-util.h:58:20: warning: ‘mask’ may be used uninitialized
> > in this function [-Wmaybe-uninitialized]
> >
> > Optimisation clearly shuffles things around here: in the code, if no max is
> > found, we return -1 and bail out before we access mask.
> >
> > Signed-off-by: Peter Hutterer <peter.hutterer at who-t.net>
> > ---
> >  libevdev/libevdev.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/libevdev/libevdev.c b/libevdev/libevdev.c
> > index 84d9195..b73c5f0 100644
> > --- a/libevdev/libevdev.c
> > +++ b/libevdev/libevdev.c
> > @@ -1089,7 +1089,7 @@ libevdev_has_event_type(const struct libevdev *dev, unsigned int type)
> >  LIBEVDEV_EXPORT int
> >  libevdev_has_event_code(const struct libevdev *dev, unsigned int type, unsigned int code)
> >  {
> > -       const unsigned long *mask;
> > +       const unsigned long *mask = NULL;
> >         int max;
> >
> >         if (!libevdev_has_event_type(dev, type))
> > --
> > 1.8.5.3
> >
> > _______________________________________________
> > Input-tools mailing list
> > Input-tools at lists.freedesktop.org
> > http://lists.freedesktop.org/mailman/listinfo/input-tools
> 


More information about the Input-tools mailing list