On Thu, Nov 10, 2016 at 08:49:43AM +0100, Hans Verkuil wrote: > The daily build produces this compiler warning: > > dib0700_core.c: In function 'dib0700_rc_urb_completion': > dib0700_core.c:787:2: warning: 'protocol' may be used uninitialized in this function [-Wmaybe-uninitialized] > rc_keydown(d->rc_dev, protocol, keycode, toggle); > ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > > This is indeed correct as there is a path in that function where protocol is > uninitialized, but I lack the knowledge how this should be fixed. > > Mauro, can you take a look? > > It goes wrong in the switch in case RC_BIT_NEC if the first 'if' is true. > Note that keycode is also uninitialized, but it is declared as uninitialized_var(), > although why you would want to do that instead of just initializing it to 0 or > something like that is a mystery to me. This is already solved in this patch: https://patchwork.linuxtv.org/patch/37516/ Sean -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
