On Tue, 20 Jun 2006 18:32:18 -0300, "Luiz Fernando N. Capitulino" <[EMAIL 
PROTECTED]> wrote:

> | > -         retval = type->attach (serial);
> | > -         module_put(type->driver.owner);
> | > -         if (retval < 0)
> | > -                 goto probe_error;
> | > -         if (retval > 0) {
> | > + retval = driver_attach_it(serial, interface);
> | > + if (retval < 0) {
> | > +         goto probe_error;
> | > + } else if (retval > 0) {

> | You just love adding these else clauses everywhere, don't you.
> 
>  They come from the original code...

The original code is quoted right above and it doesn't have the else.

-- Pete


_______________________________________________
linux-usb-devel@lists.sourceforge.net
To unsubscribe, use the last form field at:
https://lists.sourceforge.net/lists/listinfo/linux-usb-devel

Reply via email to