wine-commits October 2015

wine-commits@winehq.org

2 participants
749 discussions

Nikolay Sivov : ddraw: Use proper prototype for internal enumeration callback (PVS-Studio).
by Alexandre Julliard 20 Oct '15

20 Oct '15

Module: wine Branch: master Commit: b2855505b9cc0b206404e9a8da382bd20d7294e8 URL: http://source.winehq.org/git/wine.git/?a=commit;h=b2855505b9cc0b206404e9a8d… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Tue Oct 20 16:33:40 2015 +0300 ddraw: Use proper prototype for internal enumeration callback (PVS-Studio). Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/ddraw/main.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dlls/ddraw/main.c b/dlls/ddraw/main.c index 8631827..06b448f 100644 --- a/dlls/ddraw/main.c +++ b/dlls/ddraw/main.c @@ -48,7 +48,7 @@ struct callback_info }; /* Enumeration callback for converting DirectDrawEnumerateA to DirectDrawEnumerateExA */ -static HRESULT CALLBACK enum_callback(GUID *guid, char *description, char *driver_name, +static BOOL CALLBACK enum_callback(GUID *guid, char *description, char *driver_name, void *context, HMONITOR monitor) { const struct callback_info *info = context;

1 0

Nikolay Sivov : shell32: Cleanup ReadCabinetState exports.
by Alexandre Julliard 20 Oct '15

20 Oct '15

Module: wine Branch: master Commit: 4ee4c06410214e418d35cee3671460e2924059ba URL: http://source.winehq.org/git/wine.git/?a=commit;h=4ee4c06410214e418d35cee36… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Tue Oct 20 16:16:54 2015 +0300 shell32: Cleanup ReadCabinetState exports. Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/shell32/shell32.spec | 6 +++--- dlls/shell32/shellord.c | 15 --------------- 2 files changed, 3 insertions(+), 18 deletions(-) diff --git a/dlls/shell32/shell32.spec b/dlls/shell32/shell32.spec index 91a4138..93fd449 100644 --- a/dlls/shell32/shell32.spec +++ b/dlls/shell32/shell32.spec @@ -245,10 +245,10 @@ 650 stdcall -noname PathIsSameRoot(ptr ptr) PathIsSameRootAW - 651 stdcall -noname ReadCabinetState(long long) # OldReadCabinetState - 652 stdcall -noname WriteCabinetState(long) + 651 stdcall -noname @(ptr long) ReadCabinetState # OldReadCabinetState + 652 stdcall -noname WriteCabinetState(ptr) 653 stdcall -noname PathProcessCommand(long long long long) PathProcessCommandAW - 654 stdcall @(long long) shell32_654 # ReadCabinetState@8 + 654 stdcall ReadCabinetState(ptr long) 660 stdcall -noname FileIconInit(long) 680 stdcall IsUserAnAdmin() diff --git a/dlls/shell32/shellord.c b/dlls/shell32/shellord.c index 7387520..60492de 100644 --- a/dlls/shell32/shellord.c +++ b/dlls/shell32/shellord.c @@ -1555,21 +1555,6 @@ BOOL WINAPI SHWaitForFileToOpen( } /************************************************************************ - * @ [SHELL32.654] - * - * NOTES - * first parameter seems to be a pointer (same as passed to WriteCabinetState) - * second one could be a size (0x0c). The size is the same as the structure saved to - * HCU\Software\Microsoft\Windows\CurrentVersion\Explorer\CabinetState - * I'm (js) guessing: this one is just ReadCabinetState ;-) - */ -HRESULT WINAPI shell32_654 (CABINETSTATE *cs, int length) -{ - TRACE("%p %d\n",cs,length); - return ReadCabinetState(cs,length); -} - -/************************************************************************ * RLBuildListOfPaths [SHELL32.146] * * NOTES

1 0

Nikolay Sivov : xmllite: Don't use HRESULT codes for BOOL retval ( PVS-Studio).
by Alexandre Julliard 20 Oct '15

20 Oct '15

