> + } else do {
> +
> + // create a new image and check for success
> + result = idmouse_create_image (dev);
> + if (result)
> + break;
> +
> + // increment our usage count for the driver
> + ++dev->open;
> +
> + // save our object in the file's private structure
> + file->private_data = dev;
> +
> + } while (0);
What do you intend with that loop?
If you want goto, use it, not break.
[..]
> + --dev->open;
> +
> + if (!dev->present && !dev->open) {
How could dev->open not be 0?
Regards
Oliver
-------------------------------------------------------
This SF.Net email is sponsored by OSTG. Have you noticed the changes on
Linux.com, ITManagersJournal and NewsForge in the past few weeks? Now,
one more big change to announce. We are now OSTG- Open Source Technology
Group. Come see the changes on the new OSTG site. www.ostg.com
_______________________________________________
[EMAIL PROTECTED]
To unsubscribe, use the last form field at:
https://lists.sourceforge.net/lists/listinfo/linux-usb-devel
