August 2007 - Wine-commits - list.winehq.org

Juan Lang : crypt32: Don' t store redundant copy of crypto provider in file store.
by Alexandre Julliard 20 Aug '07

20 Aug '07

Module: wine Branch: master Commit: 9be5007669af733bf9ae81bedfca83eeda5f144f URL: http://source.winehq.org/git/wine.git/?a=commit;h=9be5007669af733bf9ae81bed… Author: Juan Lang <juan.lang(a)gmail.com> Date: Fri Aug 17 11:58:48 2007 -0700 crypt32: Don't store redundant copy of crypto provider in file store. --- dlls/crypt32/filestore.c | 2 -- 1 files changed, 0 insertions(+), 2 deletions(-) diff --git a/dlls/crypt32/filestore.c b/dlls/crypt32/filestore.c index 07ab6f1..45becca 100644 --- a/dlls/crypt32/filestore.c +++ b/dlls/crypt32/filestore.c @@ -28,7 +28,6 @@ WINE_DEFAULT_DEBUG_CHANNEL(crypt); typedef struct _WINE_FILESTOREINFO { DWORD dwOpenFlags; - HCRYPTPROV cryptProv; HCERTSTORE memStore; HANDLE file; BOOL dirty; @@ -182,7 +181,6 @@ PWINECRYPT_CERTSTORE CRYPT_FileOpenStore(HCRYPTPROV hCryptProv, DWORD dwFlags, CERT_STORE_PROV_INFO provInfo = { 0 }; info->dwOpenFlags = dwFlags; - info->cryptProv = hCryptProv; info->memStore = memStore; info->file = file; info->dirty = FALSE;

1 0

Paul Vriens : opengl32/tests: Be a bit more verbose when doing wglGetProcAddress.
by Alexandre Julliard 20 Aug '07

20 Aug '07

Module: wine Branch: master Commit: d64424216946b8cd9f5e9488db4a456ad805f294 URL: http://source.winehq.org/git/wine.git/?a=commit;h=d64424216946b8cd9f5e9488d… Author: Paul Vriens <paul.vriens.wine(a)gmail.com> Date: Fri Aug 17 19:02:46 2007 +0200 opengl32/tests: Be a bit more verbose when doing wglGetProcAddress. --- dlls/opengl32/tests/opengl.c | 19 +++++++++++++------ 1 files changed, 13 insertions(+), 6 deletions(-) diff --git a/dlls/opengl32/tests/opengl.c b/dlls/opengl32/tests/opengl.c index 2aca1d6..ccf5e3d 100644 --- a/dlls/opengl32/tests/opengl.c +++ b/dlls/opengl32/tests/opengl.c @@ -50,17 +50,24 @@ static const char* wgl_extensions = NULL; static void init_functions(void) { +#define GET_PROC(func) \ + p ## func = (void*)wglGetProcAddress(#func); \ + if(!p ## func) \ + trace("wglGetProcAddress(%s) failed\n", #func); + /* WGL_ARB_extensions_string */ - pwglGetExtensionsStringARB = (void*)wglGetProcAddress("wglGetExtensionsStringARB"); + GET_PROC(wglGetExtensionsStringARB) /* WGL_ARB_pixel_format */ - pwglChoosePixelFormatARB = (void*)wglGetProcAddress("wglChoosePixelFormatARB"); - pwglGetPixelFormatAttribivARB = (void*)wglGetProcAddress("wglGetPixelFormatAttribivARB"); + GET_PROC(wglChoosePixelFormatARB) + GET_PROC(wglGetPixelFormatAttribivARB) /* WGL_ARB_pbuffer */ - pwglCreatePbufferARB = (void*)wglGetProcAddress("wglCreatePbufferARB"); - pwglGetPbufferDCARB = (void*)wglGetProcAddress("wglGetPbufferDCARB"); - pwglReleasePbufferDCARB = (void*)wglGetProcAddress("wglReleasePbufferDCARB"); + GET_PROC(wglCreatePbufferARB) + GET_PROC(wglGetPbufferDCARB) + GET_PROC(wglReleasePbufferDCARB) + +#undef GET_PROC } static void test_pbuffers(HDC hdc)

1 0

Roderick Colenbrander : opengl32: Fix some gl type bugs.
by Alexandre Julliard 20 Aug '07

20 Aug '07

