[pulseaudio-discuss] [PATCH 2/2] pactl: Show the default sink/source in the list command.

Damir Jelić poljarinho at gmail.com
Tue Jan 29 17:12:13 PST 2013


On Wed, Jan 30, 2013 at 02:05:29AM +0100, David Henningsson wrote:
> On 01/29/2013 05:41 PM, poljar (Damir Jelić) wrote:
> >This includes exposing the information in the protocol for the
> >get sink/source info commands and adding 2 new reply fields for
> >these commands.
> >
> >Update protocol to 28.
> >---
> >  PROTOCOL                        | 10 ++++++++++
> >  configure.ac                    |  2 +-
> >  src/modules/module-tunnel.c     | 10 ++++++++++
> >  src/pulse/introspect.c          | 14 ++++++++++++++
> >  src/pulse/introspect.h          |  2 ++
> >  src/pulsecore/protocol-native.c | 18 ++++++++++++++++++
> >  src/utils/pactl.c               |  4 ++++
> >  7 files changed, 59 insertions(+), 1 deletion(-)
> >
> >diff --git a/PROTOCOL b/PROTOCOL
> >index 01abeed..d0ea360 100644
> >--- a/PROTOCOL
> >+++ b/PROTOCOL
> >@@ -333,6 +333,16 @@ PA_COMMAND_GET_CARD_INFO(_LIST)):
> >
> >  The field is added once for every port.
> >
> >+## v28, implemented by >= 3.0
> 
> You probably mean >= 4.0 ?
> 
Oh, yeah you're right about that.

> Otherwise looks good (I did a quick read review only, no testing).
> 
> (Slightly cleaner to have the pactl changes in a separate patch but
> I'm not picky about that.)
> 

I wasn't even sure if we want to print that info since it's available
with 'pactl info' and I'm not sure if I should hide that info if the
protocol doesn't support this (if it's not supported it prints always
'no').

Thanks for the review.


More information about the pulseaudio-discuss mailing list