[pulseaudio-discuss] [PATCH v3 0/8] Change the bool sink_save to char *preferred_sink

[Hui Wang]

OK, I will create a merge request.

Thanks,

Hui.

On 2019/8/23 下午3:48, Georg Chini wrote:
> On 18.08.19 07:32, Hui Wang wrote:
>> In the V3, I addressed all Tanu's and Georg's comments:
>> 0002-xxx.patch:
>> remove the pa_subscription_post() in the 
>> pa_sink_input_set_preferred_sink()
>>
>> 0003-xxx.patch:
>> Add the check of sink_input->sink->card in the subscribe_callback()
>>
>> 0004-xxx.patch:
>> Initialize the varible old_sink_is_unavailable
>> Add the check of core->default_sink == NULL || 
>> core->default_sink->unlink_requested
>> in the pa_sink_move_streams_to_default_sink()
>>
>> 0007-xxx.patch:
>> Move the calling of pa_sink_move_streams_to_default_sink() after the
>> pa_core_update_default_sink() in the pa_sink_unlink()
>> Add shutdown check in the pa_sink_move_streams_to_default_sink()
>> Change the logging message in the pa_sink_move_streams_to_default_sink()
>> Remove the sink and sink_input parts in the module-rescue-streams.c
>>
>> ============================================================
>>
>> This is based on Tanu's design:
>> https://lists.freedesktop.org/archives/pulseaudio-discuss/2018-October/030531.html 
>>
>>
>> And this is only for sink-input and sink, not for source-output and 
>> source.
>>
>> In the V2, I addressed all comments of V1 and split the big patch 
>> into small patches
>> as suggested by Tanu. It will be easy to review in the future.
>>
>>
> Thanks for the updated patches. I will be on vacation starting end of 
> next week,
> so I will not find the time for a review before end of September. Is 
> it possible
> that you open a merge request on Gitlab for this?
> https://gitlab.freedesktop.org/pulseaudio/pulseaudio/
>
> If not, I will open the merge request for you once the review is complete
> because we changed our policy so that all changes have to go through a
> merge request.
>
>