On Fri, 2013-07-12 at 15:07 -0300, [email protected] wrote: > /* Run from main thread */ > +static int add_source(struct userdata *u) { > + pa_source_new_data data; > + > + pa_assert(u->transport); > + > + pa_source_new_data_init(&data); > + data.module = u->module; > + data.card = u->card; > + data.driver = __FILE__; > + data.name = pa_sprintf_malloc("bluez_source.%s", u->device->address); > + data.namereg_fail = false; > + pa_proplist_sets(data.proplist, "bluetooth.protocol", > pa_bluetooth_profile_to_string(u->profile)); > + pa_source_new_data_set_sample_spec(&data, &u->sample_spec); > + > + connect_ports(u, &data, PA_DIRECTION_INPUT); > + > + if (!u->transport_acquired) > + switch (u->profile) { > + case PROFILE_A2DP_SOURCE: > + data.suspend_cause = PA_SUSPEND_USER; > + break; > + case PROFILE_A2DP_SINK: > + case PROFILE_OFF: > + pa_assert_not_reached(); /* Profile switch should have > failed */
This comment is correct for neither A2DP_SINK nor OFF. The same comment is also in add_sink(). There the comment is correct only for A2DP_SINK. I propose you remove the comment from add_source() and move it under PROFILE_A2DP_SINK in add_sink(). -- Tanu _______________________________________________ pulseaudio-discuss mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/pulseaudio-discuss