Module: wine Branch: master Commit: dc72103a1df686d40bd333d9f8e6b26f59884a5a URL: http://source.winehq.org/git/wine.git/?a=commit;h=dc72103a1df686d40bd333d9f… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Tue Oct 20 15:59:30 2015 +0300 xmllite: Don't use HRESULT codes for BOOL retval (PVS-Studio). Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/xmllite/reader.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dlls/xmllite/reader.c b/dlls/xmllite/reader.c index 525b4f7..8a078a4 100644 --- a/dlls/xmllite/reader.c +++ b/dlls/xmllite/reader.c @@ -2855,7 +2855,7 @@ static HRESULT WINAPI xmlreader_GetDepth(IXmlReader* iface, UINT *depth) static BOOL WINAPI xmlreader_IsEOF(IXmlReader* iface) { FIXME("(%p): stub\n", iface); - return E_NOTIMPL; + return FALSE; } static const struct IXmlReaderVtbl xmlreader_vtbl =

1 0

Nikolay Sivov : usp10/tests: Use BOOL variable for BOOL retval (PVS-Studio ).
by Alexandre Julliard 20 Oct '15

20 Oct '15

Module: wine Branch: master Commit: 9cede2f0a6e1d22d4cb5e4f9e9f0ea0d359a8fda URL: http://source.winehq.org/git/wine.git/?a=commit;h=9cede2f0a6e1d22d4cb5e4f9e… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Tue Oct 20 15:57:36 2015 +0300 usp10/tests: Use BOOL variable for BOOL retval (PVS-Studio). Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/usp10/tests/usp10.c | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/dlls/usp10/tests/usp10.c b/dlls/usp10/tests/usp10.c index bfd84d4..7e33ecc 100644 --- a/dlls/usp10/tests/usp10.c +++ b/dlls/usp10/tests/usp10.c @@ -2267,15 +2267,17 @@ static void test_ScriptTextOut2(HDC hdc) ok (hr == S_OK, "ScriptShape should return S_OK not (%08x)\n", hr); ok (psc != NULL, "psc should not be null and have SCRIPT_CACHE buffer address\n"); ok (pcGlyphs == cChars, "Chars in (%d) should equal Glyphs out (%d)\n", cChars, pcGlyphs); - if (hr ==0) { + if (hr == S_OK) { + BOOL ret; + /* Note hdc is needed as glyph info is not yet in psc */ hr = ScriptPlace(hdc2, &psc, pwOutGlyphs1, pcGlyphs, psva, &pItem[0].a, piAdvance, pGoffset, pABC); ok (hr == S_OK, "Should return S_OK not (%08x)\n", hr); /* key part!!! cached dc is being deleted */ - hr = DeleteDC(hdc2); - ok(hr == 1, "DeleteDC should return 1 not %08x\n", hr); + ret = DeleteDC(hdc2); + ok(ret, "DeleteDC should return 1 not %d\n", ret); /* At this point the cached hdc (hdc2) has been destroyed, * however, we are passing in a *real* hdc (the original hdc).

1 0

Nikolay Sivov : oleaut32/tests: Don't use HRESULT codes for BOOL retval ( PVS-Studio).
by Alexandre Julliard 20 Oct '15

20 Oct '15

Module: wine Branch: master Commit: b5699c80dd7e95c2c91f58593db9b66daec0e0f8 URL: http://source.winehq.org/git/wine.git/?a=commit;h=b5699c80dd7e95c2c91f58593… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Tue Oct 20 15:52:50 2015 +0300 oleaut32/tests: Don't use HRESULT codes for BOOL retval (PVS-Studio). Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/oleaut32/tests/safearray.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dlls/oleaut32/tests/safearray.c b/dlls/oleaut32/tests/safearray.c index dd9329a..12e1f54 100644 --- a/dlls/oleaut32/tests/safearray.c +++ b/dlls/oleaut32/tests/safearray.c @@ -212,7 +212,7 @@ static HRESULT WINAPI RecordInfo_GetFieldNames(IRecordInfo *iface, ULONG *pcName static BOOL WINAPI RecordInfo_IsMatchingType(IRecordInfo *iface, IRecordInfo *info2) { ok(0, "unexpected call\n"); - return E_NOTIMPL; + return FALSE; } static PVOID WINAPI RecordInfo_RecordCreate(IRecordInfo *iface)

