Re: [libvirt] [PATCH] virsh-domain: Remove unnecessary check and tune code in cmdDesc()

2013-09-17 Thread Hongwei Bi
I understand. Thanks for your reminding. 2013/9/17 Ján Tomko > On 09/17/2013 11:16 AM, Hongwei Bi wrote: > > Since there is a check on buf through virBufferError(), > > it is not necessary to check desc again. > > The buffer is used for the description provided on the virsh command line: > $ v

Re: [libvirt] [PATCH] virsh-domain: Remove unnecessary check and tune code in cmdDesc()

2013-09-17 Thread Ján Tomko
On 09/17/2013 11:16 AM, Hongwei Bi wrote: > Since there is a check on buf through virBufferError(), > it is not necessary to check desc again. The buffer is used for the description provided on the virsh command line: $ virsh desc Description If you don't provide the description: $ virsh desc -

[libvirt] [PATCH] virsh-domain: Remove unnecessary check and tune code in cmdDesc()

2013-09-17 Thread Hongwei Bi
Since there is a check on buf through virBufferError(), it is not necessary to check desc again. --- tools/virsh-domain.c | 62 + 1 files changed, 27 insertions(+), 35 deletions(-) diff --git a/tools/virsh-domain.c b/tools/virsh-domain.c index e47