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 _______________________________________________ pulseaudio-discuss mailing list pulseaudio-discuss@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/pulseaudio-discuss