Module: wine Branch: master Commit: c9d4aec1ae39c3971ebfdc066cd793cffae9a935 URL: http://source.winehq.org/git/wine.git/?a=commit;h=c9d4aec1ae39c3971ebfdc066… Author: Roderick Colenbrander <thunderbird2k(a)gmx.net> Date: Fri Aug 17 19:22:12 2007 +0200 opengl32: Fix some gl type bugs. --- dlls/opengl32/make_opengl | 34 ++++++++-------------------------- dlls/opengl32/opengl_ext.c | 28 ++++++++++++++-------------- 2 files changed, 22 insertions(+), 40 deletions(-) diff --git a/dlls/opengl32/make_opengl b/dlls/opengl32/make_opengl index 97a12f4..aff30fb 100755 --- a/dlls/opengl32/make_opengl +++ b/dlls/opengl32/make_opengl @@ -6,12 +6,16 @@ use strict; # make_opengl path_to_spec_file opengl_version # # - path_to_spec_file is the path to the directory where the OpenGL -# spec files are located. These files are part of the OpenGL -# sample implementation CVS tree and are located in +# spec files are located. These files are hosted in the OpenGL +# extension registry at opengl.org: +# http://www.opengl.org/registry/api/gl.spec +# http://www.opengl.org/registry/api/gl.tm +# +# The files used to be hosted and maintained by SGI. You can still find +# find them in the sample implementation CVS tree which is located at # CVS_ROOT/projects/ogl-sample/main/doc/registry/specs. -# You can find them on the web at the following URL : +# You can also find them on the web at the following URL : # http://oss.sgi.com/cgi-bin/cvsweb.cgi/projects/ogl-sample/main/doc/registry… -# You will need gl.tm and gl.spec. # # - opengl_version is the OpenGL version emulated by the library # (can be 1.0 to 1.5). @@ -324,28 +328,6 @@ while (my $line = <TYPES>) { } # This is to override the 'void' -> '*' bogus conversion $pseudo_to_opengl{"void"} = "void"; -$pseudo_to_opengl{"Int64EXT"} = "INT64"; -$pseudo_to_opengl{"UInt64EXT"} = "UINT64"; -$pseudo_to_opengl{"BufferAccessARB"} = "GLenum"; -$pseudo_to_opengl{"BufferOffsetARB"} = "GLintptr"; -$pseudo_to_opengl{"BufferOffset"} = "GLintptr"; -$pseudo_to_opengl{"BufferPNameARB"} = "GLenum"; -$pseudo_to_opengl{"BufferPointerNameARB"} = "GLenum"; -$pseudo_to_opengl{"BufferSizeARB"} = "GLsizeiptr"; -$pseudo_to_opengl{"BufferSize"} = "GLsizeiptr"; -$pseudo_to_opengl{"BufferTargetARB"} = "GLenum"; -$pseudo_to_opengl{"BufferUsageARB"} = "GLenum"; -$pseudo_to_opengl{"FramebufferAttachment"} = "GLenum"; -$pseudo_to_opengl{"FramebufferTarget"} = "GLenum"; -$pseudo_to_opengl{"ProgramParameterPName"} = "GLenum"; -$pseudo_to_opengl{"ProgramTarget"} = "GLenum"; -$pseudo_to_opengl{"RenderbufferTarget"} = "GLenum"; -$pseudo_to_opengl{"VertexAttribEnum"} = "GLenum"; -# This is a bug in the spec file... -$pseudo_to_opengl{"FfdTargetSGIX"} = "GLint"; -$pseudo_to_opengl{"FfdMaskSGIX"} = "GLint"; -$pseudo_to_opengl{"IglooFunctionSelectSGIX"} = "GLint"; -$pseudo_to_opengl{"IglooParameterSGIX"} = "GLint"; # # Then, create the list of all OpenGL functions using the 'gl.spec' diff --git a/dlls/opengl32/opengl_ext.c b/dlls/opengl32/opengl_ext.c index 1c5c39b..fc638f7 100644 --- a/dlls/opengl32/opengl_ext.c +++ b/dlls/opengl32/opengl_ext.c @@ -1173,24 +1173,24 @@ static void WINAPI wine_glCurrentPaletteMatrixARB( GLint index ) { LEAVE_GL(); } -static void WINAPI wine_glDeformSGIX( GLint mask ) { - void (*func_glDeformSGIX)( GLint ) = extension_funcs[142]; +static void WINAPI wine_glDeformSGIX( GLbitfield mask ) { + void (*func_glDeformSGIX)( GLbitfield ) = extension_funcs[142]; TRACE("(%d)\n", mask ); ENTER_GL(); func_glDeformSGIX( mask ); LEAVE_GL(); } -static void WINAPI wine_glDeformationMap3dSGIX( GLint target, GLdouble u1, GLdouble u2, GLint ustride, GLint uorder, GLdouble v1, GLdouble v2, GLint vstride, GLint vorder, GLdouble w1, GLdouble w2, GLint wstride, GLint worder, GLdouble* points ) { - void (*func_glDeformationMap3dSGIX)( GLint, GLdouble, GLdouble, GLint, GLint, GLdouble, GLdouble, GLint, GLint, GLdouble, GLdouble, GLint, GLint, GLdouble* ) = extension_funcs[143]; +static void WINAPI wine_glDeformationMap3dSGIX( GLenum target, GLdouble u1, GLdouble u2, GLint ustride, GLint uorder, GLdouble v1, GLdouble v2, GLint vstride, GLint vorder, GLdouble w1, GLdouble w2, GLint wstride, GLint worder, GLdouble* points ) { + void (*func_glDeformationMap3dSGIX)( GLenum, GLdouble, GLdouble, GLint, GLint, GLdouble, GLdouble, GLint, GLint, GLdouble, GLdouble, GLint, GLint, GLdouble* ) = extension_funcs[143]; TRACE("(%d, %f, %f, %d, %d, %f, %f, %d, %d, %f, %f, %d, %d, %p)\n", target, u1, u2, ustride, uorder, v1, v2, vstride, vorder, w1, w2, wstride, worder, points ); ENTER_GL(); func_glDeformationMap3dSGIX( target, u1, u2, ustride, uorder, v1, v2, vstride, vorder, w1, w2, wstride, worder, points ); LEAVE_GL(); } -static void WINAPI wine_glDeformationMap3fSGIX( GLint target, GLfloat u1, GLfloat u2, GLint ustride, GLint uorder, GLfloat v1, GLfloat v2, GLint vstride, GLint vorder, GLfloat w1, GLfloat w2, GLint wstride, GLint worder, GLfloat* points ) { - void (*func_glDeformationMap3fSGIX)( GLint, GLfloat, GLfloat, GLint, GLint, GLfloat, GLfloat, GLint, GLint, GLfloat, GLfloat, GLint, GLint, GLfloat* ) = extension_funcs[144]; +static void WINAPI wine_glDeformationMap3fSGIX( GLenum target, GLfloat u1, GLfloat u2, GLint ustride, GLint uorder, GLfloat v1, GLfloat v2, GLint vstride, GLint vorder, GLfloat w1, GLfloat w2, GLint wstride, GLint worder, GLfloat* points ) { + void (*func_glDeformationMap3fSGIX)( GLenum, GLfloat, GLfloat, GLint, GLint, GLfloat, GLfloat, GLint, GLint, GLfloat, GLfloat, GLint, GLint, GLfloat* ) = extension_funcs[144]; TRACE("(%d, %f, %f, %d, %d, %f, %f, %d, %d, %f, %f, %d, %d, %p)\n", target, u1, u2, ustride, uorder, v1, v2, vstride, vorder, w1, w2, wstride, worder, points ); ENTER_GL(); func_glDeformationMap3fSGIX( target, u1, u2, ustride, uorder, v1, v2, vstride, vorder, w1, w2, wstride, worder, points ); @@ -3001,8 +3001,8 @@ static void WINAPI wine_glGetProgramivNV( GLuint id, GLenum pname, GLint* params LEAVE_GL(); } -static void WINAPI wine_glGetQueryObjecti64vEXT( GLuint id, GLenum pname, INT64* params ) { - void (*func_glGetQueryObjecti64vEXT)( GLuint, GLenum, INT64* ) = extension_funcs[368]; +static void WINAPI wine_glGetQueryObjecti64vEXT( GLuint id, GLenum pname, GLint64EXT* params ) { + void (*func_glGetQueryObjecti64vEXT)( GLuint, GLenum, GLint64EXT* ) = extension_funcs[368]; TRACE("(%d, %d, %p)\n", id, pname, params ); ENTER_GL(); func_glGetQueryObjecti64vEXT( id, pname, params ); @@ -3025,8 +3025,8 @@ static void WINAPI wine_glGetQueryObjectivARB( GLuint id, GLenum pname, GLint* p LEAVE_GL(); } -static void WINAPI wine_glGetQueryObjectui64vEXT( GLuint id, GLenum pname, UINT64* params ) { - void (*func_glGetQueryObjectui64vEXT)( GLuint, GLenum, UINT64* ) = extension_funcs[371]; +static void WINAPI wine_glGetQueryObjectui64vEXT( GLuint id, GLenum pname, GLuint64EXT* params ) { + void (*func_glGetQueryObjectui64vEXT)( GLuint, GLenum, GLuint64EXT* ) = extension_funcs[371]; TRACE("(%d, %d, %p)\n", id, pname, params ); ENTER_GL(); func_glGetQueryObjectui64vEXT( id, pname, params ); @@ -3523,8 +3523,8 @@ static void WINAPI wine_glHistogramEXT( GLenum target, GLsizei width, GLenum int LEAVE_GL(); } -static void WINAPI wine_glIglooInterfaceSGIX( GLint pname, GLint* params ) { - void (*func_glIglooInterfaceSGIX)( GLint, GLint* ) = extension_funcs[432]; +static void WINAPI wine_glIglooInterfaceSGIX( GLenum pname, GLvoid* params ) { + void (*func_glIglooInterfaceSGIX)( GLenum, GLvoid* ) = extension_funcs[432]; TRACE("(%d, %p)\n", pname, params ); ENTER_GL(); func_glIglooInterfaceSGIX( pname, params ); @@ -3857,8 +3857,8 @@ static void WINAPI wine_glListParameterivSGIX( GLuint list, GLenum pname, GLint* LEAVE_GL(); } -static void WINAPI wine_glLoadIdentityDeformationMapSGIX( GLint mask ) { - void (*func_glLoadIdentityDeformationMapSGIX)( GLint ) = extension_funcs[469]; +static void WINAPI wine_glLoadIdentityDeformationMapSGIX( GLbitfield mask ) { + void (*func_glLoadIdentityDeformationMapSGIX)( GLbitfield ) = extension_funcs[469]; TRACE("(%d)\n", mask ); ENTER_GL(); func_glLoadIdentityDeformationMapSGIX( mask );

