Re: [PATCH 0/7] MR799: wineoss: Switch to mmdevapi's unixlib.h.

[Huw Davies (＠huw)]

Huw Davies (@huw) commented about dlls/wineoss.drv/oss.c:
>      return STATUS_SUCCESS;
>  }
>
>  close(mixer_fd);
>
> - if(sysinfo.version[0] < '4' || sysinfo.version[0] > '9'){
> -
>    TRACE("Priority_Low: sysinfo.version[0]: %x\n", sysinfo.version[0]);
> -
>    params->priority = Priority_Low;
> -
>    return STATUS_SUCCESS;
> - }
> - if(sysinfo.versionnum & 0x80000000){
> -
>    TRACE("Priority_Low: sysinfo.versionnum: %x\n", sysinfo.versionnum);
> -
>    params->priority = Priority_Low;
> -
>    return STATUS_SUCCESS;
> - }
> -

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`.