[PATCH 1/4] winealsa: Move get_prop_value to the unixlib.

Huw Davies huw at codeweavers.com
Tue Mar 15 03:56:16 CDT 2022


On Tue, Mar 15, 2022 at 08:38:32AM +0000, Emil Velikov wrote:
> Have you considered splitting the props handling - get_device_path,
> get_physical_speakers?
> AFAICT this should reduce the duplication on the unix side (alsa vs
> pulse vs others). More importantly it makes it easier to avoid the
> caller allocated string, by passing the vendor/device/other data
> directly.

We could split these, but I didn't want the number of syscalls
to escalate.  If there really are only two properties then
that doesn't sound too bad.

> What are your thoughts on having a single mmdev.drv, which calls into
> winealsa.drv/winepulse.drv/other? Are wineandroid/mmdev, winecoreaudio
> and wineoss in scope in the mid/long run?

Yes, long term the aim would be to have a single PE-side.  I've been
keeping the unixlib interfaces as close as possible to facilitate this.
Once all of the drivers are done, we'll had a good idea of what that'll
look like.

winecoreaudio is already done and I've started on wineoss, though
I still have the midi bits of winealsa to upstream.

Until you'd mentioned it, I'd forgotten about sound side of
wineandroid, so I guess that's one more to do :-(

Huw.



More information about the wine-devel mailing list