[PATCH 5/5] user32: Pass num_steps=0 to alloc_icon_handle if is_ani=FALSE.

Alex Henrie alexhenrie24 at gmail.com
Sun Oct 4 22:27:57 CDT 2015


Cc: Erich Hoover <ehoover at mines.edu>

num_steps is ignored if is_ani is false, and 0 makes more sense as a
placeholder than 1.

Identified by manual inspection.

See also: fa4f9c43bcc06619fcf3e71f9e0a767981f2a750

Signed-off-by: Alex Henrie <alexhenrie24 at gmail.com>
---
 dlls/user32/cursoricon.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/dlls/user32/cursoricon.c b/dlls/user32/cursoricon.c
index 6a05822..ea3f2ef 100644
--- a/dlls/user32/cursoricon.c
+++ b/dlls/user32/cursoricon.c
@@ -939,7 +939,7 @@ done:
     HeapFree( GetProcessHeap(), 0, bmi_copy );
 
     if (ret)
-        hObj = alloc_icon_handle( FALSE, 1 );
+        hObj = alloc_icon_handle( FALSE, 0 );
     if (hObj)
     {
         struct cursoricon_object *info = get_icon_ptr( hObj );
@@ -1575,7 +1575,7 @@ HICON WINAPI CopyIcon( HICON hIcon )
         SetLastError( ERROR_INVALID_CURSOR_HANDLE );
         return 0;
     }
-    if ((hNew = alloc_icon_handle( FALSE, 1 )))
+    if ((hNew = alloc_icon_handle( FALSE, 0 )))
     {
         struct cursoricon_frame *frameOld, *frameNew;
 
@@ -2181,7 +2181,7 @@ HICON WINAPI CreateIconIndirect(PICONINFO iconinfo)
 
     DeleteDC( hdc );
 
-    hObj = alloc_icon_handle( FALSE, 1 );
+    hObj = alloc_icon_handle( FALSE, 0 );
     if (hObj)
     {
         struct cursoricon_object *info = get_icon_ptr( hObj );
-- 
2.6.0




More information about the wine-patches mailing list