winemac: Fix the logic for checking if a view is already in the intended z-order.

-[NSView subviews] returns the views in back-to-front order, not front-to-back as I had thought. Signed-off-by: Ken Thomases <ken@codeweavers.com> Signed-off-by: Alexandre Julliard <julliard@winehq.org>

winemac: Fix the logic for checking if a view is already in the intended z-order.
102cd6dc · Ken Thomases · Alexandre Julliard · bdbb3514 · 102cd6dc
Commit 102cd6dc authored Jun 03, 2016 by Ken Thomases Committed by Alexandre Julliard Jun 03, 2016
Show whitespace changes
Inline Side-by-side

Showing with 3 additions and 3 deletions

cocoa_window.m dlls/winemac.drv/cocoa_window.m +3 -3

No files found.
--- a/dlls/winemac.drv/cocoa_window.m
+++ b/dlls/winemac.drv/cocoa_window.m
@@ -3343,11 +3343,11 @@ void macdrv_set_view_superview(macdrv_view v, macdrv_view s, macdrv_window w, ma
        {
            NSArray* subviews = [superview subviews];
            NSUInteger index = [subviews indexOfObjectIdenticalTo:view];
-            if (!prev && !next && index == 0)
+            if (!prev && !next && index == [subviews count] - 1)
                return;
-            if (prev && index > 0 && [subviews objectAtIndex:index - 1] == prev)
+            if (prev && index + 1 < [subviews count] && [subviews objectAtIndex:index + 1] == prev)
                return;
-            if (!prev && next && index + 1 < [subviews count] && [subviews objectAtIndex:index + 1] == next)
+            if (!prev && next && index > 0 && [subviews objectAtIndex:index - 1] == next)
                return;
        }