Bug 54244

Summary: [cleanup] combine-sink: Inconsistent way of rewinding when creating a new sink input
Product: PulseAudio Reporter: Tanu Kaskinen <tanuk>
Component: modulesAssignee: pulseaudio-bugs
Status: RESOLVED MOVED QA Contact: pulseaudio-bugs
Severity: normal    
Priority: medium CC: lennart
Version: unspecified   
Hardware: Other   
OS: All   
Whiteboard:
i915 platform: i915 features:

Description Tanu Kaskinen 2012-08-30 05:39:20 UTC
Fixing bug #54243 would make this bug irrelevant, but if that is not done for some reason, then here's the inconsistency:

In most cases, when a new sink input is created, and the sink input needs to be made audible immediately, the sink implementor requests a rewind in the sink input state change callback when the state changes from PA_SINK_INPUT_INIT to PA_SINK_INPUT_RUNNING. module-combine-sink could do the same, but instead it requests the rewind in the attach callback. That would actually be a potentially user-visible bug if the sink inputs that module-combine-sink could be moved, but since the inputs can't be moved, requesting the rewind in the attach callback is equivalent to doing it in the state change callback. There's no reason to do it differently in module-combine-sink than in other modules, so for consistency module-combine-sink should be changed to request the rewind in the state change callback like everyone else.
Comment 1 GitLab Migration User 2018-07-30 10:12:26 UTC
-- GitLab Migration Automatic Message --

This bug has been migrated to freedesktop.org's GitLab instance and has been closed from further activity.

You can subscribe and participate further through the new bug through this link to our GitLab instance: https://gitlab.freedesktop.org/pulseaudio/pulseaudio/issues/266.

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.