1 0

Paul Vriens : oleaut32/tests: Skip VarDiv tests if there is no I8 and/ or UI8 support.
by Alexandre Julliard 20 Aug '07

20 Aug '07

Module: wine Branch: master Commit: 7287027681615951c525730bb7dab102a0a5eebf URL: http://source.winehq.org/git/wine.git/?a=commit;h=7287027681615951c525730bb… Author: Paul Vriens <paul.vriens.wine(a)gmail.com> Date: Fri Aug 17 16:28:57 2007 +0200 oleaut32/tests: Skip VarDiv tests if there is no I8 and/or UI8 support. --- dlls/oleaut32/tests/vartest.c | 78 ++++++++++++++++++++++++++++++---------- 1 files changed, 58 insertions(+), 20 deletions(-) diff --git a/dlls/oleaut32/tests/vartest.c b/dlls/oleaut32/tests/vartest.c index 9051986..5d855b6 100644 --- a/dlls/oleaut32/tests/vartest.c +++ b/dlls/oleaut32/tests/vartest.c @@ -6924,11 +6924,19 @@ static void test_VarDiv(void) { SKIPTESTDIV(leftvt); + /* Check if we need/have support for I8 */ + if (leftvt == VT_I8 && !HAVE_OLEAUT32_I8) + continue; + for (rightvt = 0; rightvt <= VT_BSTR_BLOB; rightvt++) { BOOL bFail = FALSE; SKIPTESTDIV(rightvt); + /* Check if we need/have support for I8 */ + if (rightvt == VT_I8 && !HAVE_OLEAUT32_I8) + continue; + /* Native crashes with VT_BYREF */ if (ExtraFlags[i] == VT_BYREF) continue; @@ -7040,7 +7048,10 @@ static void test_VarDiv(void) VARDIV(EMPTY,0,BSTR,num2_str,R8,0.0); VARDIV(EMPTY,0,BOOL,VARIANT_TRUE,R8,0.0); VARDIV(EMPTY,0,UI1,2,R8,0.0); - VARDIV(EMPTY,0,I8,2,R8,0.0); + if (HAVE_OLEAUT32_I8) + { + VARDIV(EMPTY,0,I8,2,R8,0.0); + } VARDIV(NULL,0,EMPTY,0,NULL,0); VARDIV(NULL,0,NULL,0,NULL,0); VARDIV(NULL,0,I2,2,NULL,0); @@ -7051,7 +7062,10 @@ static void test_VarDiv(void) VARDIV(NULL,0,BSTR,num2_str,NULL,0); VARDIV(NULL,0,BOOL,VARIANT_TRUE,NULL,0); VARDIV(NULL,0,UI1,2,NULL,0); - VARDIV(NULL,0,I8,2,NULL,0); + if (HAVE_OLEAUT32_I8) + { + VARDIV(NULL,0,I8,2,NULL,0); + } VARDIV(I2,2,NULL,0,NULL,0); VARDIV(I2,1,I2,2,R8,0.5); VARDIV(I2,1,I4,2,R8,0.5); @@ -7060,7 +7074,10 @@ static void test_VarDiv(void) VARDIV(I2,1,DATE,2,R8,0.5); VARDIV(I2,1,BOOL,VARIANT_TRUE,R8,-1.0); VARDIV(I2,1,UI1,2,R8,0.5); - VARDIV(I2,1,I8,2,R8,0.5); + if (HAVE_OLEAUT32_I8) + { + VARDIV(I2,1,I8,2,R8,0.5); + } VARDIV(I4,1,NULL,0,NULL,0); VARDIV(I4,1,I2,2,R8,0.5); VARDIV(I4,1,I4,2,R8,0.5); @@ -7070,7 +7087,10 @@ static void test_VarDiv(void) VARDIV(I4,1,BSTR,num2_str,R8,0.5); VARDIV(I4,1,BOOL,VARIANT_TRUE,R8,-1.0); VARDIV(I4,1,UI1,2,R8,0.5); - VARDIV(I4,1,I8,2,R8,0.5); + if (HAVE_OLEAUT32_I8) + { + VARDIV(I4,1,I8,2,R8,0.5); + } VARDIV(R4,1.0f,NULL,0,NULL,0); VARDIV(R4,1.0f,I2,2,R4,0.5f); VARDIV(R4,1.0f,I4,2,R8,0.5); @@ -7080,7 +7100,10 @@ static void test_VarDiv(void) VARDIV(R4,1.0f,BSTR,num2_str,R8,0.5); VARDIV(R4,1.0f,BOOL,VARIANT_TRUE,R4,-1); VARDIV(R4,1.0f,UI1,2,R4,0.5f); - VARDIV(R4,1.0f,I8,2,R8,0.5); + if (HAVE_OLEAUT32_I8) + { + VARDIV(R4,1.0f,I8,2,R8,0.5); + } VARDIV(R8,1.0,NULL,0,NULL,0); VARDIV(R8,1.0,I2,2,R8,0.5); VARDIV(R8,1.0,I4,2,R8,0.5); @@ -7090,7 +7113,10 @@ static void test_VarDiv(void) VARDIV(R8,1.0,BSTR,num2_str,R8,0.5); VARDIV(R8,1.0,BOOL,VARIANT_TRUE,R8,-1.0); VARDIV(R8,1.0,UI1,2,R8,0.5); - VARDIV(R8,1.0,I8,2,R8,0.5); + if (HAVE_OLEAUT32_I8) + { + VARDIV(R8,1.0,I8,2,R8,0.5); + } VARDIV(DATE,1,NULL,0,NULL,0); VARDIV(DATE,1,I2,2,R8,0.5); VARDIV(DATE,1,I4,2,R8,0.5); @@ -7100,7 +7126,10 @@ static void test_VarDiv(void) VARDIV(DATE,1,BSTR,num2_str,R8,0.5); VARDIV(DATE,1,BOOL,VARIANT_TRUE,R8,-1.0); VARDIV(DATE,1,UI1,2,R8,0.5); - VARDIV(DATE,1,I8,2,R8,0.5); + if (HAVE_OLEAUT32_I8) + { + VARDIV(DATE,1,I8,2,R8,0.5); + } VARDIV(BSTR,num1_str,NULL,0,NULL,0); VARDIV(BSTR,num1_str,I2,2,R8,0.5); VARDIV(BSTR,num1_str,I4,2,R8,0.5); @@ -7110,7 +7139,10 @@ static void test_VarDiv(void) VARDIV(BSTR,num1_str,BSTR,num2_str,R8,0.5); VARDIV(BSTR,num1_str,BOOL,VARIANT_TRUE,R8,-1); VARDIV(BSTR,num1_str,UI1,2,R8,0.5); - VARDIV(BSTR,num1_str,I8,2,R8,0.5); + if (HAVE_OLEAUT32_I8) + { + VARDIV(BSTR,num1_str,I8,2,R8,0.5); + } VARDIV(BOOL,VARIANT_TRUE,NULL,0,NULL,0); VARDIV(BOOL,VARIANT_TRUE,I2,1,R8,-1.0); VARDIV(BOOL,VARIANT_FALSE,I2,1,R8,0.0); @@ -7125,7 +7157,10 @@ static void test_VarDiv(void) VARDIV(BOOL,VARIANT_TRUE,BOOL,VARIANT_TRUE,R8,1.0); VARDIV(BOOL,VARIANT_FALSE,BOOL,VARIANT_TRUE,R8,0.0); VARDIV(BOOL,VARIANT_TRUE,UI1,1,R8,-1.0); - VARDIV(BOOL,VARIANT_TRUE,I8,1,R8,-1.0); + if (HAVE_OLEAUT32_I8) + { + VARDIV(BOOL,VARIANT_TRUE,I8,1,R8,-1.0); + } VARDIV(UI1,1,NULL,0,NULL,0); VARDIV(UI1,1,I2,2,R8,0.5); VARDIV(UI1,1,I4,2,R8,0.5); @@ -7135,17 +7170,20 @@ static void test_VarDiv(void) VARDIV(UI1,1,BSTR,num2_str,R8,0.5); VARDIV(UI1,1,BOOL,VARIANT_TRUE,R8,-1); VARDIV(UI1,1,UI1,2,R8,0.5); - VARDIV(UI1,1,I8,2,R8,0.5); - VARDIV(I8,1,NULL,0,NULL,0); - VARDIV(I8,1,I2,2,R8,0.5); - VARDIV(I8,1,I4,2,R8,0.5); - VARDIV(I8,1,R4,2.0f,R8,0.5); - VARDIV(I8,1,R8,2.0,R8,0.5); - VARDIV(I8,1,DATE,2,R8,0.5); - VARDIV(I8,1,BSTR,num2_str,R8,0.5); - VARDIV(I8,1,BOOL,VARIANT_TRUE,R8,-1); - VARDIV(I8,1,UI1,2,R8,0.5); - VARDIV(I8,1,I8,2,R8,0.5); + if (HAVE_OLEAUT32_I8) + { + VARDIV(UI1,1,I8,2,R8,0.5); + VARDIV(I8,1,NULL,0,NULL,0); + VARDIV(I8,1,I2,2,R8,0.5); + VARDIV(I8,1,I4,2,R8,0.5); + VARDIV(I8,1,R4,2.0f,R8,0.5); + VARDIV(I8,1,R8,2.0,R8,0.5); + VARDIV(I8,1,DATE,2,R8,0.5); + VARDIV(I8,1,BSTR,num2_str,R8,0.5); + VARDIV(I8,1,BOOL,VARIANT_TRUE,R8,-1); + VARDIV(I8,1,UI1,2,R8,0.5); + VARDIV(I8,1,I8,2,R8,0.5); + } /* Manually test some VT_CY, VT_DECIMAL variants */ V_VT(&cy) = VT_CY;

