[pulseaudio-discuss] [PATCH] sink, source: Add a mode to avoid resampling if possible

[Arun Raghavan]

On Sat, 28 Jan 2017, at 04:01 PM, Peter Meerwald-Stadler wrote:
> > This adds an "avoid-resampling" option to daemon.conf that makes the
> > daemon try to use the stream sample rate if possible (the device needs
> > to support it, which currently only ALSA does), and there should not be
> > any other stream connected).
> > 
> > This should enable some of the "audiophile" use-cases where users wish
> > to play high sample rate audio files without resampling.
> 
> comments below, picking on wording
>  
> > We still will do conversion, though which means that the 96/24 case will
> > require that the default format be set to be 24-bit as well, this will
> > force all streams to be upconverted, which other than the wasted
> > resources should be relatively harmless.
> > ---
> >  man/pulse-daemon.conf.5.xml.in |  9 +++++++++
> >  src/daemon/daemon-conf.c       |  3 +++
> >  src/daemon/daemon-conf.h       |  1 +
> >  src/daemon/daemon.conf.in      |  1 +
> >  src/daemon/main.c              |  1 +
> >  src/pulsecore/core.h           |  1 +
> >  src/pulsecore/sink.c           | 10 ++++++++--
> >  src/pulsecore/source.c         | 10 ++++++++--
> >  8 files changed, 32 insertions(+), 4 deletions(-)
> > 
> > NOTE: I didn't want to editorialise the commit message itself, but if you're
> > excited about this feature, please do make sure you understand Monty's
> > excellent post on the subject of high sample rate audio at:
> > 
> >   http://people.xiph.org/~xiphmont/demo/neil-young.html
> > 
> > diff --git a/man/pulse-daemon.conf.5.xml.in b/man/pulse-daemon.conf.5.xml.in
> > index b81a549..cb09b97 100644
> > --- a/man/pulse-daemon.conf.5.xml.in
> > +++ b/man/pulse-daemon.conf.5.xml.in
> > @@ -124,6 +124,15 @@ License along with PulseAudio; if not, see <http://www.gnu.org/licenses/>.
> >      </option>
> >  
> >      <option>
> > +      <p><opt>avoid-resampling=</opt> If set, try to configure the
> > +      device to avoid resampling. This only works if on devices that
> 
> only works on devices

Fixed.

> > +      support reconfiguring their rate, and when no other streams are
> > +      already playing or capturing audio. The device will also not be
> > +      configured to a rate less than the default and alternate sample
> > +      rates.</p>
> 
> rate

Should be rates, since it's the plural form?

-- Arun