<div dir="ltr"><pre style="margin-top:0px;margin-bottom:0px;padding:0px;font-family:arial,'courier new',courier,ËÎÌå,monospace;white-space:pre-wrap;word-wrap:break-word;color:rgb(51,51,51);font-size:14px;line-height:24px;background-color:rgb(241,254,221)">
I understand. Thanks for your reminding.</pre></div><div class="gmail_extra"><br><br><div class="gmail_quote">2013/9/17 J¨¢n Tomko <span dir="ltr"><<a href="mailto:jtomko@redhat.com" target="_blank">jtomko@redhat.com</a>></span><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On 09/17/2013 11:16 AM, Hongwei Bi wrote:<br>
> Since there is a check on buf through virBufferError(),<br>
> it is not necessary to check desc again.<br>
<br>
</div>The buffer is used for the description provided on the virsh command line:<br>
$ virsh desc <domain> Description<br>
<br>
If you don't provide the description:<br>
<br>
$ virsh desc --edit <domain><br>
<br>
then 'desc' is NULL and we need to fetch the existing domain description and<br>
let the user edit it. Your patch removes this functionality.<br>
<br>
Jan<br>
<br>
</blockquote></div><br></div>