March 2015 - Wine-commits - list.winehq.org

Francois Gouget : riched20: Remove unneeded NONAMELESSXXX directives.
by Alexandre Julliard 19 Mar '15

19 Mar '15

Module: wine Branch: master Commit: cd0200113c9f9c3556645599945c449933c6207f URL: http://source.winehq.org/git/wine.git/?a=commit;h=cd0200113c9f9c35566455999… Author: Francois Gouget <fgouget(a)free.fr> Date: Wed Mar 18 18:59:06 2015 +0100 riched20: Remove unneeded NONAMELESSXXX directives. --- dlls/riched20/clipboard.c | 2 ++ dlls/riched20/editor.c | 2 ++ dlls/riched20/editstr.h | 2 -- dlls/riched20/richole.c | 1 - dlls/riched20/txthost.c | 2 -- dlls/riched20/txtsrv.c | 2 -- 6 files changed, 4 insertions(+), 7 deletions(-) diff --git a/dlls/riched20/clipboard.c b/dlls/riched20/clipboard.c index 250a233..c536665 100644 --- a/dlls/riched20/clipboard.c +++ b/dlls/riched20/clipboard.c @@ -18,6 +18,8 @@ * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA */ +#define NONAMELESSUNION + #include "editor.h" WINE_DEFAULT_DEBUG_CHANNEL(richedit); diff --git a/dlls/riched20/editor.c b/dlls/riched20/editor.c index 4da2369..146e4be 100644 --- a/dlls/riched20/editor.c +++ b/dlls/riched20/editor.c @@ -224,6 +224,8 @@ * */ +#define NONAMELESSUNION + #include "editor.h" #include "commdlg.h" #include "winreg.h" diff --git a/dlls/riched20/editstr.h b/dlls/riched20/editstr.h index 4fd965c..1b1eda6 100644 --- a/dlls/riched20/editstr.h +++ b/dlls/riched20/editstr.h @@ -32,8 +32,6 @@ #include <limits.h> #define COBJMACROS -#define NONAMELESSUNION -#define NONAMELESSSTRUCT #include <windef.h> #include <winbase.h> diff --git a/dlls/riched20/richole.c b/dlls/riched20/richole.c index c62998a..db55e84 100644 --- a/dlls/riched20/richole.c +++ b/dlls/riched20/richole.c @@ -22,7 +22,6 @@ #include <stdarg.h> #define NONAMELESSUNION -#define NONAMELESSSTRUCT #define COBJMACROS #include "windef.h" diff --git a/dlls/riched20/txthost.c b/dlls/riched20/txthost.c index 4b72d8b..af32c1a 100644 --- a/dlls/riched20/txthost.c +++ b/dlls/riched20/txthost.c @@ -21,8 +21,6 @@ #include "config.h" #include "wine/port.h" -#define NONAMELESSSTRUCT -#define NONAMELESSUNION #define COBJMACROS #include "editor.h" diff --git a/dlls/riched20/txtsrv.c b/dlls/riched20/txtsrv.c index 75bee6d..e9c1061 100644 --- a/dlls/riched20/txtsrv.c +++ b/dlls/riched20/txtsrv.c @@ -21,8 +21,6 @@ #include "config.h" #include "wine/port.h" -#define NONAMELESSSTRUCT -#define NONAMELESSUNION #define COBJMACROS #include "editor.h"

1 0

Francois Gouget : sane.ds: Remove unneeded NONAMELESSXXX directives.
by Alexandre Julliard 19 Mar '15

19 Mar '15

