<div dir="ltr">Thanks Michal,<div><br></div><div>I will look at another introductory bug to work on. Any suggestions will help.</div><div><br></div><div>Thanks,</div><div>Noella</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Apr 3, 2015 at 1:13 PM, Michal Privoznik <span dir="ltr"><<a href="mailto:mprivozn@redhat.com" target="_blank">mprivozn@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 03.04.2015 12:33, Noella Ashu wrote:<br>
> The error output of snapshot-revert should be more friendly. There is no<br>
> need to show up virDomainRevertToSnapshot to user. virError already includes<br>
>  __FUNCTION__ information in a separate member of the struct, so repeating<br>
> it in the message is redundant and leads to situations where higher level<br>
> code ends up reporting the lower level name We correctly converted the<br>
>  error output making it more succinct and user-friendly.<br>
><br>
> Resolves: <a href="https://bugzilla.redhat.com/show_bug.cgi?id=1086726" target="_blank">https://bugzilla.redhat.com/show_bug.cgi?id=1086726</a><br>
> ---<br>
>  src/libvirt-domain-snapshot.c |  30 +++----<br>
>  src/libvirt-domain.c          | 201 ++++++++++++++++++------------------------<br>
>  2 files changed, 96 insertions(+), 135 deletions(-)<br>
<br>
</span>I guess I should have been more accurate: you need to make sure the code compiles after your patch too. 'make syntax-check' and 'make check' are needed too, but make no sense without pure 'make all'. What's needed is something like following:<br>
<br>
diff --git a/src/libvirt-domain.c b/src/libvirt-domain.c<br>
index f9db3ad..0acfd13 100644<br>
--- a/src/libvirt-domain.c<br>
+++ b/src/libvirt-domain.c<br>
@@ -1438,7 +1438,7 @@ virDomainScreenshot(virDomainPtr domain,<br>
<span class="">     virCheckReadOnlyGoto(domain->conn->flags, error);<br>
<br>
     if (domain->conn != stream->conn) {<br>
</span>-        virReportInvalidArg(stream, "%s",<br>
+        virReportInvalidArg(stream,<br>
<span class="">                             _("stream must match connection of domain '%s'"),<br>
</span>                             domain->name);<br>
         goto error;<br>
@@ -2177,7 +2177,7 @@ virDomainGetMemoryParameters(virDomainPtr domain,<br>
<span class="">     /* At most one of these two flags should be set.  */<br>
     if ((flags & VIR_DOMAIN_AFFECT_LIVE) &&<br>
         (flags & VIR_DOMAIN_AFFECT_CONFIG)) {<br>
</span>-        virReportInvalidArg(flags, "%s",<br>
+        virReportInvalidArg(flags, "%s",<br>
<span class="">                             _("flags 'affect live' and 'affect config' are mutually exclusive"));<br>
         goto error;<br>
     }<br>
</span>....<br>
<br>
Anyway, I've fixed all the compilation errors, ACKed and pushed. Congratulations on your first libvirt contribution!<br>
<span class="HOEnZb"><font color="#888888"><br>
Michal<br>
</font></span></blockquote></div><br></div>