[RFC] Allow read-only or non-existing sink input volume.

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

 



'Twas brillig, and Tanu Kaskinen at 22/02/11 16:35 did gyre and gimble:
> It would be nice if someone would test this with passthrough streams
> also, but personally I think it would be ok to commit this patch to
> master now.

OK, I've applied this to master in my tree now after giving it a bit of
a review (and removing the last bit of the commit message!)

Only question I had really was related to this inline if which appears a
couple times:

 has_volume ? !pa_sink_input_is_volume_writable(s) : FALSE


Would it make sense to just return FALSE from
pa_sink_input_is_volume_writable in the case when there is no volume?

Doesn't really matter much, but would avoid a couple of the inline ifs
which is arguably "tidier".

(of course pa_sink_input_is_volume_readable() and _writeable() are
currently the same implementation so simply removing the inline if would
be sufficient at present, but I'm sure that wont last long....)


Col

-- 

Colin Guthrie
gmane(at)colin.guthr.ie
http://colin.guthr.ie/

Day Job:
  Tribalogic Limited [http://www.tribalogic.net/]
Open Source:
  Mageia Contributor [http://www.mageia.org/]
  PulseAudio Hacker [http://www.pulseaudio.org/]
  Trac Hacker [http://trac.edgewall.org/]




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

  Powered by Linux