On Tue, 20 Jun 2006 22:47:52 -0700
Pete Zaitcev <[EMAIL PROTECTED]> wrote:
| 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.
Oh.. It has an 'if'. Are you agains't the improved if-else-if
version? Why?
--
Luiz Fernando N. Capitulino
_______________________________________________
linux-usb-devel@lists.sourceforge.net
To unsubscribe, use the last form field at:
https://lists.sourceforge.net/lists/listinfo/linux-usb-devel
