[pulseaudio-discuss] [PATCH] Removed the option to specify sample format. (bug 46529)

Arun Raghavan arun.raghavan at collabora.co.uk
Fri Jan 4 19:02:42 PST 2013


On Fri, 2013-01-04 at 17:27 -0600, Pierre-Louis Bossart wrote:
> On 01/04/2013 07:59 AM, Tanu Kaskinen wrote:
> > On Thu, 2013-01-03 at 09:35 -0600, Pierre-Louis Bossart wrote:
> >>> @@ -48,7 +48,6 @@ PA_MODULE_USAGE(
> >>>            _("sink_name=<name for the sink> "
> >>>              "sink_properties=<properties for the sink> "
> >>>              "master=<name of sink to filter> "
> >>> -          "format=<sample format> "
> >>>              "rate=<sample rate> "
> >>>              "channels=<number of channels> "
> >>>              "channel_map=<channel map> "
> >>> @@ -77,7 +76,6 @@ static const char* const valid_modargs[] = {
> >>>        "sink_name",
> >>>        "sink_properties",
> >>>        "master",
> >>> -    "format",
> >>>        "rate",
> >>>        "channels",
> >>>        "channel_map",
> >> What problem are you trying to fix here? It's not obvious why the sample
> >> format information should be removed, why not the rate and channels
> >> while we are at it? And even if this is needed, I guess there's dead
> >> code to parse the argument that should be removed in the _init routine?
> >
> > The patch title refers to bug 46529, I guess you missed that? Here's the
> > link: https://bugs.freedesktop.org/show_bug.cgi?id=46529
> >
> > I hope that makes things clearer. As for removing dead code from
> > pa__init(), that's not necessary, because the format argument was parsed
> > by pa_modargs_get_sample_spec_and_channel_map(). There's no dead code to
> > remove.
> 
> My point was that there are multiple modules that were derived from this 
> initial virtual sink contributed about 3 years ago, and if you want to 
> be consistent all of them should be cleaned (source, eq, surround, 
> echo-cancel, etc). Rather than removing the option once you may want to 
> either remove this option everywhere or fix the bug by introducing the 
> required conversion.

Or, since this is merely meant to be a module that other modules are
meant to be based on, put that as a big comment for derivative modules
to deal with, and leave the modarg as is.

Tangentially, we should also not actually installing
module-virtual-{sink,source}

-- Arun



More information about the pulseaudio-discuss mailing list