pa_sink_input_request_rewind() contains this:
(flush && i->thread_info.rewrite_nbytes != 0);
The "i->thread_info.rewrite_nbytes != 0" doesn't do anything useful, so all it does is create confusion (it's otherwise harmless).
It would be good to test the change at least a bit before pushing. I'm not willing to do that right now, which is why I'm filing this bug.
Created attachment 66976 [details] [review]
Removing unecessary check
The patch just remove the i->thread_info.rewrite_nbytes != 0 check.
I've done the follow tests:
1) Moving the sink-input around two different sinks a bluetooth device and my sound card.
2) Cork/Uncork the sink-input in the bluetooth sink
3) Several volume changes
In all these three tests worked as expected.
The bluetooth device used was: Motorola EQ5 portable speaker.
Hey folks, is it necessary other tests to validate it ?
Thanks in advance,
(In reply to comment #2)
> Hey folks, is it necessary other tests to validate it ?
No, the tests that you've done are fine. The patch is pushed now, sorry for the delay. I changed the commit message a bit to align with usual conventions. Thanks for the patch!
No problem at all, thanks for applying the patch.
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct.