Thanks for the patch! (I did find it after all). A couple of minor
documentation things:

Pavel Machek <[EMAIL PROTECTED]> writes:

[...]

> +static
> +int
> +command_help( command_context* context, char*buffer )
> +{
> +  printf( " \
> +config [configuration number]\n\
> +interface [interface]\n\
> +transfer type=b size=[bytes] ep=[endpoint number] dir=[in,out] timeout=[frames]\n\
> +abort -- go and try another device with same ids\n\
> +quit\n\
> +help\n" );
> +  return 0;
> +}

I'd prefer it if the documentation string for each function were
included in the name/procedure table below. I've seen too many
programs where the command line "help" has been woefully outofdate.

>  
>  static
>  int
> @@ -374,6 +411,9 @@
>      },
>      {
>        "abort", command_abort
> +    },
> +    {
> +      "help", command_help
>      },
>      {
>        "quit", command_quit
> --- ofic.usb-robot//README    Fri Apr 14 17:48:50 2000
> +++ usb-robot-0.1.3/README    Thu Jun 29 10:42:18 2000
> @@ -61,5 +61,6 @@
>  transfer type=b size=[bytes] ep=[endpoint number] dir=[in,out] timeout=[frames]

Should be type=b or c

[...]

Unfortunately, I haven't any device that takes control packets so it
won't undergo any testing from me.

Otherwise I'm very happy to apply it! Thankyou Pavel!

-- 

        http://web.onetel.net.uk/~elephant/john

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to