1 0

Nikolay Sivov : ole32/tests: Use BOOL variables for BOOL return type ( PVS-Studio).
by Alexandre Julliard 20 Oct '15

20 Oct '15

Module: wine Branch: master Commit: 4afda020ae2185855bd296cce3083a10a3de6f17 URL: http://source.winehq.org/git/wine.git/?a=commit;h=4afda020ae2185855bd296cce… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Tue Oct 20 15:16:58 2015 +0300 ole32/tests: Use BOOL variables for BOOL return type (PVS-Studio). Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/ole32/tests/storage32.c | 59 ++++++++++++++++++++++++++------------------ 1 file changed, 35 insertions(+), 24 deletions(-) diff --git a/dlls/ole32/tests/storage32.c b/dlls/ole32/tests/storage32.c index fe5ddbc..edd0b3f 100644 --- a/dlls/ole32/tests/storage32.c +++ b/dlls/ole32/tests/storage32.c @@ -513,6 +513,7 @@ static void test_storage_stream(void) ULARGE_INTEGER p; unsigned char buffer[0x100]; IUnknown *unk; + BOOL ret; DeleteFileA(filenameA); @@ -691,8 +692,8 @@ static void test_storage_stream(void) IStorage_Release(stg); } - r = DeleteFileA(filenameA); - ok(r, "file should exist\n"); + ret = DeleteFileA(filenameA); + ok(ret, "file should exist\n"); } static BOOL touch_file(LPCSTR filename) @@ -739,6 +740,7 @@ static void test_open_storage(void) IStorage *stg = NULL, *stg2 = NULL; HRESULT r; DWORD stgm; + BOOL ret; /* try opening a zero length file - it should stay zero length */ DeleteFileA(filenameA); @@ -898,8 +900,8 @@ static void test_open_storage(void) r = StgOpenStorage( filename, NULL, STGM_NOSNAPSHOT | STGM_PRIORITY, NULL, 0, &stg); ok(r == STG_E_INVALIDFLAG, "should fail\n"); - r = DeleteFileA(filenameA); - ok(r, "file didn't exist\n"); + ret = DeleteFileA(filenameA); + ok(ret, "file didn't exist\n"); } static void test_storage_suminfo(void) @@ -1397,6 +1399,7 @@ static void test_transact(void) static const WCHAR stmname2[] = { 'F','O','O',0 }; static const WCHAR stgname[] = { 'P','E','R','M','S','T','G',0 }; static const WCHAR stgname2[] = { 'T','E','M','P','S','T','G',0 }; + BOOL ret; DeleteFileA(filenameA); @@ -1517,8 +1520,8 @@ static void test_transact(void) IStorage_Release(stg); - r = DeleteFileA(filenameA); - ok( r == TRUE, "deleted file\n"); + ret = DeleteFileA(filenameA); + ok(ret, "deleted file\n"); } static void test_substorage_share(void) @@ -1529,6 +1532,7 @@ static void test_substorage_share(void) static const WCHAR stgname[] = { 'P','E','R','M','S','T','G',0 }; static const WCHAR stmname[] = { 'C','O','N','T','E','N','T','S',0 }; static const WCHAR othername[] = { 'N','E','W','N','A','M','E',0 }; + BOOL ret; DeleteFileA(filenameA); @@ -1608,8 +1612,8 @@ static void test_substorage_share(void) IStorage_Release(stg); - r = DeleteFileA(filenameA); - ok( r == TRUE, "deleted file\n"); + ret = DeleteFileA(filenameA); + ok(ret, "deleted file\n"); } static void test_revert(void) @@ -1622,6 +1626,7 @@ static void test_revert(void) static const WCHAR stgname[] = { 'P','E','R','M','S','T','G',0 }; static const WCHAR stgname2[] = { 'T','E','M','P','S','T','G',0 }; STATSTG statstg; + BOOL ret; DeleteFileA(filenameA); @@ -1739,8 +1744,8 @@ static void test_revert(void) IStorage_Release(stg); - r = DeleteFileA(filenameA); - ok( r == TRUE, "deleted file\n"); + ret = DeleteFileA(filenameA); + ok(ret, "deleted file\n"); /* Revert only invalidates objects in transacted mode */ r = StgCreateDocfile( filename, STGM_CREATE | STGM_SHARE_EXCLUSIVE | @@ -1759,8 +1764,8 @@ static void test_revert(void) IStream_Release(stm); IStorage_Release(stg); - r = DeleteFileA(filenameA); - ok( r == TRUE, "deleted file\n"); + ret = DeleteFileA(filenameA); + ok(ret, "deleted file\n"); } static void test_parent_free(void) @@ -1772,6 +1777,7 @@ static void test_parent_free(void) static const WCHAR stgname[] = { 'P','E','R','M','S','T','G',0 }; ULONG ref; STATSTG statstg; + BOOL ret; DeleteFileA(filenameA); @@ -1821,8 +1827,8 @@ static void test_parent_free(void) IStorage_Release(stg); - r = DeleteFileA(filenameA); - ok( r == TRUE, "deleted file\n"); + ret = DeleteFileA(filenameA); + ok(ret, "deleted file\n"); } static void test_nonroot_transacted(void) @@ -1833,6 +1839,7 @@ static void test_nonroot_transacted(void) static const WCHAR stgname[] = { 'P','E','R','M','S','T','G',0 }; static const WCHAR stmname[] = { 'C','O','N','T','E','N','T','S',0 }; static const WCHAR stmname2[] = { 'F','O','O',0 }; + BOOL ret; DeleteFileA(filenameA); @@ -1941,8 +1948,8 @@ static void test_nonroot_transacted(void) IStorage_Release(stg); - r = DeleteFileA(filenameA); - ok( r == TRUE, "deleted file\n"); + ret = DeleteFileA(filenameA); + ok(ret, "deleted file\n"); } static void test_ReadClassStm(void) @@ -2925,6 +2932,7 @@ static void test_rename(void) static const WCHAR stgname2[] = { 'S','T','G',0 }; static const WCHAR stmname[] = { 'C','O','N','T','E','N','T','S',0 }; static const WCHAR stmname2[] = { 'E','N','T','S',0 }; + BOOL ret; DeleteFileA(filenameA); @@ -2981,8 +2989,8 @@ static void test_rename(void) IStorage_Release(stg); - r = DeleteFileA(filenameA); - ok( r == TRUE, "deleted file\n"); + ret = DeleteFileA(filenameA); + ok(ret, "deleted file\n"); } static void test_toplevel_stat(void) @@ -3068,6 +3076,7 @@ static void test_substorage_enum(void) HRESULT r; ULONG ref; static const WCHAR stgname[] = { 'P','E','R','M','S','T','G',0 }; + BOOL ret; DeleteFileA(filenameA); @@ -3103,8 +3112,8 @@ static void test_substorage_enum(void) IStorage_Release(stg); - r = DeleteFileA(filenameA); - ok( r == TRUE, "deleted file\n"); + ret = DeleteFileA(filenameA); + ok(ret, "deleted file\n"); } static void test_copyto_locking(void) @@ -3115,6 +3124,7 @@ static void test_copyto_locking(void) static const WCHAR stgname[] = { 'S','T','G','1',0 }; static const WCHAR stgname2[] = { 'S','T','G','2',0 }; static const WCHAR stmname[] = { 'C','O','N','T','E','N','T','S',0 }; + BOOL ret; DeleteFileA(filenameA); @@ -3154,8 +3164,8 @@ static void test_copyto_locking(void) IStorage_Release(stg2); IStorage_Release(stg); - r = DeleteFileA(filenameA); - ok( r == TRUE, "deleted file\n"); + ret = DeleteFileA(filenameA); + ok(ret, "deleted file\n"); } static void test_copyto_recursive(void) @@ -3164,6 +3174,7 @@ static void test_copyto_recursive(void) HRESULT r; static const WCHAR stgname[] = { 'S','T','G','1',0 }; static const WCHAR stgname2[] = { 'S','T','G','2',0 }; + BOOL ret; DeleteFileA(filenameA); @@ -3205,8 +3216,8 @@ static void test_copyto_recursive(void) IStorage_Release(stg2); IStorage_Release(stg); - r = DeleteFileA(filenameA); - ok( r == TRUE, "deleted file\n"); + ret = DeleteFileA(filenameA); + ok(ret, "deleted file\n"); } static void test_hglobal_storage_creation(void)

