[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] qemu-img.c: add help for each command
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [PATCH] qemu-img.c: add help for each command |
Date: |
Sat, 8 Sep 2018 04:13:23 +0100 |
On 8 September 2018 at 04:01, John Arbuckle <address@hidden> wrote:
> + /* print the help for this command */
> + if (strcmp("--help", argv[optind + 1]) == 0) {
> + if (strcmp("amend", cmdname) == 0) {
> + help_amend();
> + } else if (strcmp("bench", cmdname) == 0) {
> + help_bench();
> + } else if (strcmp("check", cmdname) == 0) {
> + help_check();
> + } else if (strcmp("commit", cmdname) == 0) {
> + help_commit();
> + } else if (strcmp("compare", cmdname) == 0) {
> + help_compare();
> + } else if (strcmp("convert", cmdname) == 0) {
> + help_convert();
> + } else if (strcmp("create", cmdname) == 0) {
> + help_create();
> + } else if (strcmp("dd", cmdname) == 0) {
> + help_dd();
> + } else if (strcmp("info", cmdname) == 0) {
> + help_info();
> + } else if (strcmp("map", cmdname) == 0) {
> + help_map();
> + } else if (strcmp("measure", cmdname) == 0) {
> + help_measure();
> + } else if (strcmp("snapshot", cmdname) == 0) {
> + help_snapshot();
> + } else if (strcmp("rebase", cmdname) == 0) {
> + help_rebase();
> + } else if (strcmp("resize", cmdname) == 0) {
> + help_resize();
Any time you find yourself writing very repetitive code like
this, it's a good idea to ask yourself "is there a way to make
this data-driven?".
thanks
-- PMM