wineoss.drv: Add mmdevapi driver.
Andrew Eikum
aeikum at codeweavers.com
Mon May 9 08:10:09 CDT 2011
On 05/07/2011 07:04 PM, Gerald Pfeifer wrote:
> On Fri, 6 May 2011, Andrew Eikum wrote:
>> If Wine is trying to build wineoss.drv, then you must have oss_sysinfo.
>> If you have oss_sysinfo, then you have OSSv4. If you have OSSv4, then
>> you should have all of the above symbols, but apparently you don't. So I
>> suspect it's something amiss with your OSSv4 implementation, and that's
>> where I would begin investigating.
>>
>> Does that help you diagnose the problem?
>
> Yes, thanks a lot! Based on yours and Farncois' input, I did some
> research and found the following, among others
>
> http://manuals.opensound.com/developer/AFMT_S24_PACKED.html
>
> and consulting Google Code Search I find that also other applications
> do something like the following here (mplayer, for example).
>
> Suggested patch below.
Makes sense to me. Thanks. Do this patch and the float format patch
together fix your build issues, Gerald?
Andrew
>
> Gerald
>
>
> ChangeLog:
> Use AFMT_S24_LE instead of AFMT_S24_PACKED.
>
> diff --git a/dlls/wineoss.drv/mmdevdrv.c b/dlls/wineoss.drv/mmdevdrv.c
> index ff21cd6..789cb1e 100644
> --- a/dlls/wineoss.drv/mmdevdrv.c
> +++ b/dlls/wineoss.drv/mmdevdrv.c
> @@ -500,7 +500,7 @@ static int get_oss_format(const WAVEFORMATEX *fmt)
> case 16:
> return AFMT_S16_LE;
> case 24:
> - return AFMT_S24_PACKED;
> + return AFMT_S24_LE;
> case 32:
> return AFMT_S32_LE;
> }
> @@ -953,7 +953,7 @@ static HRESULT WINAPI AudioClient_GetMixFormat(IAudioClient *iface,
> }else if(formats& AFMT_S32_LE){
> fmt->Format.wBitsPerSample = 32;
> fmt->SubFormat = KSDATAFORMAT_SUBTYPE_PCM;
> - }else if(formats& AFMT_S24_PACKED){
> + }else if(formats& AFMT_S24_LE){
> fmt->Format.wBitsPerSample = 24;
> fmt->SubFormat = KSDATAFORMAT_SUBTYPE_PCM;
> }else{
More information about the wine-devel
mailing list