> On Aug 2, 2018, at 10:10 PM, Fam Zheng <f...@redhat.com> wrote: > > On Thu, 08/02 20:50, John Arbuckle wrote: >> Add an examples section to the help output. >> >> Signed-off-by: John Arbuckle <programmingk...@gmail.com> >> --- >> qemu-img.c | 11 +++++++++++ >> 1 file changed, 11 insertions(+) >> >> diff --git a/qemu-img.c b/qemu-img.c >> index 1acddf693c..f77c82695d 100644 >> --- a/qemu-img.c >> +++ b/qemu-img.c >> @@ -199,6 +199,17 @@ static void QEMU_NORETURN help(void) >> >> printf("%s\nSupported formats:", help_msg); >> bdrv_iterate_format(format_print, NULL); >> + >> + printf("\n\nExamples:\n\n" >> + "Create: qemu-img create -f qcow2 image.qcow2 10G\n\n" >> + "Info: qemu-img info image.qcow2\n\n" >> + "Resize: qemu-img resize image.qcow2 20G\n\n" >> + "Convert: qemu-img convert -f raw -O qcow2 image.img >> image.qcow2\n\n" >> + "Check: qemu-img check image.qcow2\n\n" >> + "Map: qemu-img map -f qcow2 --output=human image.qcow2\n\n" >> + "Rebase: qemu-img rebase -b new_backing_file.qcow2 image.qcow2" >> + ); > > The text looks good but maybe it's better to condense the section by using > only > one \n between command lines instead of two? > > Fam
It would be harder on the user's eyes if I did that. Thank you for reviewing my patch.