Module: wine Branch: master Commit: 305f63a22b1492cd6121693cae04090817724a69 URL: http://source.winehq.org/git/wine.git/?a=commit;h=305f63a22b1492cd6121693ca… Author: Francois Gouget <fgouget(a)free.fr> Date: Wed Mar 18 19:01:07 2015 +0100 sane.ds: Remove unneeded NONAMELESSXXX directives. --- dlls/sane.ds/capability.c | 3 --- dlls/sane.ds/ui.c | 2 +- 2 files changed, 1 insertion(+), 4 deletions(-) diff --git a/dlls/sane.ds/capability.c b/dlls/sane.ds/capability.c index 7bb64b3..7d3376f 100644 --- a/dlls/sane.ds/capability.c +++ b/dlls/sane.ds/capability.c @@ -16,9 +16,6 @@ * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA */ -#define NONAMELESSUNION -#define NONAMELESSSTRUCT - #include "config.h" #include <stdarg.h> diff --git a/dlls/sane.ds/ui.c b/dlls/sane.ds/ui.c index 07d5a59..5574c5a 100644 --- a/dlls/sane.ds/ui.c +++ b/dlls/sane.ds/ui.c @@ -25,7 +25,7 @@ #include <stdio.h> #define NONAMELESSUNION -#define NONAMELESSSTRUCT + #include "sane_i.h" #include "winuser.h" #include "winnls.h"

1 0

Francois Gouget : scsiport.sys: Remove unneeded NONAMELESSXXX directives.
by Alexandre Julliard 19 Mar '15

19 Mar '15

Module: wine Branch: master Commit: e0483c2355d770d108841a3a5feba80f8b3b17d9 URL: http://source.winehq.org/git/wine.git/?a=commit;h=e0483c2355d770d108841a3a5… Author: Francois Gouget <fgouget(a)free.fr> Date: Wed Mar 18 19:01:14 2015 +0100 scsiport.sys: Remove unneeded NONAMELESSXXX directives. --- dlls/scsiport.sys/scsiport.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/dlls/scsiport.sys/scsiport.c b/dlls/scsiport.sys/scsiport.c index f689153..baafaec 100644 --- a/dlls/scsiport.sys/scsiport.c +++ b/dlls/scsiport.sys/scsiport.c @@ -18,9 +18,6 @@ #include <stdarg.h> -#define NONAMELESSUNION -#define NONAMELESSSTRUCT - #include "ntstatus.h" #define WIN32_NO_STATUS #include "windef.h"

1 0

Francois Gouget : setupapi: Remove unneeded NONAMELESSXXX directives.
by Alexandre Julliard 19 Mar '15

19 Mar '15

Module: wine Branch: master Commit: 16019848b8cbecc00409f9230212d0f71aab5ecd URL: http://source.winehq.org/git/wine.git/?a=commit;h=16019848b8cbecc00409f9230… Author: Francois Gouget <fgouget(a)free.fr> Date: Wed Mar 18 19:01:26 2015 +0100 setupapi: Remove unneeded NONAMELESSXXX directives. --- dlls/setupapi/fakedll.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/dlls/setupapi/fakedll.c b/dlls/setupapi/fakedll.c index b543ade..fdd9bea 100644 --- a/dlls/setupapi/fakedll.c +++ b/dlls/setupapi/fakedll.c @@ -35,8 +35,7 @@ #define COBJMACROS #define ATL_INITGUID -#define NONAMELESSSTRUCT -#define NONAMELESSUNION + #include "ntstatus.h" #define WIN32_NO_STATUS #include "windef.h"

1 0

Francois Gouget : rpcrt4: Remove an unneeded NONAMELESSSTRUCT directive.
by Alexandre Julliard 19 Mar '15

19 Mar '15

Module: wine Branch: master Commit: 13ad265b23db14151b60c6e056312c32e4c94714 URL: http://source.winehq.org/git/wine.git/?a=commit;h=13ad265b23db14151b60c6e05… Author: Francois Gouget <fgouget(a)free.fr> Date: Wed Mar 18 19:01:32 2015 +0100 rpcrt4: Remove an unneeded NONAMELESSSTRUCT directive. --- dlls/rpcrt4/ndr_ole.c | 1 - 1 file changed, 1 deletion(-) diff --git a/dlls/rpcrt4/ndr_ole.c b/dlls/rpcrt4/ndr_ole.c index 143bba7..0f0b7e7 100644 --- a/dlls/rpcrt4/ndr_ole.c +++ b/dlls/rpcrt4/ndr_ole.c @@ -27,7 +27,6 @@ #define COBJMACROS #define NONAMELESSUNION -#define NONAMELESSSTRUCT #include "windef.h" #include "winbase.h"

