On Sat, Oct 19, 2013 at 04:39:09PM +0200, Dominik Paulus wrote:
> +const char *usbip_net_strerror(int status)
> +{
> +     static const char *const errs[] = {
> +             /* ERR_OK */ "Success",
> +             /* ERR_NA */ "Command failed",
> +             /* ERR_MISMATCH */ "Protocol version mismatch",
> +             /* ERR_SYSERR */ "System error",
> +             /* ERR_UNEXPECTED */ "Unexpected opcode received",
> +             /* ERR_AUTHREQ */ "Server requires authentication",
> +             /* ERR_PERM */ "Permission denied",
> +             /* ERR_NOTFOUND */ "Requested device not found",
> +             /* ERR_NOAUTH */ "Server doesn't support authentication"
> +     };
> +     if (status < 0)
> +             status = -status;
> +     if (status >= (int) (sizeof(errs) / sizeof(*errs)))

Not a big deal, but for future reference, the cast here is unneeded.

> +             return "Invalid";
> +     return errs[status];
>  }

_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to