On Wed, 2016-02-17 at 18:47 +0530, Arun Raghavan wrote:
> On Mon, 2016-02-15 at 13:12 +0200, Tanu Kaskinen wrote:
> > On Mon, 2016-01-18 at 13:06 +0530, arun at accosted.net wrote:
> > > +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â pa_log("The beamformer currently only supports targeting along the azimuth");
> > > +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â goto fail;
> > > +Â Â Â Â Â Â Â Â Â Â Â Â }
> > > +
> > > +Â Â Â Â Â Â Â Â Â Â Â Â direction.s[0] = f[0];
> > > +Â Â Â Â Â Â Â Â Â Â Â Â direction.s[1] = f[1];
> > > +Â Â Â Â Â Â Â Â Â Â Â Â direction.s[2] = f[2];
> > > +Â Â Â Â Â Â Â Â }
> > > +
> > > +Â Â Â Â Â Â Â Â /* The beamformer gives us a single channel */
> > > +Â Â Â Â Â Â Â Â out_ss->channels = 1;
> > > +Â Â Â Â Â Â Â Â pa_channel_map_init_mono(out_map);
> >
> > Could this be moved to fixate_spec()? I don't like the audio format
> > parameters being calculated all over the place.
>
> I wanted to do that, but we can't because we need the capture channel
> count for parsing the mic geometry (there is a comment explaining that
> where if (beamforming) { ... } is).
I meant moving only the out_ss and out_map assignments. Those don't
depend on the capture channel count. fixate_spec() will then need to
take the beamforming bool as a parameter, but I think that's fine.
--Â
Tanu
