[Pidgin] #34: Voice/video support

Pidgin trac at pidgin.im
Tue Mar 18 16:28:36 EDT 2008


#34: Voice/video support
--------------------------+-------------------------------------------------
  Reporter:  MarkDoliner  |       Owner:  seanegan       
      Type:  enhancement  |      Status:  assigned       
  Priority:  minor        |   Milestone:  Patches welcome
 Component:  libpurple    |     Version:                 
Resolution:               |    Keywords:                 
   Pending:  0            |  
--------------------------+-------------------------------------------------
Comment (by malu):

 Replying to [comment:88 malu]:
 > Replying to [comment:86 malu]:
 >
 > Added a new patch.
 > This patch is against the current (as of 2008-03-18) i.p.p.vv
 >
 > Replaced fprintf with purple_debug_info.
 > Changed malloc and free to g_new and g_free.
 >
 > This version changed send "content-replace" instead of "content-
 modified" when connection established (to comply with updated XEP).
 >
 > It also shows when the call has started in the initiator's conversation
 window.
 >
 > Issues:
 > * Responder does not send "content-accept" when recieving "session-
 initiate" (I get a crash when seding it, which I haven't figured out
 yet...)
 >
 > * There is still two call widgets added to the conversation window of
 the initiator.
 >
 > * Calling again after having hanged up doesn't work.
 >
 > * It doesn't handle the case when no codecs of the initator and
 responder matches.
 >
 > * There is no way for the initiator to cancel a call before the
 responder as answered.

 And I forgot to mention:
 I changed the functions can_[recieve|send]_[audio|video] with one function
 in the prpl's
 can_do_media that takes a PurpleConnection, a buddy (const char *who) and
 a PurpleMediaType and returns TRUE or FALSE depending on if that
 configuration will be supported.

 (Sadrul, I don't know if you check this in your finch modification...)

 Anyway, currently this uncondionally returns TRUE for XMPP.

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


More information about the Tracker mailing list