1 0

Paul Vriens : oleaut32/tests: Skip VarCat tests if there is no I8 and/ or UI8 support.
by Alexandre Julliard 20 Aug '07

20 Aug '07

Module: wine Branch: master Commit: c36acfce381e68bd6c263fde48e475661987a4cf URL: http://source.winehq.org/git/wine.git/?a=commit;h=c36acfce381e68bd6c263fde4… Author: Paul Vriens <paul.vriens.wine(a)gmail.com> Date: Fri Aug 17 15:48:22 2007 +0200 oleaut32/tests: Skip VarCat tests if there is no I8 and/or UI8 support. --- dlls/oleaut32/tests/vartest.c | 9 +++++++++ 1 files changed, 9 insertions(+), 0 deletions(-) diff --git a/dlls/oleaut32/tests/vartest.c b/dlls/oleaut32/tests/vartest.c index be96f25..756c477 100644 --- a/dlls/oleaut32/tests/vartest.c +++ b/dlls/oleaut32/tests/vartest.c @@ -102,6 +102,9 @@ static void init(void) #define HAVE_FUNC(func) ((void*)GetProcAddress(hOleaut32, #func) != NULL) HAVE_OLEAUT32_I8 = HAVE_FUNC(VarI8FromI1); + if (!HAVE_OLEAUT32_I8) + skip("No support for I8 and UI8 data types\n"); + HAVE_OLEAUT32_RECORD = HAVE_FUNC(SafeArraySetRecordInfo); IS_ANCIENT = (!HAVE_FUNC(VarI1FromI2)); @@ -5150,6 +5153,12 @@ static void test_VarCat(void) leftvt == 15 || rightvt == 15 /* Undefined type */) continue; + /* Check if we need/have support for I8 and/or UI8 */ + if ((leftvt == VT_I8 || leftvt == VT_UI8 || + rightvt == VT_I8 || rightvt == VT_UI8) && + !HAVE_OLEAUT32_I8) + continue; + if (leftvt == VT_NULL && rightvt == VT_NULL) resultvt = VT_NULL; else if (leftvt == VT_VARIANT && (rightvt == VT_ERROR ||

