Bugzilla – Bug 53923
pa_sink_input_request_rewind() has a nonsensical check
Last modified: 2012-09-17 14:55:52 UTC
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.