On Tue, 2013-01-29 at 17:41 +0100, poljar (Damir Jeli?) wrote: > This adds two new commands to pactl: > set-sink-set-default > set-source-set-default In my opinion, "default sink" should be a property of the core, so the command should be "set-default-sink". Incidentally, there was recently a long debate at #bluez regarding whether "default adapter" should be a property of an adapter object or the adapter manager object. During the debate, it was argued that having an "is default" property in the adapter object is usually easier for clients than "default adapter" in the adapter manager, but on the other hand, when the default adapter changes, it won't appear as an atomic event (first the property changes on one adapter and then on another adapter). No consensus was found, and the resolution was not to have a "default adapter" property at all :D The same arguments apply to pulseaudio's default sink and source. I'm in the "default sink as a core property" camp (and that's how it's already implemented internally). Since I'm against an "is default" property in sinks and sources, I'm also against the protocol extension that you added in patch 2. The protocol already supports querying the default sink, so pactl can display the "Default: yes/no" information without any protocol extensions. -- Tanu