1 0

Francois Gouget : krnl386.exe16: Mark internal symbols with hidden visibility.
by Alexandre Julliard 19 Mar '15

19 Mar '15

Module: wine Branch: master Commit: 40d213a297031f9ad546781f33665fea88a2338e URL: http://source.winehq.org/git/wine.git/?a=commit;h=40d213a297031f9ad546781f3… Author: Francois Gouget <fgouget(a)free.fr> Date: Wed Mar 18 19:05:17 2015 +0100 krnl386.exe16: Mark internal symbols with hidden visibility. --- dlls/krnl386.exe16/dosexe.h | 178 +++++++++++++++++----------------- dlls/krnl386.exe16/kernel16_private.h | 111 ++++++++++----------- dlls/krnl386.exe16/vga.h | 62 ++++++------ dlls/krnl386.exe16/wowthunk.c | 2 - 4 files changed, 176 insertions(+), 177 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=40d213a297031f9ad5467…

1 0

Francois Gouget : mmsystem.dll16: Mark internal symbols with hidden visibility.
by Alexandre Julliard 19 Mar '15

19 Mar '15

Module: wine Branch: master Commit: 40d806dc95cc9e6c5921863b3d6ae69f7445068c URL: http://source.winehq.org/git/wine.git/?a=commit;h=40d806dc95cc9e6c5921863b3… Author: Francois Gouget <fgouget(a)free.fr> Date: Wed Mar 18 19:05:47 2015 +0100 mmsystem.dll16: Mark internal symbols with hidden visibility. --- dlls/mmsystem.dll16/winemm16.h | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/dlls/mmsystem.dll16/winemm16.h b/dlls/mmsystem.dll16/winemm16.h index e76ee34..3d8e566 100644 --- a/dlls/mmsystem.dll16/winemm16.h +++ b/dlls/mmsystem.dll16/winemm16.h @@ -24,7 +24,7 @@ /* mmsystem (16 bit files) only functions */ void MMDRV_Init16(void); -void MMSYSTEM_MMTIME32to16(LPMMTIME16 mmt16, const MMTIME* mmt32); +void MMSYSTEM_MMTIME32to16(LPMMTIME16 mmt16, const MMTIME* mmt32) DECLSPEC_HIDDEN; typedef LONG (*MCIPROC16)(DWORD, HDRVR16, WORD, DWORD, DWORD); @@ -61,7 +61,7 @@ typedef enum { MMSYSTEM_MAP_OKMEM, /* ok, some memory allocated, need to call UnMapMsg. to be sent to the proc. */ } MMSYSTEM_MapType; -extern CRITICAL_SECTION mmdrv_cs; +extern CRITICAL_SECTION mmdrv_cs DECLSPEC_HIDDEN; enum MMSYSTEM_DriverType { @@ -73,11 +73,11 @@ enum MMSYSTEM_DriverType MMSYSTDRV_MAX }; -extern struct mmsystdrv_thunk* MMSYSTDRV_AddThunk(DWORD callback, DWORD flags, enum MMSYSTEM_DriverType kind); -extern void MMSYSTDRV_DeleteThunk(struct mmsystdrv_thunk* thunk); -extern void MMSYSTDRV_SetHandle(struct mmsystdrv_thunk* thunk, void* h); -extern void MMSYSTDRV_CloseHandle(void* h); -extern DWORD MMSYSTDRV_Message(void* h, UINT msg, DWORD_PTR param1, DWORD_PTR param2); +extern struct mmsystdrv_thunk* MMSYSTDRV_AddThunk(DWORD callback, DWORD flags, enum MMSYSTEM_DriverType kind) DECLSPEC_HIDDEN; +extern void MMSYSTDRV_DeleteThunk(struct mmsystdrv_thunk* thunk) DECLSPEC_HIDDEN; +extern void MMSYSTDRV_SetHandle(struct mmsystdrv_thunk* thunk, void* h) DECLSPEC_HIDDEN; +extern void MMSYSTDRV_CloseHandle(void* h) DECLSPEC_HIDDEN; +extern DWORD MMSYSTDRV_Message(void* h, UINT msg, DWORD_PTR param1, DWORD_PTR param2) DECLSPEC_HIDDEN; #define WINE_MMTHREAD_CREATED 0x4153494C /* "BSIL" */ #define WINE_MMTHREAD_DELETED 0xDEADDEAD

