Pushed, thanks. (This one makes sense independently of other patches.) Also noted that it looks lik Patchwork actually picked it up: remote: I: 1 patch(es) updated to state Accepted. ...but the patch still lists as "New" in http://patchwork.freedesktop.org/patch/62756/ // David On 2015-10-23 12:56, Tanu Kaskinen wrote: > pa_device_port.profiles is initialized in pa_device_port_new(), so no > need to do that in alsa-ucm.c. > --- > src/modules/alsa/alsa-ucm.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/src/modules/alsa/alsa-ucm.c b/src/modules/alsa/alsa-ucm.c > index ff5693d..42f3242 100644 > --- a/src/modules/alsa/alsa-ucm.c > +++ b/src/modules/alsa/alsa-ucm.c > @@ -778,7 +778,6 @@ static void ucm_add_port_combination( > > pa_hashmap_put(ports, port->name, port); > pa_log_debug("Add port %s: %s", port->name, port->description); > - port->profiles = pa_hashmap_new(pa_idxset_string_hash_func, pa_idxset_string_compare_func); > } > > port->priority = priority; > -- David Henningsson, Canonical Ltd. https://launchpad.net/~diwic