1 0

Paul Vriens : oleaut32/tests: Skip VarAnd tests if there is no I8 and/ or UI8 support.
by Alexandre Julliard 20 Aug '07

20 Aug '07

Module: wine Branch: master Commit: 60819600259752f578d608632612bc18420a31bc URL: http://source.winehq.org/git/wine.git/?a=commit;h=60819600259752f578d608632… Author: Paul Vriens <paul.vriens.wine(a)gmail.com> Date: Fri Aug 17 15:53:50 2007 +0200 oleaut32/tests: Skip VarAnd tests if there is no I8 and/or UI8 support. --- dlls/oleaut32/tests/vartest.c | 6 ++++++ 1 files changed, 6 insertions(+), 0 deletions(-) diff --git a/dlls/oleaut32/tests/vartest.c b/dlls/oleaut32/tests/vartest.c index 756c477..9051986 100644 --- a/dlls/oleaut32/tests/vartest.c +++ b/dlls/oleaut32/tests/vartest.c @@ -5470,6 +5470,12 @@ static void test_VarAnd(void) BOOL bFail = FALSE; SKIPTESTAND(rightvt); + /* Check if we need/have support for I8 and/or UI8 */ + if ((leftvt == VT_I8 || leftvt == VT_UI8 || + rightvt == VT_I8 || rightvt == VT_UI8) && + !HAVE_OLEAUT32_I8) + continue; + memset(&left, 0, sizeof(left)); memset(&right, 0, sizeof(right)); V_VT(&left) = leftvt | ExtraFlags[i];