1 0

Ken Thomases : winejoystick: Fix a crash on accessing a CFArray past its end due to an off-by-one error in the Mac code.
by Alexandre Julliard 19 Mar '15

19 Mar '15

Module: wine Branch: master Commit: 77432ac45db59440af1aa75e36c93cd025e4e5fc URL: http://source.winehq.org/git/wine.git/?a=commit;h=77432ac45db59440af1aa75e3… Author: Ken Thomases <ken(a)codeweavers.com> Date: Wed Mar 18 14:02:13 2015 -0500 winejoystick: Fix a crash on accessing a CFArray past its end due to an off-by-one error in the Mac code. --- dlls/winejoystick.drv/joystick_osx.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dlls/winejoystick.drv/joystick_osx.c b/dlls/winejoystick.drv/joystick_osx.c index c8850af..cb66c47 100644 --- a/dlls/winejoystick.drv/joystick_osx.c +++ b/dlls/winejoystick.drv/joystick_osx.c @@ -528,7 +528,7 @@ static BOOL open_joystick(joystick_t* joystick) } index = joystick - joysticks; - if (index > CFArrayGetCount(device_main_elements)) + if (index >= CFArrayGetCount(device_main_elements)) return FALSE; joystick->element = (IOHIDElementRef)CFArrayGetValueAtIndex(device_main_elements, index);

1 0

Nikolay Sivov : scrrun: Implement IEnumVARIANT support for dictionary.
by Alexandre Julliard 19 Mar '15

19 Mar '15

