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