[libvirt] [RFC Patch 2/3]virsh: Change log level order

Supriya Kannery supriyak at in.ibm.com
Fri May 20 09:51:15 UTC 2011


On 05/20/2011 05:13 AM, Eric Blake wrote:
> On 05/09/2011 01:08 AM, Supriya Kannery wrote:
>> @@ -2103,7 +2103,7 @@ cmdDominfo(vshControl *ctl, const vshCmd
>>
>>       /* Check and display whether the domain is persistent or not */
>>       persistent = virDomainIsPersistent(dom);
>> -    vshDebug(ctl, 5, "Domain persistent flag value: %d\n", persistent);
>> +    vshDebug(ctl, 4, "Domain persistent flag value: %d\n", persistent);
>
> Please resubmit by using VSH_ERR_DEBUG rather than the magic number '5'
> (or '4') in all of these vshDebug statements.  If we're going to clean
> up this code, let's clean it up right.
>

Thanks! for reviewing this patchset. Posted version 2 of this patch,
which replaces magic numbers with corresponding VSH_ERR_xx.
http://www.mail-archive.com/libvir-list@redhat.com/msg38280.html

Rgds,
Supriya




More information about the libvir-list mailing list