On Fri, 2014-12-05 at 15:00 +0100, David Henningsson wrote: > > On 2014-12-04 19:44, Tanu Kaskinen wrote: > > When using PulseAudio in system mode, it may be interesting to know > > which seats the devices are assigned to. This will be used in the > > tunnel manager to filter out the devices that belong to other users. > > I'd prefer if PA_PROP_UDEV_SEAT was called PA_PROP_DEVICE_SEAT, because > the seat a device belongs to could be generically useful to e g volume > control clients. If we make the seat id field non-udev specific, I'm happy with that. However, in that case I'd prefer to use a separate field in the appropriate structs instead of using proplists. As I've explained before, I don't like using proplists for stable core APIs. > I'd also prefer if it was not filled in in case there was no such udev > property, to rule out any possibility it gets filled in with the wrong > value. If a udev device has the "seat" tag, but the "ID_SEAT" property isn't set, then the implied seat id is "seat0". This is documented behaviour of udev. -- Tanu
