On Fri, Aug 12, 2011 at 06:39:46PM +0300, Pekka Enberg wrote: >On Fri, 12 Aug 2011, walimis wrote: >>>@@ -24,6 +24,7 @@ struct cmd_struct kvm_commands[] = { >>> { "balloon", kvm_cmd_balloon, NULL, 0 }, >>> { "list", kvm_cmd_list, NULL, 0 }, >>> { "version", kvm_cmd_version, NULL, 0 }, >>>+ { "--version", kvm_cmd_version, NULL, 0 }, >>Although it works, I think it's not good way to implement a option >>as a command. > >Well, I already merged the patch because the whole series was so >super-cool. What downsides are there to implementing the alias like >this? no downsides, I just consider that a option is preceded with "--", but the struct cmd_struct is to describe a command. walimis > > Pekka -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html