Huw Davies : opengl/tests: Skip tests if we can't find a pixel format.
Huw Davies
huw at codeweavers.com
Mon Dec 8 06:03:50 CST 2008
On Fri, Dec 05, 2008 at 11:04:07AM -0600, Alexandre Julliard wrote:
> Module: wine
> Branch: master
> Commit: e86ff2a3128f4a0157ecfa10fee31d1416312c71
> URL: http://source.winehq.org/git/wine.git/?a=commit;h=e86ff2a3128f4a0157ecfa10fee31d1416312c71
>
> Author: Huw Davies <huw at codeweavers.com>
> Date: Fri Dec 5 14:19:25 2008 +0000
>
> opengl/tests: Skip tests if we can't find a pixel format.
>
> ---
>
> dlls/opengl32/tests/opengl.c | 9 ++++++++-
> 1 files changed, 8 insertions(+), 1 deletions(-)
>
> diff --git a/dlls/opengl32/tests/opengl.c b/dlls/opengl32/tests/opengl.c
> index 35eb8ae..a3ac3af 100644
> --- a/dlls/opengl32/tests/opengl.c
> +++ b/dlls/opengl32/tests/opengl.c
> @@ -555,7 +555,12 @@ START_TEST(opengl)
> hdc = GetDC(hwnd);
>
> iPixelFormat = ChoosePixelFormat(hdc, &pfd);
> - ok(iPixelFormat > 0, "No pixelformat found!\n"); /* This should never happen as ChoosePixelFormat always returns a closest match */
> + if(iPixelFormat == 0)
> + {
> + /* This should never happen as ChoosePixelFormat always returns a closest match, but currently this fails in Wine if we don't have glX */
> + win_skip("Unable to find pixel format.\n");
> + goto cleanup;
> + }
>
> /* We shouldn't be able to create a context from a hdc which doesn't have a pixel format set */
> hglrc = wglCreateContext(hdc);
Hi Alexandre,
This wasn't what I intended my patch to do. The problem is that
Wine's ChoosePixelFormat() rets 0 on XServers without glX. The
win_skip results in a test failure. My patch was intended to mark
this as a todo_wine.
Huw.
--
huw at codeweavers.com
More information about the wine-devel
mailing list