diff options
| author |   Matt Porter <mporter@konsulko.com> | 2017-04-19 12:53:55 -0400 |
|---|---|---|
| committer | Matt Porter <mporter@konsulko.com> | 2017-04-20 22:00:09 -0400 |
| commit | b8423505e6e7360b509cdfd22f8d86f45169443d (patch) | |
| tree | bfd146499506418a2200e3a78aa0b81af62b907e /app/paclient.cpp | |
| parent | 3fd71f4b6bc026f2f9054140f7bf612855e45d7d (diff) | |
Convert the sink/source volume caches from a list to hashes
Maintain the cached sink/source volume state in separate hashes
for quick lookup. This will further simplify external volume event
support that will need to update the cache.
AGL-Bug: SPEC-548
Change-Id: I47b8e070318f3992a5343d1753c50baa8c1d9cb3
Signed-off-by: Matt Porter <mporter@konsulko.com>
Diffstat (limited to 'app/paclient.cpp')
| -rw-r--r-- | app/paclient.cpp | 32 |
1 files changed, 15 insertions, 17 deletions
diff --git a/app/paclient.cpp b/app/paclient.cpp index 78567ce..9360685 100644 --- a/app/paclient.cpp +++ b/app/paclient.cpp @@ -55,16 +55,12 @@ void PaClient::setVolume(uint32_t type, uint32_t index, uint32_t channel, uint32 { pa_operation *o; pa_context *c = context(); - CState *cstate = NULL; - - foreach (cstate, m_cstatelist) - if ((cstate->index == index) && (cstate->type == type)) - break; - - cstate->cvolume.values[channel] = volume; + pa_cvolume *cvolume = NULL; if (type == C_SINK) { - if (!(o = pa_context_set_sink_volume_by_index(c, index, &cstate->cvolume, set_sink_volume_cb, NULL))) { + cvolume = m_sink_states.value(index); + cvolume->values[channel] = volume; + if (!(o = pa_context_set_sink_volume_by_index(c, index, cvolume, set_sink_volume_cb, NULL))) { qWarning() << "PaClient: set sink #" << index << " channel #" << channel << " volume: " << pa_strerror(pa_context_errno(c)); @@ -72,7 +68,9 @@ void PaClient::setVolume(uint32_t type, uint32_t index, uint32_t channel, uint32 } pa_operation_unref(o); } else if (type == C_SOURCE) { - if (!(o = pa_context_set_source_volume_by_index(c, index, &cstate->cvolume, set_source_volume_cb, NULL))) { + cvolume = m_sink_states.value(index); + cvolume->values[channel] = volume; + if (!(o = pa_context_set_source_volume_by_index(c, index, cvolume, set_source_volume_cb, NULL))) { qWarning() << "PaClient: set source #" << index << " channel #" << channel << " volume: " << pa_strerror(pa_context_errno(c)); @@ -215,14 +213,14 @@ void PaClient::init() void PaClient::addOneControlState(int type, int index, const pa_cvolume *cvolume) { int i; - CState *cstate; + pa_cvolume *cvolume_new; - cstate = new CState; - cstate->type = type; - cstate->index = index; - cstate->cvolume.channels = cvolume->channels; + cvolume_new = new pa_cvolume; + cvolume_new->channels = cvolume->channels; for (i = 0; i < cvolume->channels; i++) - cstate->cvolume.values[i] = cvolume->values[i]; - - m_cstatelist.append(cstate); + cvolume_new->values[i] = cvolume->values[i]; + if (type == C_SINK) + m_sink_states.insert(index, cvolume_new); + else if (type == C_SOURCE) + m_source_states.insert(index, cvolume_new); } |