1 0

Paul Vriens : oleaut32/tests: Use GetModuleHandle and add a few skip's.
by Alexandre Julliard 20 Aug '07

20 Aug '07

Module: wine Branch: master Commit: 9b396275ad637697815e24582f7420088a7baf83 URL: http://source.winehq.org/git/wine.git/?a=commit;h=9b396275ad637697815e24582… Author: Paul Vriens <paul.vriens.wine(a)gmail.com> Date: Fri Aug 17 14:10:32 2007 +0200 oleaut32/tests: Use GetModuleHandle and add a few skip's. --- dlls/oleaut32/tests/olefont.c | 5 ++++- dlls/oleaut32/tests/olepicture.c | 5 ++++- dlls/oleaut32/tests/safearray.c | 2 +- dlls/oleaut32/tests/varformat.c | 2 +- dlls/oleaut32/tests/vartype.c | 5 +---- 5 files changed, 11 insertions(+), 8 deletions(-) diff --git a/dlls/oleaut32/tests/olefont.c b/dlls/oleaut32/tests/olefont.c index 69621cf..1e6678b 100644 --- a/dlls/oleaut32/tests/olefont.c +++ b/dlls/oleaut32/tests/olefont.c @@ -798,10 +798,13 @@ static void test_AddRefHfont(void) START_TEST(olefont) { - hOleaut32 = LoadLibraryA("oleaut32.dll"); + hOleaut32 = GetModuleHandleA("oleaut32.dll"); pOleCreateFontIndirect = (void*)GetProcAddress(hOleaut32, "OleCreateFontIndirect"); if (!pOleCreateFontIndirect) + { + skip("OleCreateFontIndirect not available\n"); return; + } test_QueryInterface(); test_type_info(); diff --git a/dlls/oleaut32/tests/olepicture.c b/dlls/oleaut32/tests/olepicture.c index 00027de..99cefd2 100644 --- a/dlls/oleaut32/tests/olepicture.c +++ b/dlls/oleaut32/tests/olepicture.c @@ -426,11 +426,14 @@ static void test_OleCreatePictureIndirect(void) START_TEST(olepicture) { - hOleaut32 = LoadLibraryA("oleaut32.dll"); + hOleaut32 = GetModuleHandleA("oleaut32.dll"); pOleLoadPicture = (void*)GetProcAddress(hOleaut32, "OleLoadPicture"); pOleCreatePictureIndirect = (void*)GetProcAddress(hOleaut32, "OleCreatePictureIndirect"); if (!pOleLoadPicture) + { + skip("OleLoadPicture is not available\n"); return; + } /* Test regular 1x1 pixel images of gif, jpg, bmp type */ test_pic(gifimage, sizeof(gifimage)); diff --git a/dlls/oleaut32/tests/safearray.c b/dlls/oleaut32/tests/safearray.c index 429f08c..1857ab5 100644 --- a/dlls/oleaut32/tests/safearray.c +++ b/dlls/oleaut32/tests/safearray.c @@ -1682,7 +1682,7 @@ static void test_SafeArrayDestroyData (void) START_TEST(safearray) { - hOleaut32 = LoadLibraryA("oleaut32.dll"); + hOleaut32 = GetModuleHandleA("oleaut32.dll"); GETPTR(SafeArrayAllocDescriptorEx); GETPTR(SafeArrayCopyData); diff --git a/dlls/oleaut32/tests/varformat.c b/dlls/oleaut32/tests/varformat.c index afec605..f7c7c28 100644 --- a/dlls/oleaut32/tests/varformat.c +++ b/dlls/oleaut32/tests/varformat.c @@ -394,7 +394,7 @@ static void test_VarFormat(void) START_TEST(varformat) { - hOleaut32 = LoadLibraryA("oleaut32.dll"); + hOleaut32 = GetModuleHandleA("oleaut32.dll"); test_VarFormatNumber(); test_VarFormat(); diff --git a/dlls/oleaut32/tests/vartype.c b/dlls/oleaut32/tests/vartype.c index d6f2693..49106ff 100644 --- a/dlls/oleaut32/tests/vartype.c +++ b/dlls/oleaut32/tests/vartype.c @@ -5765,10 +5765,7 @@ static void test_ChangeType_keep_dst(void) START_TEST(vartype) { - hOleaut32 = LoadLibraryA("oleaut32.dll"); - ok(hOleaut32 != 0, "Failed to load oleaut32.dll\n"); - if (!hOleaut32) - return; + hOleaut32 = GetModuleHandleA("oleaut32.dll"); trace("LCID's: System=0x%08x, User=0x%08x\n", GetSystemDefaultLCID(), GetUserDefaultLCID());