Module: wine Branch: master Commit: ba482174299ff165c49e4d7d226f9799a2d8c5f3 URL: http://source.winehq.org/git/wine.git/?a=commit;h=ba482174299ff165c49e4d7d2… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Wed Mar 18 15:11:32 2015 +0300 scrrun: Implement IEnumVARIANT support for dictionary. --- dlls/scrrun/dictionary.c | 78 ++++++++++++++++++++++++--- dlls/scrrun/tests/dictionary.c | 118 +++++++++++++++++++++++++++++++++++++++++ 2 files changed, 188 insertions(+), 8 deletions(-) diff --git a/dlls/scrrun/dictionary.c b/dlls/scrrun/dictionary.c index b74f2a7..5c770b6 100644 --- a/dlls/scrrun/dictionary.c +++ b/dlls/scrrun/dictionary.c @@ -71,6 +71,7 @@ typedef struct LONG count; struct list pairs; struct list buckets[BUCKET_COUNT]; + struct list notifier; } dictionary; struct dictionary_enum { @@ -78,6 +79,8 @@ struct dictionary_enum { LONG ref; dictionary *dict; + struct list *cur; + struct list notify; }; static inline dictionary *impl_from_IDictionary(IDictionary *iface) @@ -248,6 +251,7 @@ static ULONG WINAPI dict_enum_Release(IEnumVARIANT *iface) ref = InterlockedDecrement(&This->ref); if(ref == 0) { + list_remove(&This->notify); IDictionary_Release(&This->dict->IDictionary_iface); heap_free(This); } @@ -258,29 +262,67 @@ static ULONG WINAPI dict_enum_Release(IEnumVARIANT *iface) static HRESULT WINAPI dict_enum_Next(IEnumVARIANT *iface, ULONG count, VARIANT *keys, ULONG *fetched) { struct dictionary_enum *This = impl_from_IEnumVARIANT(iface); - FIXME("(%p)->(%u %p %p): stub\n", This, count, keys, fetched); - return E_NOTIMPL; + struct keyitem_pair *pair; + ULONG i = 0; + + TRACE("(%p)->(%u %p %p)\n", This, count, keys, fetched); + + if (fetched) + *fetched = 0; + + if (!count) + return S_OK; + + while (This->cur && i < count) { + pair = LIST_ENTRY(This->cur, struct keyitem_pair, entry); + VariantCopy(&keys[i], &pair->key); + This->cur = list_next(&This->dict->pairs, This->cur); + i++; + } + + if (fetched) + *fetched = i; + + return i < count ? S_FALSE : S_OK; } static HRESULT WINAPI dict_enum_Skip(IEnumVARIANT *iface, ULONG count) { struct dictionary_enum *This = impl_from_IEnumVARIANT(iface); - FIXME("(%p)->(%u): stub\n", This, count); - return E_NOTIMPL; + + TRACE("(%p)->(%u)\n", This, count); + + if (!count) + return S_OK; + + if (!This->cur) + return S_FALSE; + + while (count--) { + This->cur = list_next(&This->dict->pairs, This->cur); + if (!This->cur) break; + } + + return count == 0 ? S_OK : S_FALSE; } static HRESULT WINAPI dict_enum_Reset(IEnumVARIANT *iface) { struct dictionary_enum *This = impl_from_IEnumVARIANT(iface); - FIXME("(%p): stub\n", This); - return E_NOTIMPL; + + TRACE("(%p)\n", This); + + This->cur = list_head(&This->dict->pairs); + return S_OK; } +static HRESULT create_dict_enum(dictionary*, IUnknown**); + static HRESULT WINAPI dict_enum_Clone(IEnumVARIANT *iface, IEnumVARIANT **cloned) { struct dictionary_enum *This = impl_from_IEnumVARIANT(iface); - FIXME("(%p)->(%p): stub\n", This, cloned); - return E_NOTIMPL; + TRACE("(%p)->(%p)\n", This, cloned); + return create_dict_enum(This->dict, (IUnknown**)cloned); } static const IEnumVARIANTVtbl dictenumvtbl = { @@ -305,6 +347,8 @@ static HRESULT create_dict_enum(dictionary *dict, IUnknown **ret) This->IEnumVARIANT_iface.lpVtbl = &dictenumvtbl; This->ref = 1; + This->cur = list_head(&dict->pairs); + list_add_tail(&dict->notifier, &This->notify); This->dict = dict; IDictionary_AddRef(&dict->IDictionary_iface); @@ -312,6 +356,21 @@ static HRESULT create_dict_enum(dictionary *dict, IUnknown **ret) return S_OK; } +static void notify_remove_pair(struct list *notifier, struct list *pair) +{ + struct dictionary_enum *dict_enum; + struct list *cur; + + LIST_FOR_EACH(cur, notifier) { + dict_enum = LIST_ENTRY(cur, struct dictionary_enum, notify); + if (!pair) + dict_enum->cur = list_head(&dict_enum->dict->pairs); + else if (dict_enum->cur == pair) { + dict_enum->cur = list_next(&dict_enum->dict->pairs, dict_enum->cur); + } + } +} + static HRESULT WINAPI dictionary_QueryInterface(IDictionary *iface, REFIID riid, void **obj) { dictionary *This = impl_from_IDictionary(iface); @@ -616,6 +675,7 @@ static HRESULT WINAPI dictionary_Remove(IDictionary *iface, VARIANT *key) if (!(pair = get_keyitem_pair(This, key))) return CTL_E_ELEMENT_NOT_FOUND; + notify_remove_pair(&This->notifier, &pair->entry); list_remove(&pair->entry); list_remove(&pair->bucket); This->count--; @@ -634,6 +694,7 @@ static HRESULT WINAPI dictionary_RemoveAll(IDictionary *iface) if (This->count == 0) return S_OK; + notify_remove_pair(&This->notifier, NULL); LIST_FOR_EACH_ENTRY_SAFE(pair, pair2, &This->pairs, struct keyitem_pair, entry) { list_remove(&pair->entry); list_remove(&pair->bucket); @@ -796,6 +857,7 @@ HRESULT WINAPI Dictionary_CreateInstance(IClassFactory *factory,IUnknown *outer, This->method = BinaryCompare; This->count = 0; list_init(&This->pairs); + list_init(&This->notifier); memset(This->buckets, 0, sizeof(This->buckets)); *obj = &This->IDictionary_iface; diff --git a/dlls/scrrun/tests/dictionary.c b/dlls/scrrun/tests/dictionary.c index 927ac05..1869861 100644 --- a/dlls/scrrun/tests/dictionary.c +++ b/dlls/scrrun/tests/dictionary.c @@ -618,6 +618,123 @@ static void test_Add(void) IDictionary_Release(dict); } +static void test_IEnumVARIANT(void) +{ + IUnknown *enum1, *enum2; + IEnumVARIANT *enumvar; + VARIANT key, item; + IDictionary *dict; + ULONG fetched; + HRESULT hr; + + hr = CoCreateInstance(&CLSID_Dictionary, NULL, CLSCTX_INPROC_SERVER|CLSCTX_INPROC_HANDLER, + &IID_IDictionary, (void**)&dict); + ok(hr == S_OK, "got 0x%08x\n", hr); + +if (0) /* crashes on native */ + hr = IDictionary__NewEnum(dict, NULL); + + hr = IDictionary__NewEnum(dict, &enum1); + ok(hr == S_OK, "got 0x%08x\n", hr); + + hr = IDictionary__NewEnum(dict, &enum2); + ok(hr == S_OK, "got 0x%08x\n", hr); + ok(enum1 != enum2, "got %p, %p\n", enum2, enum1); + IUnknown_Release(enum2); + + hr = IUnknown_QueryInterface(enum1, &IID_IEnumVARIANT, (void**)&enumvar); + ok(hr == S_OK, "got 0x%08x\n", hr); + IUnknown_Release(enum1); + + /* dictionary is empty */ + hr = IEnumVARIANT_Skip(enumvar, 1); + ok(hr == S_FALSE, "got 0x%08x\n", hr); + + hr = IEnumVARIANT_Skip(enumvar, 0); + ok(hr == S_OK, "got 0x%08x\n", hr); + + V_VT(&key) = VT_I2; + V_I2(&key) = 1; + V_VT(&item) = VT_I4; + V_I4(&item) = 100; + hr = IDictionary_Add(dict, &key, &item); + ok(hr == S_OK, "got 0x%08x\n", hr); + + hr = IEnumVARIANT_Skip(enumvar, 0); + ok(hr == S_OK, "got 0x%08x\n", hr); + + hr = IEnumVARIANT_Reset(enumvar); + ok(hr == S_OK, "got 0x%08x\n", hr); + hr = IEnumVARIANT_Skip(enumvar, 1); + ok(hr == S_OK, "got 0x%08x\n", hr); + hr = IEnumVARIANT_Skip(enumvar, 1); + ok(hr == S_FALSE, "got 0x%08x\n", hr); + + V_VT(&key) = VT_I2; + V_I2(&key) = 4000; + V_VT(&item) = VT_I4; + V_I4(&item) = 200; + hr = IDictionary_Add(dict, &key, &item); + ok(hr == S_OK, "got 0x%08x\n", hr); + + V_VT(&key) = VT_I2; + V_I2(&key) = 0; + V_VT(&item) = VT_I4; + V_I4(&item) = 300; + hr = IDictionary_Add(dict, &key, &item); + ok(hr == S_OK, "got 0x%08x\n", hr); + + hr = IEnumVARIANT_Reset(enumvar); + ok(hr == S_OK, "got 0x%08x\n", hr); + + VariantInit(&key); + hr = IEnumVARIANT_Next(enumvar, 1, &key, &fetched); + ok(hr == S_OK, "got 0x%08x\n", hr); + ok(V_VT(&key) == VT_I2, "got %d\n", V_VT(&key)); + ok(V_I2(&key) == 1, "got %d\n", V_I2(&key)); + ok(fetched == 1, "got %u\n", fetched); + + hr = IEnumVARIANT_Reset(enumvar); + ok(hr == S_OK, "got 0x%08x\n", hr); + + hr = IDictionary_Remove(dict, &key); + ok(hr == S_OK, "got 0x%08x\n", hr); + + VariantInit(&key); + hr = IEnumVARIANT_Next(enumvar, 1, &key, &fetched); + ok(hr == S_OK, "got 0x%08x\n", hr); + ok(V_VT(&key) == VT_I2, "got %d\n", V_VT(&key)); + ok(V_I2(&key) == 4000, "got %d\n", V_I2(&key)); + ok(fetched == 1, "got %u\n", fetched); + + VariantInit(&key); + hr = IEnumVARIANT_Next(enumvar, 1, &key, &fetched); + ok(hr == S_OK, "got 0x%08x\n", hr); + ok(V_VT(&key) == VT_I2, "got %d\n", V_VT(&key)); + ok(V_I2(&key) == 0, "got %d\n", V_I2(&key)); + ok(fetched == 1, "got %u\n", fetched); + + /* enumeration reached the bottom, add one more pair */ + VariantInit(&key); + hr = IEnumVARIANT_Next(enumvar, 1, &key, &fetched); + ok(hr == S_FALSE, "got 0x%08x\n", hr); + + V_VT(&key) = VT_I2; + V_I2(&key) = 13; + V_VT(&item) = VT_I4; + V_I4(&item) = 350; + hr = IDictionary_Add(dict, &key, &item); + ok(hr == S_OK, "got 0x%08x\n", hr); + + /* still doesn't work until Reset() */ + VariantInit(&key); + hr = IEnumVARIANT_Next(enumvar, 1, &key, &fetched); + ok(hr == S_FALSE, "got 0x%08x\n", hr); + + IEnumVARIANT_Release(enumvar); + IDictionary_Release(dict); +} + START_TEST(dictionary) { IDispatch *disp; @@ -642,6 +759,7 @@ START_TEST(dictionary) test_Remove(); test_Item(); test_Add(); + test_IEnumVARIANT(); CoUninitialize(); }

1 0

Nikolay Sivov : wmvcore: Return S_OK from WMCreateReader.
by Alexandre Julliard 19 Mar '15

19 Mar '15

Module: wine Branch: master Commit: 069406fe94a41752b4f739b24049b395e3c5a7b3 URL: http://source.winehq.org/git/wine.git/?a=commit;h=069406fe94a41752b4f739b24… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Wed Mar 18 22:52:55 2015 +0300 wmvcore: Return S_OK from WMCreateReader. --- dlls/wmvcore/wmvcore_main.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dlls/wmvcore/wmvcore_main.c b/dlls/wmvcore/wmvcore_main.c index 4698e62..879c156 100644 --- a/dlls/wmvcore/wmvcore_main.c +++ b/dlls/wmvcore/wmvcore_main.c @@ -217,7 +217,7 @@ HRESULT WINAPI WMCreateReader(IUnknown *reserved, DWORD rights, IWMReader **ret_ reader->ref = 1; *ret_reader = &reader->IWMReader_iface; - return E_NOTIMPL; + return S_OK; } HRESULT WINAPI WMCreateSyncReader(IUnknown *pcert, DWORD rights, IWMSyncReader **syncreader)

1 0