On Mon, Jan 25, 2010 at 12:39:42PM +0300, Sergei Shtylyov wrote:
> >> diff -urp -X dontdiff linux-2.6-gregkh/drivers/usb/serial/generic.c linux-2.6-tip/drivers/usb/serial/generic.c
> >> --- linux-2.6-gregkh/drivers/usb/serial/generic.c 2010-01-22 13:50:50.433875777 -0700
> >> +++ linux-2.6-tip/drivers/usb/serial/generic.c 2010-01-05 11:32:02.346260069 -0700
> >> @@ -387,10 +387,13 @@ int usb_serial_generic_chars_in_buffer(s
> >> dbg("%s - port %d", __func__, port->number);
> >>
> >> spin_lock_irqsave(&port->lock, flags);
> >> - if (serial->type->max_in_flight_urbs)
> >> + if (serial->type->max_in_flight_urbs) {
> >>
> >
> > Superfluous {.
> >
>
> It's not really. CodingStyle has it that if you have {} on one of the
> *if* branches, the other branch should also have it.
My mistake, sorry.
Thanks,
Johan
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