1 0

Paul Vriens : oleaut32/tests: Add a few skip's to the vartype tests.
by Alexandre Julliard 20 Aug '07

20 Aug '07

Module: wine Branch: master Commit: 7c0f0abe153b41e4baaf4dd7ae94b10b2632c5eb URL: http://source.winehq.org/git/wine.git/?a=commit;h=7c0f0abe153b41e4baaf4dd7a… Author: Paul Vriens <paul.vriens.wine(a)gmail.com> Date: Fri Aug 17 14:21:09 2007 +0200 oleaut32/tests: Add a few skip's to the vartype tests. --- dlls/oleaut32/tests/vartype.c | 14 +++++++++++++- 1 files changed, 13 insertions(+), 1 deletions(-) diff --git a/dlls/oleaut32/tests/vartype.c b/dlls/oleaut32/tests/vartype.c index 49106ff..4558d82 100644 --- a/dlls/oleaut32/tests/vartype.c +++ b/dlls/oleaut32/tests/vartype.c @@ -46,7 +46,7 @@ static HMODULE hOleaut32; /* Get a conversion function ptr, return if function not available */ #define CHECKPTR(func) p##func = (void*)GetProcAddress(hOleaut32, #func); \ if (!p##func) { \ - trace("function " # func " not available, not testing it\n"); return; } + skip("function " # func " not available, not testing it\n"); return; } /* Is a given function exported from oleaut32? */ #define HAVE_FUNC(func) ((void*)GetProcAddress(hOleaut32, #func) != NULL) @@ -2347,7 +2347,10 @@ static void test_VarI8Copy(void) LONGLONG in = 1; if (!HAVE_OLEAUT32_I8) + { + skip("I8 and UI8 data types are not available\n"); return; + } VariantInit(&vSrc); VariantInit(&vDst); @@ -2374,7 +2377,10 @@ static void test_VarI8ChangeTypeEx(void) VARIANTARG vSrc, vDst; if (!HAVE_OLEAUT32_I8) + { + skip("I8 and UI8 data types are not available\n"); return; + } in = 1; @@ -2603,7 +2609,10 @@ static void test_VarUI8Copy(void) ULONGLONG in = 1; if (!HAVE_OLEAUT32_I8) + { + skip("I8 and UI8 data types are not available\n"); return; + } VariantInit(&vSrc); VariantInit(&vDst); @@ -2630,7 +2639,10 @@ static void test_VarUI8ChangeTypeEx(void) VARIANTARG vSrc, vDst; if (!HAVE_OLEAUT32_I8) + { + skip("I8 and UI8 data types are not available\n"); return; + } in = 1;

