Re: [Qemu-block] [Qemu-devel] [PATCH] qemu-img.c: Add examples section
> On Aug 3, 2018, at 4:46 AM, Fam Zheng wrote: > > On Thu, 08/02 22:34, Programmingkid wrote: >> >>> On Aug 2, 2018, at 10:10 PM, Fam Zheng wrote: >>> >>> On Thu, 08/02 20:50, John Arbuckle wrote: Add an examples section to the help output. Signed-off-by: John Arbuckle --- 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. > > This is objective, but being compact is more consistent in density with the > above "Command syntax" section in this function. I want to break away from that very dense format. It was just too hard to read.
Re: [Qemu-block] [Qemu-devel] [PATCH] qemu-img.c: Add examples section
On Thu, 08/02 22:34, Programmingkid wrote: > > > On Aug 2, 2018, at 10:10 PM, Fam Zheng wrote: > > > > On Thu, 08/02 20:50, John Arbuckle wrote: > >> Add an examples section to the help output. > >> > >> Signed-off-by: John Arbuckle > >> --- > >> 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. This is objective, but being compact is more consistent in density with the above "Command syntax" section in this function. Fam
Re: [Qemu-block] [Qemu-devel] [PATCH] qemu-img.c: Add examples section
> On Aug 2, 2018, at 10:10 PM, Fam Zheng wrote: > > On Thu, 08/02 20:50, John Arbuckle wrote: >> Add an examples section to the help output. >> >> Signed-off-by: John Arbuckle >> --- >> 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.
Re: [Qemu-block] [Qemu-devel] [PATCH] qemu-img.c: Add examples section
On Thu, 08/02 20:50, John Arbuckle wrote: > Add an examples section to the help output. > > Signed-off-by: John Arbuckle > --- > 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 > + > printf("\n\n" QEMU_HELP_BOTTOM "\n"); > exit(EXIT_SUCCESS); > } > -- > 2.14.3 (Apple Git-98) > >