Browse Source

Setting the same mouse cursor twice is a no-op

This fixes extremely poor event polling performance on MacOS when using
Dear ImGui, which sets the mouse cursor every frame.
Sean Ridenour 2 years ago
parent
commit
af5efadd9f
1 changed files with 5 additions and 0 deletions
  1. 5 0
      src/events/SDL_mouse.c

+ 5 - 0
src/events/SDL_mouse.c

@@ -1333,6 +1333,11 @@ void SDL_SetCursor(SDL_Cursor *cursor)
 {
     SDL_Mouse *mouse = SDL_GetMouse();
 
+    /* Return immediately if setting the cursor to the currently set one (fixes #7151) */
+    if (cursor == mouse->cur_cursor) {
+        return;
+    }
+
     /* Set the new cursor */
     if (cursor) {
         /* Make sure the cursor is still valid for this mouse */