Re: wined3d: CheckDeviceMultiSampleType

[H. Verbeet]

On 05/02/06, Vitaly Budovski  wrote:
> Can I have some feedback on the attached patch? I'm not entirely sure
> it's correct.
>
> Thanks,
>
> Vitaly
>
> diff --git a/dlls/wined3d/directx.c b/dlls/wined3d/directx.c
> index 7aa8d5d..8394362 100644
> --- a/dlls/wined3d/directx.c
> +++ b/dlls/wined3d/directx.c
> @@ -1245,6 +1245,13 @@ HRESULT WINAPI IWineD3DImpl_CheckDeviceM
>                                                         BOOL Windowed, D3DMULTISAMPLE_TYPE MultiSampleType, DWORD*   pQualityLevels) {
>  IWineD3DImpl *This = (IWineD3DImpl *)iface;
> - HDC deviceContext;
> - Display * pDisplay;
> - GLXFBConfig * pConfig;
> - int attributeList[5] = {GLX_SAMPLE_BUFFERS_ARB, 1, GLX_SAMPLES_ARB, MultiSampleType, 0};
> - int configCount;
> - int attributeValue;
> - TRACE_(d3d_caps)("(%p)-> (STUB) (Adptr:%d, DevType:(%x,%s), SurfFmt:(%x,%s), Win?%d, MultiSamp:%x, pQual:%p)\n",
>          This,
>          Adapter,
>
> @@ -1267,7 +1274,32 @@ HRESULT WINAPI IWineD3DImpl_CheckDeviceM
>          *pQualityLevels = 1; /* Guess at a value! */
>      }
> - if (D3DMULTISAMPLE_NONE == MultiSampleType) return D3D_OK;
> - if(D3DMULTISAMPLE_NONE == MultiSampleType) return D3D_OK;
> -
> - /* Get the display */
> - deviceContext = GetDC(0);
> - pDisplay = get_display(deviceContext);
> -
> - ENTER_GL();
> -
> - pConfig = glXChooseFBConfig(pDisplay, 0, attributeList, &configCount);
> - if(pConfig) {
> -
>    if(!glXGetFBConfigAttrib(pDisplay, *pConfig, GLX_SAMPLES, &attributeValue)) {
> -
>        XFree(pConfig);
> -
> -
>        LEAVE_GL();
> -
>        ReleaseDC(0, deviceContext);
> -
> -
>        return D3D_OK;
> -
>    }
> - } else {
> -
>    FIXME("Error retrieving FBConfig\n");
> - }
> - XFree(pConfig);
> -
> - LEAVE_GL();
> - ReleaseDC(0, deviceContext);
> - return D3DERR_NOTAVAILABLE;
>
> }

What is the patch supposed to do? Did you verify that it does that?