1 0

Nikolay Sivov : comctl32/syslink: Simplify conditional expression ( PVS-Studio).
by Alexandre Julliard 20 Oct '15

20 Oct '15

Module: wine Branch: master Commit: e0c421ff62add33b3e570d19144773740d4596b5 URL: http://source.winehq.org/git/wine.git/?a=commit;h=e0c421ff62add33b3e570d191… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Tue Oct 20 14:03:51 2015 +0300 comctl32/syslink: Simplify conditional expression (PVS-Studio). Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/comctl32/syslink.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dlls/comctl32/syslink.c b/dlls/comctl32/syslink.c index eeb46b9..233169c 100644 --- a/dlls/comctl32/syslink.c +++ b/dlls/comctl32/syslink.c @@ -1387,7 +1387,7 @@ static PDOC_ITEM SYSLINK_LinkAtPt (const SYSLINK_INFO *infoPtr, const POINT *pt, for(Current = infoPtr->Items; Current != NULL; Current = Current->Next) { if((Current->Type == slLink) && SYSLINK_PtInDocItem(Current, *pt) && - (!MustBeEnabled || (MustBeEnabled && (Current->u.Link.state & LIS_ENABLED)))) + (!MustBeEnabled || (Current->u.Link.state & LIS_ENABLED))) { if(LinkId != NULL) {

