On 03/14/13 17:36, Hans de Goede wrote: > + if (!dev->watch) { > + dev->watch = qemu_chr_fe_add_watch(dev->cs, G_IO_OUT, > + usbredir_write_unblocked, > dev); > + } > + if (r < 0) > + r = 0; > + } > + return r;
=== checkpatch complains === WARNING: braces {} are necessary for all arms of this statement #48: FILE: hw/usb/redirect.c:290: + if (r < 0) [...] total: 0 errors, 1 warnings, 65 lines checked cheers, Gerd