[pulseaudio-tickets] [Bug 55280] tunnel modules cannot figure out adress/auth automatically

[bugzilla-daemon at freedesktop.org]

https://bugs.freedesktop.org/show_bug.cgi?id=55280

--- Comment #9 from Pierre Ossman <pierre-bugzilla at ossman.eu> ---
(In reply to comment #7)
> The tunnel module should get rewritten so that it uses libpulse, and this
> bug should get resolved as a side effect. There is no 100% guarantee that
> the rewrite project will finish, so if you want this patch applied anyway,
> I'm not against that. Some review comments follow.

I'd appreciate that. The rewrite is very interesting to us, but it has a bit
more to go before it can replace module-tunnel. So we'll probably be doing
relevant fixes and improvements to module-tunnel until then.

> 
> In pa__init(), the server variable is never freed, even if it should be:
> 

Will have a look at that. Thanks.

> It doesn't seem like a good idea to connect to a local server by default. If
> the user loads module-tunnel-sink without any arguments, I think it should
> fail, because tunneling to a local daemon doesn't usually make sense. I
> think it would be better if connecting to a local server would require an
> explicit indication that the user really wants to do that.

I can sympathise with that. Would something like an "auto" argument suffice?

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/pulseaudio-bugs/attachments/20130714/347d1640/attachment.html>