1 0

Nikolay Sivov : shell32: Remove another recurring return code check ( PVS-Studio).
by Alexandre Julliard 20 Oct '15

20 Oct '15

Module: wine Branch: master Commit: 95ecc29a50ae47107f59b35ce9d4702f620b4bb3 URL: http://source.winehq.org/git/wine.git/?a=commit;h=95ecc29a50ae47107f59b35ce… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Tue Oct 20 13:59:37 2015 +0300 shell32: Remove another recurring return code check (PVS-Studio). Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/shell32/shellpath.c | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/dlls/shell32/shellpath.c b/dlls/shell32/shellpath.c index 9440563..cb79d6f 100644 --- a/dlls/shell32/shellpath.c +++ b/dlls/shell32/shellpath.c @@ -5589,10 +5589,7 @@ static HRESULT register_folder(const KNOWNFOLDERID *rfid, const KNOWNFOLDER_DEFI } if(SUCCEEDED(hr) && pKFD->category != KF_CATEGORY_VIRTUAL && pKFD->pszRelativePath) - { - if(SUCCEEDED(hr)) - hr = HRESULT_FROM_WIN32(RegSetValueExW(hKey, szRelativePath, 0, REG_SZ, (LPBYTE)pKFD->pszRelativePath, (lstrlenW(pKFD->pszRelativePath)+1)*sizeof(WCHAR) )); - } + hr = HRESULT_FROM_WIN32(RegSetValueExW(hKey, szRelativePath, 0, REG_SZ, (LPBYTE)pKFD->pszRelativePath, (lstrlenW(pKFD->pszRelativePath)+1)*sizeof(WCHAR) )); RegCloseKey(hKey);

1 0

Dmitry Timoshkov : gdi32: Fix heap corruption.
by Alexandre Julliard 20 Oct '15

20 Oct '15

