7 Sep
2022
7 Sep
'22
9:47 p.m.
On Wed Sep 7 09:11:20 2022 +0000, Huw Davies wrote:
I'd prefer that we didn't make these sort of changes (and if we do, it should be in a separate commit). Having the unixlib return the priority seems reasonable to me, and it wouldn't be hard to change the pulse driver to do likewise. It might make sense to rename the call from `test_connect` to `get_priority`.
We have make the change in `winealsa` too, but it's not an issue.
Returning the priority actually makes more sense in my opinion, as it's the driver itself that is supposed to know whether it can work reliably.
I also agree with the name change, but I would do it in a separate MR, perhaps when merging the various `mmdevdrv.c` files.