'Twas brillig, and Colin Guthrie at 11/10/10 17:27 did gyre and gimble:
> 'Twas brillig, and Daniel Chen at 11/10/10 16:47 did gyre and gimble:
>> On Mon, Oct 11, 2010 at 9:58 AM, Colin Guthrie <gmane at colin.guthr.ie> wrote:
>>> diff --git a/src/pulsecore/sink.c b/src/pulsecore/sink.c
>>> index ff4cc17..3224467 100644
>>> --- a/src/pulsecore/sink.c
>>> +++ b/src/pulsecore/sink.c
>>> @@ -2708,7 +2708,8 @@ pa_bool_t
>>> pa_device_init_intended_roles(pa_proplist *p) {
>>> return TRUE;
>>>
>>> if ((s = pa_proplist_gets(p, PA_PROP_DEVICE_FORM_FACTOR)))
>>> - if (pa_streq(s, "handset") || pa_streq(s, "hands-free")) {
>>> + if (pa_streq(s, "handset") || pa_streq(s, "hands-free")
>>> + || pa_streq(s, "headset")) {
>>> pa_proplist_sets(p, PA_PROP_DEVICE_INTENDED_ROLES, "phone");
>>> return TRUE;
>>> }
>>
>> Looks fine in principle. Have you seen any mislabeled headsets?
>
> Not personally, but then I don't really have much in the way of h/w,
> which is the main reason I'm asking for feedback :D
No one has shouted against this so, I've added it. If it turns out this
is evil there is always git revert :)
Col
--
Colin Guthrie
gmane(at)colin.guthr.ie
http://colin.guthr.ie/
Day Job:
Tribalogic Limited [http://www.tribalogic.net/]
Open Source:
Mageia Contributor [http://www.mageia.org/]
PulseAudio Hacker [http://www.pulseaudio.org/]
Trac Hacker [http://trac.edgewall.org/]