Module: wine Branch: master Commit: d774442e8d65dc8ef1ad3d9a677fd0da623ff174 URL: http://source.winehq.org/git/wine.git/?a=commit;h=d774442e8d65dc8ef1ad3d9a6… Author: Dmitry Timoshkov <dmitry(a)baikal.ru> Date: Tue Oct 20 12:25:54 2015 +0800 gdi32: Fix heap corruption. Signed-off-by: Dmitry Timoshkov <dmitry(a)baikal.ru> Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/gdi32/freetype.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dlls/gdi32/freetype.c b/dlls/gdi32/freetype.c index 95df22a..9f2a9fb 100644 --- a/dlls/gdi32/freetype.c +++ b/dlls/gdi32/freetype.c @@ -5189,7 +5189,7 @@ static void fill_fileinfo_from_face( GdiFont *font, Face *face ) strcpyW(font->fileinfo->path, face->file); } else - memset(&font->fileinfo, 0, sizeof(*font->fileinfo) + len * sizeof(WCHAR)); + memset(font->fileinfo, 0, sizeof(*font->fileinfo) + len * sizeof(WCHAR)); } /*************************************************************

1 0

Dmitry Timoshkov : ole32: PropVariantCopy should return DISP_E_BADVARTYPE in case of bad source variant type.
by Alexandre Julliard 20 Oct '15

20 Oct '15

