2 minor issues noticed with padsp

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, 2011-04-11 at 20:16 +0530, Arun Raghavan wrote:
> On Thu, 2011-04-07 at 17:25 -0400, Drew Ogle wrote:
> > Hi,
> > 
> > I have a really old application that uses oss interfaces, and noticed
> > that when attempting to run via padsp it would fail ( as opposed to
> > with aoss ).
> > After talking with Ford_Prefect in #pulseaudio, we narrowed down the
> > problem to protocol-native.c
> > Attached is a patch which allows padsp's get_sink_info call to success
> > on the server side ( drop-invalid-check ).
> > He mentioned that name is allowed to be null; which makes me think it
> > might be less confusing to just do a *name check and then no others.
> 
> Ack from me on the first patch.
> 
> > Additionally, I noticed that in padsp.c it was recieving two callbacks
> > in both sink and source_info_cb. As this appears to be the desired
> > behavior in introspect.c, I modified the _cb functions to only notify
> > if success was not already set.
> 
> This second one can be tweaked so that we just drop the !si in both
> those conditions - don't need to chagne anything else.

Whoops, forgot about these. I've fixed them up and pushed to master.

Thanks!
Arun



[Index of Archives]     [Linux Audio Users]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux