On 03/08/2013 12:14 AM, Chris Evich wrote:
> On 02/28/2013 03:05 AM, Yu Mingfei wrote:
>> +                    paused_after_start_vm = yes
>> +                    dumpxml_vm_state = paused
>
> Overall, LGTM.
>
> One suggestion/reminder:  Try to put "'s around string values to
> protect against trailing whitespace.  i.e. "yes" != " yes" != " yes "
> != "yes\n".  It's not critical, it just makes my left eye-twitch from
> OCD :D
>

Thanks, you are right.

Yu

_______________________________________________
Virt-test-devel mailing list
[email protected]
https://www.redhat.com/mailman/listinfo/virt-test-devel

Reply via email to