[Pidgin] #14936: Port to Farstream
Pidgin
trac at pidgin.im
Tue Apr 3 20:49:35 EDT 2012
#14936: Port to Farstream
-----------------------------------------+----------------------------------
Reporter: ocrete | Owner: Maiku
Type: patch | Status: new
Milestone: Patches Needing Improvement | Component: Voice and Video
Version: 2.10.1 | Resolution:
Keywords: |
-----------------------------------------+----------------------------------
Comment(by rlaager):
I don't know about that. The change to libpurple/media.c seems like it
could be accepted immediately, as it seems like a trivial cleanup
unrelated to farsight->farstream.
The configure test could support both with minor tweaks. (The farsight
check could set FARSTREAM_CFLAGS, to avoid extra changes to the
Makefile.am.)
The pile of #ifdefs is probably still less code than duplicating
libpurple/media/backend-fs2.c.
I'm not sure why we have a dependency on farsight in
purple_media_manager_get_pipeline(), but I don't really understand the vv
code because I haven't looked at it much. I agree it'd be nice to get rid
of that dependency, if possible.
--
Ticket URL: <http://developer.pidgin.im/ticket/14936#comment:9>
Pidgin <http://pidgin.im>
Pidgin
More information about the Tracker
mailing list