Hi Tanu,<br><br>Thanks for this review !<br><br><div class="gmail_quote">On Tue, May 29, 2012 at 2:24 PM, Tanu Kaskinen <span dir="ltr">&lt;<a href="mailto:tanuk@iki.fi" target="_blank">tanuk@iki.fi</a>&gt;</span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div>
&gt; +        sink = pa_namereg_get_default_sink(m-&gt;core);<br>
<br>
</div></div>This still overrides the routing modules, right?<br>
<br>
I guess this is done, because otherwise you don&#39;t know how to initialize<br>
ss and map? I think a better solution would be to specify the<br>
PA_SINK_INPUT_FIX_FORMAT flag and its friends when calling<br>
pa_sink_input_new() when neither sink or source has been given. After<br>
the pa_sink_input_new() call, ss and map can be copied from the sink<br>
input, which has got them from the sink to which it was routed.<br></blockquote><div><br>PA_SINK_INPUT_FIX_FORMAT doesn&#39;t seem to work as expected.<br>If I set this flag without defining the sample_spec, I get an assertion<br>
in pa_sink_input_new when pa_format_info_from_sample_spec is called.<br><br>At first I considered that loading module loopback with specifying<br>anything is not really a frequent use case. That make a lot of changes<br>to add this feature I think.<br>
<br><br>
</div><blockquote class="gmail_quote" style="margin:0pt 0pt 0pt 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Now if sink or source is not set, the device description and icon name<br>
properties are left at a suboptimal state. Instead of checking for the<br>
sink and source pointers, maybe this proplist handling could be moved<br>
after pa_sink_input_new() and pa_source_output_new() have been called?<br>
At that time the sink and source would be available.<span><font color="#888888"><br></font></span></blockquote><div><br>That makes sense. I&#39;m just wondering if any application listening to sink creation<br>will get updated from the sink creation, or if this would create an additional round<br>
of IPC for the property change ? Could this be a potential problem ?<br><br>Frédéric<br><br></div></div>