[4/8] scrrun: IEnumVARIANT::Skip shouldn't skip anything if asked number of folder elements is 0.

Dmitry Timoshkov dmitry at baikal.ru
Mon Jan 27 20:35:56 CST 2014


---
 dlls/scrrun/filesystem.c       | 2 ++
 dlls/scrrun/tests/filesystem.c | 1 -
 2 files changed, 2 insertions(+), 1 deletion(-)

diff --git a/dlls/scrrun/filesystem.c b/dlls/scrrun/filesystem.c
index 7d09a82..0292ab8 100644
--- a/dlls/scrrun/filesystem.c
+++ b/dlls/scrrun/filesystem.c
@@ -557,6 +557,8 @@ static HRESULT WINAPI foldercoll_enumvariant_Skip(IEnumVARIANT *iface, ULONG cel
 
     TRACE("(%p)->(%d)\n", This, celt);
 
+    if (!celt) return S_OK;
+
     if (!handle)
     {
         handle = start_enumeration(This->data.u.foldercoll.coll->path, &data);
diff --git a/dlls/scrrun/tests/filesystem.c b/dlls/scrrun/tests/filesystem.c
index d6e094a..b1de309 100644
--- a/dlls/scrrun/tests/filesystem.c
+++ b/dlls/scrrun/tests/filesystem.c
@@ -931,7 +931,6 @@ todo_wine
     hr = IEnumVARIANT_Skip(enumvar, 2);
     ok(hr == S_OK, "got 0x%08x\n", hr);
     hr = IEnumVARIANT_Skip(enumvar, 0);
-todo_wine
     ok(hr == S_OK, "got 0x%08x\n", hr);
 
     VariantInit(&var2[0]);
-- 
1.8.5.3




More information about the wine-patches mailing list