<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, May 26, 2014 at 10:52 AM, Christophe Fergeau <span dir="ltr"><<a href="mailto:cfergeau@redhat.com" target="_blank">cfergeau@redhat.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div>On Sat, May 24, 2014 at 02:49:39AM +0200, Fabiano Fidêncio wrote:<br>
> osinfo_loader_nodeset() can return -1 without set an error<br>
<br>
</div>Do you have more details about the scenario in which it occurs? It seems<br>
this can only happen happening through one of the<br>
g_return_val_if_fail()?<br></blockquote><div><br></div><div>You're right, this is the only possible scenario.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


g_return_if_fail() are used to flag programmers' errors, after such an<br>
error is reported, behaviour is undefined, so we generally don't try to<br>
handle them.<br></blockquote><div><br></div><div>I tried to handle the g_return_var_if_fail().<br></div><div>If we generally don't try to handle then, just ignore the patch.</div><div> </div></div><div>Best Regards,</div>
-- <br><div>Fabiano Fidêncio</div>
</div></div>