On Sun, 30 Jan 2005 15:28:23 +0300, Michael Tokarev <[EMAIL PROTECTED]> wrote:
> We have sizeof(), don't we? Ie, why > snprintf(hid->name, 128, "%s", buf); > instead of > snprintf(hid->name, sizeof(hid->name), "%s", buf); I always fight this sort of patches, because the moment hid->name becomes a pointer (and it's almost inevitable for most structures), this scheme breaks down. The right approach is to have a symbolic constant HID_NAME_SIZE or such. -- Pete ------------------------------------------------------- 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 _______________________________________________ [email protected] To unsubscribe, use the last form field at: https://lists.sourceforge.net/lists/listinfo/linux-usb-devel
