Peter Oberndorfer : comctl32: EnumMRUListA/ W fix getting list size by negative item pos.

[Alexandre Julliard]

Module: wine
Branch: master
Commit: 429d0d3eb330562648f5d0b609101e5252d0ddbc
URL:    http://source.winehq.org/git/wine.git/?a=commit;h=429d0d3eb330562648f5d0b609...

Author: Peter Oberndorfer 
Date:   Wed Dec  5 20:10:55 2007 +0100

comctl32: EnumMRUListA/W fix getting list size by negative item pos.

---

dlls/comctl32/comctl32undoc.c |    4 ++--
 dlls/comctl32/tests/mru.c     |    2 --
 2 files changed, 2 insertions(+), 4 deletions(-)

diff --git a/dlls/comctl32/comctl32undoc.c b/dlls/comctl32/comctl32undoc.c
index 6f434a6..8ada782 100644
--- a/dlls/comctl32/comctl32undoc.c
+++ b/dlls/comctl32/comctl32undoc.c
@@ -834,8 +834,8 @@ INT WINAPI EnumMRUListW (HANDLE hList, INT nItemPos, LPVOID lpBuffer,
     const WINEMRUITEM *witem;
     INT desired, datasize;

-    if (nItemPos >= mp->cursize) return -1;
     if ((nItemPos < 0) || !lpBuffer) return mp->cursize;
+    if (nItemPos >= mp->cursize) return -1;
     desired = mp->realMRU[nItemPos];
     desired -= 'a';
     TRACE("nItemPos=%d, desired=%d\n", nItemPos, desired);
@@ -860,8 +860,8 @@ INT WINAPI EnumMRUListA (HANDLE hList, INT nItemPos, LPVOID lpBuffer,
     INT desired, datasize;
     DWORD lenA;

-    if (nItemPos >= mp->cursize) return -1;
     if ((nItemPos < 0) || !lpBuffer) return mp->cursize;
+    if (nItemPos >= mp->cursize) return -1;
     desired = mp->realMRU[nItemPos];
     desired -= 'a';
     TRACE("nItemPos=%d, desired=%d\n", nItemPos, desired);
diff --git a/dlls/comctl32/tests/mru.c b/dlls/comctl32/tests/mru.c
index bfcc385..5201a95 100644
--- a/dlls/comctl32/tests/mru.c
+++ b/dlls/comctl32/tests/mru.c
@@ -288,7 +288,6 @@ static void test_MRUListA(void)
         iRet = pEnumMRUList(hMRU, 0, NULL, 0);
         ok(iRet == 3, "EnumMRUList expected %d, got %d\n", LIST_SIZE, iRet);

-        todo_wine{
         /* negative item pos = get list size */
         iRet = pEnumMRUList(hMRU, -1, NULL, 0);
         ok(iRet == 3, "EnumMRUList expected %d, got %d\n", LIST_SIZE, iRet);
@@ -304,7 +303,6 @@ static void test_MRUListA(void)
         /* negative item pos = get list size */
         iRet = pEnumMRUList(hMRU, -5, buffer, 255);
         ok(iRet == 3, "EnumMRUList expected %d, got %d\n", LIST_SIZE, iRet);
-        }

/* check entry 0 */
         buffer[0] = 0;