1 0

Paul Vriens : oleaut32/tests: Fix a few typos.
by Alexandre Julliard 20 Aug '07

20 Aug '07

Module: wine Branch: master Commit: 9226cee6e9271cb6c54327b6d9adbf74181b1c93 URL: http://source.winehq.org/git/wine.git/?a=commit;h=9226cee6e9271cb6c54327b6d… Author: Paul Vriens <paul.vriens.wine(a)gmail.com> Date: Fri Aug 17 15:25:35 2007 +0200 oleaut32/tests: Fix a few typos. --- dlls/oleaut32/tests/vartest.c | 6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/dlls/oleaut32/tests/vartest.c b/dlls/oleaut32/tests/vartest.c index 9c5dd52..be96f25 100644 --- a/dlls/oleaut32/tests/vartest.c +++ b/dlls/oleaut32/tests/vartest.c @@ -6857,7 +6857,7 @@ static void test_VarPow(void) hres = pVarPow(&dec, &right, &result); ok(hres == S_OK && V_VT(&result) == VT_R8, - "VARPOW: expected coerced hres 0x%X type VT_R8, got hres 0x%X type%s!\n", + "VARPOW: expected coerced hres 0x%X type VT_R8, got hres 0x%X type %s!\n", S_OK, hres, vtstr(V_VT(&result))); ok(hres == S_OK && EQ_DOUBLE(V_R8(&result), 4.0), "VARPOW: DECIMAL value %f, expected %f\n", V_R8(&result), 4.0); @@ -7682,7 +7682,7 @@ static void test_VarIdiv(void) hres = VarIdiv(&dec, &right, &result); ok(hres == S_OK && V_VT(&result) == VT_I8, - "VARIDIV: expected coerced hres 0x%X type VT_I8, got hres 0x%X type%s!\n", + "VARIDIV: expected coerced hres 0x%X type VT_I8, got hres 0x%X type %s!\n", S_OK, hres, vtstr(V_VT(&result))); ok(hres == S_OK && V_I8(&result) == 1, "VARIDIV: DECIMAL value 0x%x%08x, expected %d\n", @@ -8200,7 +8200,7 @@ static void test_VarImp(void) hres = pVarImp(&dec, &right, &result); ok(hres == S_OK && V_VT(&result) == VT_I8, - "VARIMP: expected coerced hres 0x%X type VT_I8, got hres 0x%X type%s!\n", + "VARIMP: expected coerced hres 0x%X type VT_I8, got hres 0x%X type %s!\n", S_OK, hres, vtstr(V_VT(&result))); ok(hres == S_OK && V_I8(&result) == -3, "VARIMP: DECIMAL value 0x%x%08x, expected %d\n",

1 0

Mikolaj Zalewski : msvcrt: __mb_cur_max is not updated in _setmbcp ( with testcase) and should not be used in _ismbblead.
by Alexandre Julliard 17 Aug '07

17 Aug '07

Module: wine Branch: master Commit: e9ec5d4aff33a76890ff6cb7254e8483267d7e08 URL: http://source.winehq.org/git/wine.git/?a=commit;h=e9ec5d4aff33a76890ff6cb72… Author: Mikolaj Zalewski <mikolajz(a)google.com> Date: Thu Aug 16 11:14:48 2007 -0700 msvcrt: __mb_cur_max is not updated in _setmbcp (with testcase) and should not be used in _ismbblead. --- dlls/msvcrt/mbcs.c | 2 +- dlls/msvcrt/tests/string.c | 2 ++ 2 files changed, 3 insertions(+), 1 deletions(-) diff --git a/dlls/msvcrt/mbcs.c b/dlls/msvcrt/mbcs.c index 07e38ec..0dc0120 100644 --- a/dlls/msvcrt/mbcs.c +++ b/dlls/msvcrt/mbcs.c @@ -911,7 +911,7 @@ int CDECL _ismbckata(unsigned int c) int CDECL _ismbblead(unsigned int c) { /* FIXME: should reference MSVCRT_mbctype */ - return MSVCRT___mb_cur_max > 1 && MSVCRT_isleadbyte(c); + return MSVCRT_isleadbyte(c); } diff --git a/dlls/msvcrt/tests/string.c b/dlls/msvcrt/tests/string.c index 7b434f7..d7148da 100644 --- a/dlls/msvcrt/tests/string.c +++ b/dlls/msvcrt/tests/string.c @@ -78,8 +78,10 @@ static void test_swab( void ) { static void test_ismbblead(void) { unsigned int s = '\354'; + int mb_orig_max = __mb_cur_max; _setmbcp(936); + ok(__mb_cur_max == mb_orig_max, "__mb_cur_max shouldn't be updated (is %d != %d)\n", __mb_cur_max, mb_orig_max); todo_wine ok(_ismbblead(s), "got result %d\n", _ismbblead(s)); _setmbcp(1252); }

1 0