<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Feb 28, 2014 at 12:19 AM, Aleksander Morgado <span dir="ltr"><<a href="mailto:aleksander@aleksander.es" target="_blank">aleksander@aleksander.es</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5"><br>
</div></div>This is actually wrong.<br>
<br>
This is the _finish() method, and the target of the method is to get the<br>
result or error from the GAsyncResult and propagate it to the output GError.<br>
<br>
If the GAsyncResult had an error, it is propagated to the output GError<br>
at the beginning of the method:<br>
    if (g_simple_async_result_propagate_error (G_SIMPLE_ASYNC_RESULT<br>
(res), error))<br>
        return NULL;<br>
<br>
If the GAsyncResult didn't have an error, we keep on building the info<br>
list to return. If building the list fails, we *don't* need to propagate<br>
that GError to the GAsyncResult, we just need to make sure that the<br>
output GError gets it, and that is already done implicitly (both<br>
mbim_message_command_done_get_result() and<br>
mbim_message_visible_providers_response_parse() use directly the output<br>
GError to set the possible error).<br>
<span class="HOEnZb"><font color="#888888"><br></font></span></blockquote><div><br></div><div>Oops...sorry, I must have a blank moment today :p  I was scanning for missing checks on command-done messages and came across this function... and I didn't recognize my code</div>

</div></div></div>