<p dir="ltr"><br>
2016. 3. 11. 오전 12:29에 "Tanu Kaskinen" <<a href="mailto:tanuk@iki.fi">tanuk@iki.fi</a>>님이 작성:<br>
><br>
> On Mon, 2016-02-22 at 11:39 +0100, Georg Chini wrote:<br>
> > On 22.02.2016 07:27, Arun Raghavan wrote:<br>
> > ><br>
> > > On Thu, 2016-02-18 at 17:09 +0100, Georg Chini wrote:<br>
> > > ><br>
> > > > On 18.02.2016 10:40, Arun Raghavan wrote:<br>
> > > > ><br>
> > > > > On Thu, 2016-02-11 at 23:11 +0900, Sangchul Lee wrote:<br>
> > > > > ><br>
> > > > > > Fix improper unducking behavior when using this module with<br>
> > > > > > "global=1" argument.<br>
> > > > > ><br>
> > > > > > BugLink: <a href="https://bugs.freedesktop.org/show_bug.cgi?id=93994">https://bugs.freedesktop.org/show_bug.cgi?id=93994</a><br>
> > > > > ><br>
> > > > > > Signed-off-by: Sangchul Lee <<a href="mailto:sc11.lee@samsung.com">sc11.lee@samsung.com</a>><br>
> > > > > > ---<br>
> > > > > Looks good to me, pushed thanks.<br>
> > > > ><br>
> > > > > -- Arun<br>
> > > > ><br>
> > > > Hi Arun,<br>
> > > ><br>
> > > > there has been a patch series I sent a while ago which combines module<br>
> > > > role-cork<br>
> > > > and module-role-ducking because they contain lots of duplicated code.<br>
> > > > This series has<br>
> > > > never been reviewed. Maybe you can take a look at it?<br>
> > > > You can find it at<br>
> > > > <a href="http://thread.gmane.org/gmane.comp.audio.pulseaudio.general/23189">http://thread.gmane.org/gmane.comp.audio.pulseaudio.general/23189</a><br>
> > > I think this does make sense. The stream selection logic is the same<br>
> > > between the two modules, and it's only what we do after selecting the<br>
> > > stream that changes.<br>
> > ><br>
> > > I'll try to get to this after reviewing the RAOP series.<br>
> > ><br>
> > > Thank you for your patience.<br>
> > ><br>
> > ><br>
> > If you review it, please include the additional patch at<br>
> > <a href="http://thread.gmane.org/gmane.comp.audio.pulseaudio.general/23189">http://thread.gmane.org/gmane.comp.audio.pulseaudio.general/23189</a><br>
> > which fixes incorrect behavior when a stream starts corked. The bug<br>
> > also exists in the current version of module-role-cork.<br>
><br>
> Did youo mean to give some other link? The link points to the patches<br>
> for merging role-cork and role-ducking, but it sounds like you may have<br>
> meant some other patch.<br></p>
<p dir="ltr">I think he might mean the additional patch regarding incorrect behavior of role-cork.<br>
Please find this at<br>
 <a href="http://thread.gmane.org/gmane.comp.audio.pulseaudio.general/25238">http://thread.gmane.org/gmane.comp.audio.pulseaudio.general/25238</a> (not reviewed yet)<br></p>
<p dir="ltr">Thanks,<br>
Sangchul Lee<br>
</p>