Module: wine Branch: master Commit: f9bc636d357221dfc036aacf84858257f045a1fe URL: http://source.winehq.org/git/wine.git/?a=commit;h=f9bc636d357221dfc036aacf8… Author: Dmitry Timoshkov <dmitry(a)baikal.ru> Date: Mon Oct 19 10:28:10 2015 +0800 ole32: PropVariantCopy should return DISP_E_BADVARTYPE in case of bad source variant type. In some rare (notably for some VT_VECTOR) cases Windows implementation returns STG_E_INVALIDPARAMETER, but returning DISP_E_BADVARTYPE should cover most of cases. Signed-off-by: Dmitry Timoshkov <dmitry(a)baikal.ru> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/ole32/ole2.c | 2 +- dlls/ole32/tests/propvariant.c | 25 +++++++++++++++---------- 2 files changed, 16 insertions(+), 11 deletions(-) diff --git a/dlls/ole32/ole2.c b/dlls/ole32/ole2.c index 09cbccf..40f7238 100644 --- a/dlls/ole32/ole2.c +++ b/dlls/ole32/ole2.c @@ -3041,7 +3041,7 @@ HRESULT WINAPI PropVariantCopy(PROPVARIANT *pvarDest, /* [out] */ hr = PROPVARIANT_ValidateType(pvarSrc->vt); if (FAILED(hr)) - return hr; + return DISP_E_BADVARTYPE; /* this will deal with most cases */ *pvarDest = *pvarSrc; diff --git a/dlls/ole32/tests/propvariant.c b/dlls/ole32/tests/propvariant.c index c45ca2a..0786a8e 100644 --- a/dlls/ole32/tests/propvariant.c +++ b/dlls/ole32/tests/propvariant.c @@ -130,7 +130,7 @@ static const char* wine_vtypes[VT_CLSID+1] = }; -static void expect(HRESULT hr, VARTYPE vt, BOOL copy) +static void expect(HRESULT hr, VARTYPE vt, BOOL copy, int line) { int idx = vt & VT_TYPEMASK; BYTE flags; @@ -158,7 +158,12 @@ static void expect(HRESULT hr, VARTYPE vt, BOOL copy) } if(flags == PROP_INV) - ok(hr == copy ? DISP_E_BADVARTYPE : STG_E_INVALIDPARAMETER, "%s (%s): got %08x\n", wine_vtypes[idx], modifier, hr); + { + if (copy && (vt & VT_VECTOR)) + ok(hr == DISP_E_BADVARTYPE || hr == STG_E_INVALIDPARAMETER, "%s (%s): got %08x (line %d)\n", wine_vtypes[idx], modifier, hr, line); + else + ok(hr == (copy ? DISP_E_BADVARTYPE : STG_E_INVALIDPARAMETER), "%s (%s): got %08x (line %d)\n", wine_vtypes[idx], modifier, hr, line); + } else if(flags == PROP_V0) ok(hr == S_OK, "%s (%s): got %08x\n", wine_vtypes[idx], modifier, hr); else if(flags & PROP_TODO) @@ -210,7 +215,7 @@ static void test_validtypes(void) vt = propvar.vt = i; memset(&copy, 0x77, sizeof(copy)); hr = PropVariantCopy(&copy, &propvar); - expect(hr, vt, TRUE); + expect(hr, vt, TRUE, __LINE__); if (hr == S_OK) { ok(copy.vt == propvar.vt, "expected %d, got %d\n", propvar.vt, copy.vt); @@ -224,7 +229,7 @@ static void test_validtypes(void) ok(!ret || broken(ret) /* win2000 */, "%d: copy should stay unchanged\n", i); } hr = PropVariantClear(&propvar); - expect(hr, vt, FALSE); + expect(hr, vt, FALSE, __LINE__); ok(propvar.vt == 0, "expected 0, got %d\n", propvar.vt); ok(U(propvar).uhVal.QuadPart == 0, "%u: expected 0, got %#x/%#x\n", i, U(propvar).uhVal.u.LowPart, U(propvar).uhVal.u.HighPart); @@ -234,7 +239,7 @@ static void test_validtypes(void) vt = propvar.vt = i | VT_ARRAY; memset(&copy, 0x77, sizeof(copy)); hr = PropVariantCopy(&copy, &propvar); - expect(hr, vt, TRUE); + expect(hr, vt, TRUE, __LINE__); if (hr == S_OK) { ok(copy.vt == propvar.vt, "expected %d, got %d\n", propvar.vt, copy.vt); @@ -247,7 +252,7 @@ static void test_validtypes(void) ok(!ret || broken(ret) /* win2000 */, "%d: copy should stay unchanged\n", i); } hr = PropVariantClear(&propvar); - expect(hr, vt, FALSE); + expect(hr, vt, FALSE, __LINE__); ok(propvar.vt == 0, "expected 0, got %d\n", propvar.vt); ok(U(propvar).uhVal.QuadPart == 0, "%u: expected 0, got %#x/%#x\n", i, U(propvar).uhVal.u.LowPart, U(propvar).uhVal.u.HighPart); @@ -258,7 +263,7 @@ static void test_validtypes(void) vt = propvar.vt = i | VT_VECTOR; memset(&copy, 0x77, sizeof(copy)); hr = PropVariantCopy(&copy, &propvar); - expect(hr, vt, TRUE); + expect(hr, vt, TRUE, __LINE__); if (hr == S_OK) { ok(copy.vt == propvar.vt, "expected %d, got %d\n", propvar.vt, copy.vt); @@ -271,7 +276,7 @@ static void test_validtypes(void) ok(!ret || broken(ret) /* win2000 */, "%d: copy should stay unchanged\n", i); } hr = PropVariantClear(&propvar); - expect(hr, vt, FALSE); + expect(hr, vt, FALSE, __LINE__); ok(propvar.vt == 0, "expected 0, got %d\n", propvar.vt); ok(U(propvar).uhVal.QuadPart == 0, "%u: expected 0, got %#x/%#x\n", i, U(propvar).uhVal.u.LowPart, U(propvar).uhVal.u.HighPart); @@ -281,7 +286,7 @@ static void test_validtypes(void) vt = propvar.vt = i | VT_BYREF; memset(&copy, 0x77, sizeof(copy)); hr = PropVariantCopy(&copy, &propvar); - expect(hr, vt, TRUE); + expect(hr, vt, TRUE, __LINE__); if (hr == S_OK) { ok(copy.vt == propvar.vt, "expected %d, got %d\n", propvar.vt, copy.vt); @@ -295,7 +300,7 @@ static void test_validtypes(void) ok(!ret || broken(ret) /* win2000 */, "%d: copy should stay unchanged\n", i); } hr = PropVariantClear(&propvar); - expect(hr, vt, FALSE); + expect(hr, vt, FALSE, __LINE__); ok(propvar.vt == 0, "expected 0, got %d\n", propvar.vt); ok(U(propvar).uhVal.QuadPart == 0, "%u: expected 0, got %#x/%#x\n", i, U(propvar).uhVal.u.LowPart, U(propvar).uhVal.u.HighPart);

1 0

← Newer
1
...
23
24
25
26
27
28
29
...
75
Older →

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

2003

2002

2001

wine-commits October 2015