On Fri, Nov 23, 2012 at 8:47 AM, Dong Xu Wang <wdon...@linux.vnet.ibm.com> wrote: > +/* Create a new QemuOptsList and make its desc to the merge of first and > second. > + * It will allocate space for one new QemuOptsList plus enouth space for
s/enouth/enough/ > + dest->name = "append_opts_list"; > + dest->implied_opt_name = NULL; > + dest->merge_lists = false; > + QTAILQ_INIT(&dest->head); > + while (first && (first->desc[i].name)) { > + if (!find_desc_by_name(dest->desc, first->desc[i].name)) { > + dest->desc[index].name = g_strdup(first->desc[i].name); > + dest->desc[index].help = g_strdup(first->desc[i].help); > + dest->desc[index].type = first->desc[i].type; > + dest->desc[index].def_print_str = > + g_strdup(first->desc[i].def_print_str); > + ++index; > + } > + i++; 4-space indentation? scripts/checkpatch.pl normally detects problems with whitespace.