<div dir="ltr">Patch adds a basic check to test-thread.c. It checks that the return value is NULL before negotiation is complete, and non-NULL after. Not sure it's worth trying to make a more elaborate test, what do you think?<br>
</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Nov 20, 2013 at 3:48 PM, Olivier Crête <span dir="ltr"><<a href="mailto:olivier.crete@collabora.com" target="_blank">olivier.crete@collabora.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
<br>
I applied your patch, a test would be greatly appreciated.<br>
<br>
Olivier<br>
<div class="HOEnZb"><div class="h5"><br>
On Sat, 2013-11-16 at 17:17 -0600, Bryce Allen wrote:<br>
> Forgot to CC the list. I'll work on adding a test too, as Rohan<br>
> suggested.<br>
><br>
> -Bryce<br>
><br>
><br>
> Begin forwarded message:<br>
><br>
> Date: Thu, 14 Nov 2013 16:35:25 -0600<br>
> From: Bryce Allen <<a href="mailto:ballen@ci.uchicago.edu">ballen@ci.uchicago.edu</a>><br>
> To: Olivier Crête <<a href="mailto:olivier.crete@collabora.com">olivier.crete@collabora.com</a>><br>
> Subject: Re: [Nice] patch: nice_agent_get_selected_socket<br>
><br>
><br>
> On Thu, 14 Nov 2013 17:10:58 -0500<br>
> Olivier Crête <<a href="mailto:olivier.crete@collabora.com">olivier.crete@collabora.com</a>> wrote:<br>
> > I also think returning a GSocket is the right level.<br>
> ><br>
> > I think you're not the first one to want this. One question, why do<br>
> > you just not return a GSocket * or NULL instead of having a separate<br>
> > boolean and GSocket** ?<br>
> ><br>
> I was trying to be consistent with nice_agent_get_selected_pair, but<br>
> I guess it's using out variables because it needs to return two<br>
> things. Attached is a modified version that returns GSocket *.<br>
><br>
> Thanks,<br>
> Bryce<br>
</div></div><div class="HOEnZb"><div class="h5">> _______________________________________________<br>
> nice mailing list<br>
> <a href="mailto:nice@lists.freedesktop.org">nice@lists.freedesktop.org</a><br>
> <a href="http://lists.freedesktop.org/mailman/listinfo/nice" target="_blank">http://lists.freedesktop.org/mailman/listinfo/nice</a><br>
<br>
</div></div><span class="HOEnZb"><font color="#888888">--<br>
Olivier Crête<br>
<a href="mailto:olivier.crete@collabora.com">olivier.crete@collabora.com</a><br>
<br>
</font></span></blockquote></div><br></div>