[Pidgin] #14936: Port to Farstream

Pidgin trac at pidgin.im
Thu Feb 16 16:56:42 EST 2012


#14936: Port to Farstream
------------------------------------+---------------------------------------
 Reporter:  ocrete                  |        Owner:  Maiku          
     Type:  patch                   |       Status:  new            
Milestone:  Patches Needing Review  |    Component:  Voice and Video
  Version:  2.10.1                  |   Resolution:                 
 Keywords:                          |  
------------------------------------+---------------------------------------

Comment(by ocrete):

 Actually, in farsight2, there is only a fs_stream_set_remote_candidates().
 In Farstream, this has been split into fs_stream_add_remote_candidates()
 and fs_stream_force_remote_candidates(). The main difference between them
 is that _force_() implies that there is no negotiation, while _add_()
 gives them to some negotiation engine (like libnice).

 So this change is purely for Farstream.

 The only bit of the patch that can be used with Farsight2 is the use of
 the default element properties from Farsight2/Farstream, but even this
 requires a relatively recent version of Farsight2.

 The rest is pure porting. Farstream has a couple new APIs that can reduce
 the amount of code a bit, but I didn't even port to those.

 We can add ifdefs all over, and since Farsight2 and Farstream are not
 parallel installable, there is no point in having separate backends.

-- 
Ticket URL: <http://developer.pidgin.im/ticket/14936#comment:3>
Pidgin <http://pidgin.im>
Pidgin


More information about the Tracker mailing list