Bug 26244

Summary:	Handle more than just one audio stream.
Product:	Telepathy	Reporter:	David Laban <david.laban>
Component:	fargo	Assignee:	David Laban <david.laban>
Status:	NEW ---	QA Contact:	Simon McVittie <smcv>
Severity:	normal
Priority:	medium	Keywords:	patch
Version:	unspecified
Hardware:	Other
OS:	All
URL:	http://git.collabora.co.uk/?p=user/alsuren/telepathy-fargo.git;a=shortlog;h=refs/heads/video-squashed
Whiteboard:
i915 platform:		i915 features:

Description David Laban 2010-01-26 06:27:40 UTC

Current code assumes that we only want a single audio stream. We should probably relax this assumption.

Comment 1 David Laban 2010-12-09 12:28:04 UTC

I actually ended up doing some serious refactoring to make this work.

I'm pretty happy with the shape of the code now. I need to track down a leak again, but I'd quite like to get some review on the stuff I've got so far.

There are also some other misc bugfixes in that branch, and the actual fix for this bug is in the last commit only. There is also a -unsquashed version of the same branch, but it's more difficult to follow.

Comment 2 David Laban 2010-12-10 07:59:14 UTC

woop! Leak fixed and appended to this branch. That's got to be my first @inlineCallbacks-related leak. objgraph saves the day again.

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.