Bug 100684 - Filter modules should use "master_sink" and "master_source" as the modarg names
Summary: Filter modules should use "master_sink" and "master_source" as the modarg names
Status: RESOLVED WONTFIX
Alias: None
Product: PulseAudio
Classification: Unclassified
Component: modules (show other bugs)
Version: unspecified
Hardware: Other All
: medium normal
Assignee: pulseaudio-bugs
QA Contact: pulseaudio-bugs
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 99067
  Show dependency treegraph
 
Reported: 2017-04-14 16:23 UTC by Tanu Kaskinen
Modified: 2017-04-21 13:10 UTC (History)
1 user (show)

See Also:
i915 platform:
i915 features:


Attachments

Description Tanu Kaskinen 2017-04-14 16:23:49 UTC
During the 11.0 release cycle, module-ladspa-sink and module-virtual-surround-sink changed the "master" modarg to "sink_master", because module-filter-apply uses "sink_master". However, if we're unifying the modarg naming, I think "master_sink" (and "master_source") would be a better choice.

I'll mark this as a release blocker, because if this is not fixed before the release, we'll have to have compatibility code for two old versions of the modarg name in module-ladspa-sink and module-virtual-surround-sink instead of just one.
Comment 1 Tanu Kaskinen 2017-04-21 13:10:49 UTC
My intention was to change the modarg names only if there's a consensus that it's a good idea, but Peter was against it[1], so I'm not going to bother. I'll mark this bug as "wontfix", but if someone else feels strongly about this, the discussion can be continued.

[1] https://lists.freedesktop.org/archives/pulseaudio-discuss/2017-April/028013.html


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.