mirrors/pulseaudio
1
0
Fork 0
mirror of https://gitlab.freedesktop.org/pulseaudio/pulseaudio.git synced 2025-11-03 09:01:50 -05:00
Code Issues Projects Releases Packages Wiki Activity Actions

device-manager: Don't notify clients on every subscription (it happens all the time).

Browse source 
Also compare the entries fully before saving.
This commit is contained in:
Colin Guthrie 2009-09-27 16:55:31 +01:00
parent bbf67019df
commit 8977abdc84
1 changed files with 18 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

24
src/modules/module-device-manager.c
View file

@ -323,8 +323,16 @@ static void trigger_save(struct userdata *u) {
} }
static pa_bool_t entries_equal(const struct entry *a, const struct entry *b) { static pa_bool_t entries_equal(const struct entry *a, const struct entry *b) {
/** @todo: Compare the priority lists too */
if (strncmp(a->description, b->description, sizeof(a->description))) pa_assert(a);
pa_assert(b);
if (strncmp(a->description, b->description, sizeof(a->description))
|| strncmp(a->icon, b->icon, sizeof(a->icon)))
return FALSE;
for (int i=0; i < NUM_ROLES; ++i)
if (a->priority[i] != b->priority[i])
return FALSE; return FALSE;
return TRUE; return TRUE;
@ -699,10 +707,6 @@ static void subscribe_callback(pa_core *c, pa_subscription_event_type_t t, uint3
pa_xfree(old); pa_xfree(old);
pa_xfree(name); pa_xfree(name);
/* Even if the entries are equal, the availability or otherwise
of the sink/source may have changed so we notify clients all the same */
notify_subscribers(u);
return; return;
} }
@ -857,6 +861,8 @@ static pa_hook_result_t sink_put_hook_callback(pa_core *c, PA_GCC_UNUSED pa_sink
pa_assert(u->core == c); pa_assert(u->core == c);
pa_assert(u->on_hotplug); pa_assert(u->on_hotplug);
notify_subscribers(u);
return route_sink_inputs(u, NULL); return route_sink_inputs(u, NULL);
} }
@ -866,6 +872,8 @@ static pa_hook_result_t source_put_hook_callback(pa_core *c, PA_GCC_UNUSED pa_so
pa_assert(u->core == c); pa_assert(u->core == c);
pa_assert(u->on_hotplug); pa_assert(u->on_hotplug);
notify_subscribers(u);
return route_source_outputs(u, NULL); return route_source_outputs(u, NULL);
} }
@ -880,6 +888,8 @@ static pa_hook_result_t sink_unlink_hook_callback(pa_core *c, pa_sink *sink, str
if (c->state == PA_CORE_SHUTDOWN) if (c->state == PA_CORE_SHUTDOWN)
return PA_HOOK_OK; return PA_HOOK_OK;
notify_subscribers(u);
return route_sink_inputs(u, sink); return route_sink_inputs(u, sink);
} }
@ -894,6 +904,8 @@ static pa_hook_result_t source_unlink_hook_callback(pa_core *c, pa_source *sourc
if (c->state == PA_CORE_SHUTDOWN) if (c->state == PA_CORE_SHUTDOWN)
return PA_HOOK_OK; return PA_HOOK_OK;
notify_subscribers(u);
return route_source_outputs(u, source); return route_source_outputs(u, source);
} }