[Nice] complete example
rohan16garg at gmail.com
Thu Jan 31 23:07:43 PST 2013
Hey everyone :)
I went over the program a bit, and I'm attaching a patch that fixes a
couple of issues, just minor stuff regarding the thread_mainloop and
making the code a wee bit more async.
There also seems to be a bug on line 241 of the original code where it
only stops at \0 , I believe that should be \n. When using \0 the
program simply crashes at " remote_candidates =
g_slist_reverse(remote_candidates); " in parse_remote_data.
On Fri, Feb 1, 2013 at 6:05 AM, Youness Alaoui
> I have a plan of adding a new API call for transforming the candidates into SDP
> and to parse candidates from SDP. I think it will help people, and I think it
> would be a good idea to port your code to it.
> I'll let you know tomorrow what I find!
Could you explain this a bit on the list in another thread? :)
> On 01/31/2013 07:30 PM, Bryce Allen wrote:
>> I expanded my nicetest.c into a complete example, using a simple
>> serialization format and copying/pasting the data between clients. It's
>> inspired by icedemo from pjnath, but I didn't find having separate
>> commands for each phase useful for testing and it made reading the code
>> more difficult, so I just have three phases:
>> 1) gather candidates, print serialized local data
>> 2) prompt user for remote data, parse and set when entered
>> 3) if successful, prompt for data to send to remote end
>> I'm probably missing some failure cases and some error checking, but it
>> seems like it could be a useful addition to the main distribution after
>> a fixup. I can re-license under LGPL/MPL if needed.
>> Also please let me know if I'm doing anything fundamentally wrong here.
>> I'm using pretty much the same flow for a real application, with
>> condition variables to synchronize instead busy waiting.
>> nice mailing list
>> nice at lists.freedesktop.org
> nice mailing list
> nice at lists.freedesktop.org
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 2348 bytes
Desc: not available
More information about the nice