[pulseaudio-discuss] [PATCH v2 2/3] bluetooth: separate HSP and HFP

Tanu Kaskinen tanuk at iki.fi
Sun Aug 21 18:03:13 UTC 2016


On Sun, 2016-08-21 at 15:19 +0300, Tanu Kaskinen wrote:
> On Sat, 2016-08-20 at 15:03 -0700, James Bottomley wrote:
> >  static const char* const valid_modargs[] = {
> >      "path",
> > +    "disable_profile_hfp",
> 
> We try to avoid negative option names on the basis that double
> negatives are not nice ("disable_foo=no"), so I'd prefer
> "enable_profile_hfp".

Hmm, maybe that's not a good name, if the option only affects HFP HF
support, and not HFP AG. Would "enable_profile_hfp_hf" work? It's
pretty awful from user-friendliness perspective, but I have trouble
coming up with better names.

-- 
Tanu


More information about the pulseaudio-discuss mailing list