Re: [PATCH 06/12] Remove the (broken) passthru option

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Nit: I'd add a "vscclient:" prefix to the log.

On Thu, Oct 08, 2015 at 05:40:35PM +0200, marcandre.lureau@xxxxxxxxxx wrote:
> From: Jeremy White <jwhite@xxxxxxxxxxxxxxx>
> 
> That option can be achieved using -e "use_hw=yes hw_type=passthru"

Would the new "-e passthru" alias work here too?

Christophe

> 
> Signed-off-by: Jeremy White <jwhite@xxxxxxxxxxxxxxx>
> Reviewed-by: Marc-André Lureau <marcandre.lureau@xxxxxxxxxx>
> ---
>  src/vscclient.c | 16 +++-------------
>  1 file changed, 3 insertions(+), 13 deletions(-)
> 
> diff --git a/src/vscclient.c b/src/vscclient.c
> index e9fd3aa..dd20842 100644
> --- a/src/vscclient.c
> +++ b/src/vscclient.c
> @@ -49,14 +49,8 @@ print_byte_array(
>  
>  static void
>  print_usage(void) {
> -    printf("vscclient [-c <certname> .. -e <emul_args> -d <level>%s] "
> -            "<host> <port>\n",
> -#ifdef USE_PASSTHRU
> -    " -p");
> -    printf(" -p use passthrough mode\n");
> -#else
> -   "");
> -#endif
> +    printf("vscclient [-c <certname> .. -e <emul_args> -d <level>] "
> +            "<host> <port>\n");
>      vcard_emul_usage();
>  }
>  
> @@ -684,7 +678,7 @@ main(
>      }
>  #endif
>  
> -    while ((c = getopt(argc, argv, "c:e:pd:")) != -1) {
> +    while ((c = getopt(argc, argv, "c:e:d:")) != -1) {
>          switch (c) {
>          case 'c':
>              if (cert_count >= MAX_CERTS) {
> @@ -696,10 +690,6 @@ main(
>          case 'e':
>              emul_args = optarg;
>              break;
> -        case 'p':
> -            print_usage();
> -            exit(4);
> -            break;
>          case 'd':
>              verbose = get_id_from_string(optarg, 1);
>              break;
> -- 
> 2.4.3
> 
> _______________________________________________
> Spice-devel mailing list
> Spice-devel@xxxxxxxxxxxxxxxxxxxxx
> http://lists.freedesktop.org/mailman/listinfo/spice-devel

Attachment: signature.asc
Description: PGP signature

_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/spice-devel

[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]