Re: [pulseaudio-discuss] [PATCH v2] call-state-tracker: New component.

2011-04-20 Thread Tanu Kaskinen
On Wed, 2011-04-20 at 15:19 +0300, Colin Guthrie wrote: > OK, so the manager object is separate but kept in the core? Seems fine > by me to keep things neatly separated. Yep. > I thinking of taking a pop at this at some point soon (if you don't beat > me to it), so can we decide on the name now?

Re: [pulseaudio-discuss] [PATCH v2] call-state-tracker: New component.

2011-04-20 Thread Colin Guthrie
'Twas brillig, and Tanu Kaskinen at 20/04/11 11:54 did gyre and gimble: > On Wed, 2011-04-20 at 11:36 +0300, Colin Guthrie wrote: >>> The api object header could be located in >>> src/extra_apis - that would make the api not directly dependent on a >>> particular module. >> >> If the API is not de

Re: [pulseaudio-discuss] [PATCH v2] call-state-tracker: New component.

2011-04-20 Thread Tanu Kaskinen
On Wed, 2011-04-20 at 11:36 +0300, Colin Guthrie wrote: > > The api object header could be located in > > src/extra_apis - that would make the api not directly dependent on a > > particular module. > > If the API is not dependant on a particular module, what code calls > pa_extra_api_register? I'

Re: [pulseaudio-discuss] [PATCH v2] call-state-tracker: New component.

2011-04-20 Thread Colin Guthrie
'Twas brillig, and Tanu Kaskinen at 18/04/11 10:31 did gyre and gimble: > For publishing APIs: > > /* Returns a negative error code if the extra api is already registered. */ > int pa_extra_api_register(const char *name, void *api); > > void pa_extra_api_unregister(const_char *name); > > For con

Re: [pulseaudio-discuss] [PATCH v2] call-state-tracker: New component.

2011-04-18 Thread Tanu Kaskinen
On Mon, 2011-04-18 at 11:35 +0300, Colin Guthrie wrote: > > I'd be interested in implementing at some point (no promises or > > timelines) a small framework for making inter-module communication > > easier, or at least cleaner (this kind of hacks in pulsecore are > > actually very easy to work with

Re: [pulseaudio-discuss] [PATCH v2] call-state-tracker: New component.

2011-04-18 Thread Colin Guthrie
'Twas brillig, and Tanu Kaskinen at 06/04/11 16:23 did gyre and gimble: > On Wed, 2011-04-06 at 14:23 +0100, Colin Guthrie wrote: >> 'Twas brillig, and Tanu Kaskinen at 06/04/11 12:33 did gyre and gimble: >>> +/* This is a shared singleton object, currently used by Meego's voice and >>> + * policy

Re: [pulseaudio-discuss] [PATCH v2] call-state-tracker: New component.

2011-04-06 Thread Tanu Kaskinen
On Wed, 2011-04-06 at 14:23 +0100, Colin Guthrie wrote: > 'Twas brillig, and Tanu Kaskinen at 06/04/11 12:33 did gyre and gimble: > > +/* This is a shared singleton object, currently used by Meego's voice and > > + * policy enforcement modules. The purpose of the object is just to > > maintain > >

Re: [pulseaudio-discuss] [PATCH v2] call-state-tracker: New component.

2011-04-06 Thread Colin Guthrie
'Twas brillig, and Tanu Kaskinen at 06/04/11 12:33 did gyre and gimble: > +/* This is a shared singleton object, currently used by Meego's voice and > + * policy enforcement modules. The purpose of the object is just to maintain > + * a boolean state of "call is active" or "call is not active", and

[pulseaudio-discuss] [PATCH v2] call-state-tracker: New component.

2011-04-06 Thread Tanu Kaskinen
From: Tanu Kaskinen See call-state-tracker.h for details. Changed in patch v2: Don't take a reference of pa_core. The previous version would have reintroduced the bug that was fixed in 1e381fbf. --- src/Makefile.am|1 + src/pulsecore/call-state-tracker.c | 125 +