winemac.drv: Fix redundant calls to [NSCursor set].

Since self.cursorFrames is a copy property, its pointer value will only equal frames when both are nil. Also return early from the function if the array contents are equivalent.

winemac.drv: Fix redundant calls to [NSCursor set].
8f7483d0 · Brendan Shanks · Alexandre Julliard · a6710afc · 8f7483d0
Commit 8f7483d0 authored Oct 12, 2023 by Brendan Shanks Committed by Alexandre Julliard Oct 13, 2023
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 1 deletion

cocoa_app.m dlls/winemac.drv/cocoa_app.m +1 -1

No files found.
--- a/dlls/winemac.drv/cocoa_app.m
+++ b/dlls/winemac.drv/cocoa_app.m
@@ -1048,7 +1048,7 @@ static NSString* WineLocalizedString(unsigned int stringID)

    - (void) setCursorWithFrames:(NSArray*)frames
    {
-        if (self.cursorFrames == frames)
+        if (self.cursorFrames == frames || [self.cursorFrames isEqualToArray:frames])
            return;

        self.cursorFrames = frames;