[pulseaudio-discuss] [PATCH 3/8] sink, source: Don't care about default and alternate rate in passthrough mode

[Tanu Kaskinen]

On Mon, 2013-08-12 at 09:18 +0200, David Henningsson wrote:
> On 08/09/2013 08:57 AM, Tanu Kaskinen wrote:
> > In passthrough mode the device rate is set to match the stream rate,
> > and the default and alternate rates are ignored.
> > ---
> >  src/pulsecore/sink.c   | 2 +-
> >  src/pulsecore/source.c | 2 +-
> >  2 files changed, 2 insertions(+), 2 deletions(-)
> > 
> > diff --git a/src/pulsecore/sink.c b/src/pulsecore/sink.c
> > index 2f5174e..a4ad21a 100644
> > --- a/src/pulsecore/sink.c
> > +++ b/src/pulsecore/sink.c
> > @@ -1389,7 +1389,7 @@ bool pa_sink_update_rate(pa_sink *s, uint32_t rate, bool passthrough) {
> >      if (!s->update_rate)
> >          return false;
> >  
> > -    if (PA_UNLIKELY(default_rate == alternate_rate)) {
> > +    if (PA_UNLIKELY(default_rate == alternate_rate && !passthrough)) {
> >          pa_log_warn("Default and alternate sample rates are the same.");
> 
> I think this message could go away altogether. There are plenty of
> scenarios where you prefer to always use one sample rate.
> 
> But if you like it the way it is, feel free to push.

Removing the message is fine by me.

-- 
Tanu