On Mon, 2012-08-20 at 09:37 +0200, David Henningsson wrote:
> On 08/19/2012 01:55 PM, Tanu Kaskinen wrote:
> > The sink has different frame size than the sink input, so
> > the max_rewind value of the sink input needs to be converted
> > when setting the sink max_rewind value.
> >
> > The conversion is already done correctly in
> > sink_input_update_max_rewind_cb().
> > ---
> > src/modules/module-virtual-surround-sink.c | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/src/modules/module-virtual-surround-sink.c b/src/modules/module-virtual-surround-sink.c
> > index fc8f898..df36876 100644
> > --- a/src/modules/module-virtual-surround-sink.c
> > +++ b/src/modules/module-virtual-surround-sink.c
> > @@ -386,7 +386,7 @@ static void sink_input_attach_cb(pa_sink_input *i) {
> >
> > /* FIXME: Too small max_rewind:
> > * https://bugs.freedesktop.org/show_bug.cgi?id=53709 */
> > - pa_sink_set_max_rewind_within_thread(u->sink, pa_sink_input_get_max_rewind(i));
> > + pa_sink_set_max_rewind_within_thread(u->sink, pa_sink_input_get_max_rewind(i) * u->sink_fs / u->fs);
>
> The same seems to apply to max_request as well?
Indeed, thanks. I sent an updated patch.
--
Tanu
