-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
Greg KH wrote:
| Ah, one more common style fixup that you should fixup:
|
|
|>+static void sisusb_kill_all_busy(struct sisusb_usb_data *sisusb)
|>+{
|>+ int i;
|>+
|>+ if (sisusb_all_free(sisusb)) {
|>+ return;
|>+ }
|
|
| For if and for loops that only have 1 statement, don't add the { } to
| them, it's not needed.
OK, I know it's not needed. But leaving them out is more error prone and
makes the code less readable due to code entities not appearing in the
same way clearly. Don't you think you're a bit too picky now? ;)
Thomas
- --
Thomas Winischhofer
Vienna/Austria
thomas AT winischhofer DOT net http://www.winischhofer.net/
twini AT xfree86 DOT org
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.5 (GNU/Linux)
Comment: Using GnuPG with Thunderbird - http://enigmail.mozdev.org
iD8DBQFB7uoKzydIRAktyUcRAtTXAJ4p4kK2cxTMaZhtmPGTwA1X2vpM8ACgp1hD
jFdwyy3BQC26ENbbfmcf1n8=
=1Wv+
-----END PGP SIGNATURE-----
-------------------------------------------------------
This SF.Net email is sponsored by: IntelliVIEW -- Interactive Reporting
Tool for open source databases. Create drag-&-drop reports. Save time
by over 75%! Publish reports on the web. Export to DOC, XLS, RTF, etc.
Download a FREE copy at http://www.intelliview.com/go/osdn_nl
_______________________________________________
linux-usb-devel@lists.sourceforge.net
To unsubscribe, use the last form field at:
https://lists.sourceforge.net/lists/listinfo/linux-usb-devel
